[PATCH] D78272: [PowerPC] DAG Combine to transform shifts into multiply-high

Wed May 6 03:44:03 PDT 2020

RKSimon added a subscriber: craig.topper.
RKSimon added a comment.

In D78272#2021795 <https://reviews.llvm.org/D78272#2021795>, @amyk wrote:

> In D78272#1997042 <https://reviews.llvm.org/D78272#1997042>, @RKSimon wrote:
>
> > Is there anything that would stop us making this a generic combine in DAGCombiner?
>
>
> Is there a preference to have this target independent instead? It may be possible if there is a preference/demand for it to be.

@craig.topper - any thoughts? Given how expensive PMULLD/PMULLQ can be, we don't much to encourage PMULH generation.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78272/new/

https://reviews.llvm.org/D78272