[PATCH] D50496: [RISCV] Implment pseudo instructions for load/store from a symbol address.

Kito Cheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 10 07:53:10 PST 2019


kito-cheng updated this revision to Diff 181063.
kito-cheng added a comment.

Changes:

- Merge emitLoadLocalAddress/emitLoadSymbol/emitStoreSymbol
- Fix wrong operand type and syntax for floating point load instruction.

However this item is still a kind of regression:

- `Downgrading all errors to just "invalid operand for instruction" is an unfortunate regression`


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

https://reviews.llvm.org/D50496

Files:
  lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp
  lib/Target/RISCV/RISCVInstrFormats.td
  lib/Target/RISCV/RISCVInstrInfo.td
  lib/Target/RISCV/RISCVInstrInfoD.td
  lib/Target/RISCV/RISCVInstrInfoF.td
  test/MC/RISCV/rv32d-invalid.s
  test/MC/RISCV/rv32f-invalid.s
  test/MC/RISCV/rv32i-invalid.s
  test/MC/RISCV/rvi-pseudos.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50496.181063.patch
Type: text/x-patch
Size: 14890 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190110/ca84181d/attachment.bin>


More information about the llvm-commits mailing list