Address review comments

Author: lenary

llvm/lib/Target/RISCV/MCTargetDesc/RISCVAsmBackend.cpp

@@ -699,9 +699,6 @@ bool RISCVAsmBackend::addReloc(const MCFragment &F, const MCFixup &Fixup,
       (getFixupKindInfo(Fixup.getKind()).Flags & MCFixupKindInfo::FKF_IsPCRel))
     IsResolved = isPCRelFixupResolved(Target.getAddSym(), F);
 
-  if (IsResolved && shouldForceRelocation(Fixup, Target))
-    IsResolved = false;
-
   if (!IsResolved) {
     // Some Fixups require a vendor relocation, record it (directly) before we
     // add the relocation.

llvm/lib/Target/RISCV/MCTargetDesc/RISCVAsmBackend.h

@@ -21,7 +21,6 @@ namespace llvm {
 class MCAssembler;
 class MCObjectTargetWriter;
 class raw_ostream;
-class MCSymbolELF;
 
 class RISCVAsmBackend : public MCAsmBackend {
   const MCSubtargetInfo &STI;
@@ -52,7 +51,7 @@ class RISCVAsmBackend : public MCAsmBackend {
                            uint64_t &Value) override;
 
   std::optional<StringRef>
-  getVendorIdentifierForFixup(unsigned TargetFixupKind) const;
+  getVendorIdentifierForFixup(unsigned FixupKind) const;
 
   bool addReloc(const MCFragment &, const MCFixup &, const MCValue &,
                 uint64_t &FixedValue, bool IsResolved) override;