[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.

So if we call ReplaceNode, do I still continue on to selecting that node? Which I think means we select that node earlier than its other users and prevents the users from seeing it as an 'and' that they can match in their patterns?


Repository:
  rL LLVM

https://reviews.llvm.org/D48789