[PATCH] D54721: [mips][mc] Add basic support for R_MIPS_JALR/R_MICROMIPS_JALR

Vladimir Stefanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 20 10:10:25 PST 2018


vstefanovic marked an inline comment as done.
vstefanovic added inline comments.


================
Comment at: lib/Target/Mips/MCTargetDesc/MipsAsmBackend.cpp:503
+    { "fixup_MICROMIPS_SUB",              0,     64,   0 },
+    { "fixup_Mips_JALR",                  0,     64,   0 },
+    { "fixup_MICROMIPS_JALR",             0,     32,   0 }
----------------
atanasyan wrote:
> Do we really need to make this fixup 64-bit wide? It's a hint for instruction relaxation so 32-bit would be enough.
True.


Repository:
  rL LLVM

https://reviews.llvm.org/D54721





More information about the llvm-commits mailing list