[PATCH] D52299: [RISCV][MC] Accept %lo and %pcrel_lo on operands to li

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 21 02:24:52 PDT 2018


asb updated this revision to Diff 166419.
asb added a comment.

Thanks Ana, it was indeed intentional that this patch allows `li a0, %lo(1234)` but the test coverage was lacking. Updated to improve tests.


https://reviews.llvm.org/D52299

Files:
  lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp
  lib/Target/RISCV/RISCVInstrInfo.td
  test/MC/RISCV/rv32i-aliases-valid.s
  test/MC/RISCV/rv64i-aliases-valid.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52299.166419.patch
Type: text/x-patch
Size: 6220 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180921/40036d3c/attachment.bin>


More information about the llvm-commits mailing list