[PATCH] D22726: [DAGCombine] Match shift amount by value rather than relying on common sub-expressions.

[Simon Pilgrim via llvm-commits]

RKSimon added a comment.

In https://reviews.llvm.org/D22726#518905, @bryant wrote:

> No. There are earlier combine rules that match specifically match on `iN, N > 64` that distort the `(shl (srl x, c), c)` pattern.


Are you saying that they prevent the combine from happening or that the earlier combines assert due to still using getZExtValue() ?


Repository:
  rL LLVM

https://reviews.llvm.org/D22726