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

[Nemanja Ivanovic via Phabricator via llvm-commits]

nemanjai added a comment.

In D78272#2001767 <https://reviews.llvm.org/D78272#2001767>, @nemanjai wrote:

> The description mentions i32/i64 explicitly, but there does not seem to be anything in the combine that narrows it down to those two types. In fact, it appears that it will combine it for any type (scalar or vector) as long as the MULH on the narrow type is legal.


Just to be clear, I am suggesting the description should be updated to match the combine, not the other way around.


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

https://reviews.llvm.org/D78272