[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