[PATCH] D133739: [RISCV][WIP] Form more VW instructions
Quentin Colombet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 15 10:44:07 PDT 2022
qcolombet added a comment.
> My point about restricting that transform to one use while not restricting the variant that doesn't require the shift at all still seems reasonable
Just to react on that, although this could be a relatively easy win (see next sentence), in our motivating example it would not help because we keep the intermediate extends, while having several uses.
Going back to dropping the one-use check for the variant that completely fold, I'm unclear how we could make sure this is always a net win. We still have the register pressure issue if the extension is not completely eliminated.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133739/new/
https://reviews.llvm.org/D133739
More information about the llvm-commits
mailing list