[PATCH] D46118: [RISCV] AsmParser support for the li pseudo instruction

[Mario Werner via Phabricator via llvm-commits]

niosHD updated this revision to Diff 149090.
niosHD added a comment.

Updated the patch to emit ADDIW again and rebased it.

Especially for RV64, there are definitely a several missed opportunities regarding compression as can be seen by the check lines of the `rv64c-aliases-valid.s` test file. However, having a correct baseline is definitely more important than performing aggressive optimisation at the moment. Thanks again to Alex for finding the flaw in my reasoning.


https://reviews.llvm.org/D46118

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D46118.149090.patch
Type: text/x-patch
Size: 27073 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180530/98d318d1/attachment.bin>