[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