[PATCH] D132818: [LoongArch] Fixup value adjustment in applyFixup

[WÁNG Xuěruì via Phabricator via llvm-commits]

xen0n added inline comments.


================
Comment at: llvm/lib/Target/LoongArch/MCTargetDesc/LoongArchAsmBackend.cpp:189-190
   MCFixupKindInfo Info = getFixupKindInfo(Kind);
-  // TODO: Apply any target-specific value adjustments.
+  if (!Value)
+    return; // Doesn't change encoding.
+  // Apply any target-specific value adjustments.
----------------
This is placed here due to review comment on D132323, does the situation change enough to make this change appropriate?


================
Comment at: llvm/lib/Target/LoongArch/MCTargetDesc/LoongArchMCCodeEmitter.cpp:266
         break;
+      case LoongArch::B:
+        FixupKind = LoongArch::fixup_loongarch_b26;
----------------
Do we want to also handle `BL` here? Given it's the only other insn of the format.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132818/new/

https://reviews.llvm.org/D132818