[PATCH] D67800: [InstCombine] Fold a shifty implementation of clamp-to-allones

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.

Thanks, LG.

I do think this is the correct canonicalization;
All those mca numbers for non-constants seem good to me, @spatel ?



================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp:2678
+  {
+    Value *X, *V;
+    const APInt *ShAmt;
----------------
Likewise, can we s/X/Y/ s/V/X/ them?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67800