Update for review comments.

Author: resistor

lld/ELF/Relocations.cpp

@@ -1524,7 +1524,7 @@ void RelocationScanner::scanOne(typename Relocs<RelTy>::const_iterator &i) {
     return;
 
   RelExpr expr;
-  if (rv_vendor.empty()) {
+  if (LLVM_LIKELY(rv_vendor.empty())) {
     expr = ctx.target->getRelExpr(type, sym, sec->content().data() + offset);
   } else {
     expr = ctx.target->getVendorRelExpr(

lld/ELF/Target.h

@@ -35,6 +35,7 @@ class TargetInfo {
                              const uint8_t *loc) const = 0;
   virtual RelExpr getVendorRelExpr(RelType type, const Symbol &s,
                                    const uint8_t *loc, StringRef vendor) const {
+    Err(ctx) << "target doesn't support vendor-specific relocations";
     return R_NONE;
   };
   virtual RelType getDynRel(RelType type) const { return 0; }