[PATCH] D83059: [RISCV] Use Generated Instruction Uncompresser

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 26 12:27:07 PST 2020


lenary updated this revision to Diff 307911.
lenary added a comment.

So, looking further into this for the failing test case:

- The patch as-was relaxed far more instructions than necessary in e.g. the example @asb notes is failing.
- I went back to using the explicit list of instructions that can be relaxed, from feedback from @jrtc27 - as the list is not likely to grow.
- This change is now probably pointless, as the uncompresser is *huge* and the list of instructions we should uncompress is all of 4.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83059

Files:
  llvm/lib/Target/RISCV/MCTargetDesc/RISCVAsmBackend.cpp
  llvm/lib/Target/RISCV/MCTargetDesc/RISCVAsmBackend.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D83059.307911.patch
Type: text/x-patch
Size: 4542 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201126/d2ab5233/attachment.bin>


More information about the llvm-commits mailing list