[PATCH] D136742: [llvm][LoongArch] Improve the "out of range" error information reported by `adjustFixupValue`, NFC

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 26 11:41:06 PDT 2022


MaskRay added inline comments.


================
Comment at: llvm/lib/Target/LoongArch/MCTargetDesc/LoongArchAsmBackend.cpp:93
+      Ctx.reportError(Fixup.getLoc(),
+                      "fixup value out of range represented by 18 bit");
     if (Value % 4)
----------------
SixWeining wrote:
> Should be "bits" or something else?
You can define a method to return a string `fixup value out of range [-xxx, xxx]`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136742



More information about the llvm-commits mailing list