[PATCH] D55277: [RISCV] Match GNU tools canonical JALR and add aliases

James Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 4 08:52:09 PST 2018


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


================
Comment at: test/CodeGen/RISCV/branch-relaxation.ll:35
 ; CHECK-NEXT:    lui a0, %hi(.LBB1_2)
-; CHECK-NEXT:    jalr zero, a0, %lo(.LBB1_2)
+; CHECK-NEXT:    jalr zero, %lo(.LBB1_2)(a0)
 ; CHECK-NEXT:  .LBB1_1: # %iftrue
----------------
Currently the `MCOperandPredicate` for `simm12` doesn't match things other than literal constants and bare symbols so our aliases don't match; in future it should be modified to match the various LO12 variant kinds.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55277





More information about the llvm-commits mailing list