[PATCH] D56694: [mips] Emit .reloc R_{MICRO}MIPS_JALR along with j(al)r(c) $25

[Simon Atanasyan via Phabricator via llvm-commits]

atanasyan added inline comments.


================
Comment at: lib/Target/Mips/MipsAsmPrinter.cpp:165
+                                   const MipsSubtarget &Subtarget) {
+  if (!(MI.isReturn() || MI.isCall() || MI.isIndirectBranch()))
+    return;
----------------
  # Let's move this `if` statement where we call `emitDirectiveRelocJalr`. So we will not call `emitDirectiveRelocJalr` for move, compare etc instructions at all.
  # If I comment out this statement no one test fail. I think we need a test checks that we do not emit `JALR` for inappropriate instructions.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56694