[PATCH] D51732: [RISCV][MC] Reject bare symbols for the simm12 operand type

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 6 06:53:37 PDT 2018


asb created this revision.
asb added a reviewer: apazos.
Herald added subscribers: jocewei, PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, rogfer01, mgrang, edward-jones, zzheng, shiva0217, kito-cheng, niosHD, sabuasal, simoncook, johnrusso, rbar.

addi a0, a0, foo and lw a0, foo(a0) and similar are now rejected. An explicit %lo and %pcrel_lo modifier is required. This matches gas behaviour.


https://reviews.llvm.org/D51732

Files:
  lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp
  test/MC/RISCV/rv32d-invalid.s
  test/MC/RISCV/rv32f-invalid.s
  test/MC/RISCV/rv32i-invalid.s
  test/MC/RISCV/rv32i-valid.s
  test/MC/RISCV/rv64i-invalid.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51732.164206.patch
Type: text/x-patch
Size: 9600 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180906/c0c2e101/attachment.bin>


More information about the llvm-commits mailing list