[PATCH] D148414: [InstCombine] Expand `foldSelectICmpAndOr` -> `foldSelectICmpAndBinOp` to work for more binops

[NAKAMURA Takumi via Phabricator via llvm-commits]

chapuni added a comment.

Thanks for the fix.



================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineSelect.cpp:694-697
+///   IF C2 u>= C1
+///     (BinOp (shl (and X, C1), C3), Y)
+///   ELSE
+///     (BinOp (lshr (and X, C1), C3), Y)
----------------
Could you update expressions in the comment please?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148414