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

[Simon Atanasyan via Phabricator via llvm-commits]

atanasyan 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 }
----------------
Do we really need to make this fixup 64-bit wide? It's a hint for instruction relaxation so 32-bit would be enough.


Repository:
  rL LLVM

https://reviews.llvm.org/D54721