[PATCH] D71774: [RISCV] Optimize seteq/setne pattern expansions for better code size
Sam Elliott via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 4 05:56:06 PST 2020
lenary accepted this revision.
lenary added a comment.
This revision is now accepted and ready to land.
In D71774#1856710 <https://reviews.llvm.org/D71774#1856710>, @lewis-revill wrote:
> In D71774#1821760 <https://reviews.llvm.org/D71774#1821760>, @lenary wrote:
>
> > Have you seen any regressions in code generation from this change?
>
>
> That's a fair comment, though I'd hope the situation where the result of the xor in this pattern is used for anything else later on would be extremely rare/impossible?
Ah, yes, that does make sense, given the `addi` is generated with one use.
I'm happy for this to land now!
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71774/new/
https://reviews.llvm.org/D71774
More information about the llvm-commits
mailing list