[PATCH] D23561: [RISCV 4/10] Add basic RISCV{InstrFormats, InstrInfo, RegisterInfo, }.td

Alex Bradbury via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 26 05:20:36 PDT 2016


asb updated this revision to Diff 69349.
asb marked 5 inline comments as done.
asb added a comment.

Changed the order of parameters in RISCVInstrFormats.td so it more closely matches the RISC-V spec. As usggested by @t.p.northover, changed the way of naming the immediate in FUJ and FSB (refer to them as imm20 and imm12 which matches the number of bits stored in the instruction). Address comment from @reames about Ri being rather terse.


https://reviews.llvm.org/D23561

Files:
  lib/Target/RISCV/CMakeLists.txt
  lib/Target/RISCV/RISCV.td
  lib/Target/RISCV/RISCVInstrFormats.td
  lib/Target/RISCV/RISCVInstrInfo.td
  lib/Target/RISCV/RISCVRegisterInfo.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23561.69349.patch
Type: text/x-patch
Size: 11092 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160826/d9c1f06a/attachment.bin>


More information about the llvm-commits mailing list