[PATCH] D115224: [RISCV] Support named opcodes in .insn directive.

Nelson Chu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 13 18:43:43 PST 2021


Nelson1225 updated this revision to Diff 394106.
Nelson1225 marked 3 inline comments as done.
Nelson1225 added a comment.

For llvm/lib/Target/RISCV/RISCVInstrFormats.td:126, add extra spaces after the commas to make the name and values aligned, this should looks more readable.


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

https://reviews.llvm.org/D115224

Files:
  llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp
  llvm/lib/Target/RISCV/MCTargetDesc/RISCVBaseInfo.cpp
  llvm/lib/Target/RISCV/MCTargetDesc/RISCVBaseInfo.h
  llvm/lib/Target/RISCV/RISCVInstrFormats.td
  llvm/lib/Target/RISCV/RISCVInstrInfo.td
  llvm/test/MC/RISCV/insn-invalid.s
  llvm/test/MC/RISCV/insn.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D115224.394106.patch
Type: text/x-patch
Size: 19009 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211214/e2f486a3/attachment.bin>


More information about the llvm-commits mailing list