[PATCH] D139948: [RISCV][WIP] Add pass to remove W suffix from ADDIW and SLLIW to improve compressibility

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 13 12:23:18 PST 2022


jrtc27 added a comment.

I don't understand this, your patch subject says you're adding a pass to remove the W suffix from instructions, but the actual diff doesn't add a pass and the test changes all add W suffices



================
Comment at: llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp:2286
 // before doing this, but that would be more complicated.
 // TODO: Does this need to look through AND/OR/XOR to their users to find more
 // opportunities.
----------------
That's what you've implemented?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139948



More information about the llvm-commits mailing list