[PATCH] D78272: [PowerPC] DAG Combine to transform shifts into multiply-high
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 11 14:35:21 PDT 2020
RKSimon added a comment.
In D78272#2029966 <https://reviews.llvm.org/D78272#2029966>, @craig.topper wrote:
> Yes our combine will still run. I have no issue putting this in target-independent combine. I was answering from the position of why putting it in target combine doesn't get rid of X86 specific code. Which I assume was at least part of @RKSimon's motivation for moving it.
Yes this was a query as to whether it would help on top of the existing x86 specific combines.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78272/new/
https://reviews.llvm.org/D78272
More information about the llvm-commits
mailing list