[PATCH] D48789: [X86] Replace (32/64 - n) shift amounts with (neg n) since the shift amount is masked in hardware

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

I’m pretty sure the case where UpdateNodeOperands returns an existing node instead of updating N doesn’t work. But I haven’t been able to trigger it with a test.


Repository:
  rL LLVM

https://reviews.llvm.org/D48789