[PATCH] D78272: [PowerPC] DAG Combine to transform shifts into multiply-high
Sam Elliott via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 13 03:43:53 PDT 2020
lenary added a comment.
In D78272#2033067 <https://reviews.llvm.org/D78272#2033067>, @nemanjai wrote:
> Since it would seem that this can't immediately be combined with the only other target that seems to want this, I would recommend that we keep this in the PPC back end for now. If there is interest in commoning it up in the future, we revisit this then.
> Does that sound like a good plan?
> And of course, thanks for all your feedback @craig.topper @RKSimon!
I can see this combine being useful on RISC-V (where we have a `mulh[[s]u]` instruction) - would it be useful for me to work on some testcases for it?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78272/new/
https://reviews.llvm.org/D78272
More information about the llvm-commits
mailing list