[PATCH] D144081: InstCombine: Fold and/or of fcmp into class

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp:1503
+  // single fcmp with fneg and fabs, that's likely a better canonical form.
+  if (LHS->hasOneUse() && RHS->hasOneUse()) {
+    auto [ClassValRHS, ClassMaskRHS] = fcmpToClassTest(PredR, RHS0, RHS1);
----------------
foad wrote:
> Do you need to worry about `IsLogicalSelect` here? (@nikic, @craig.topper?)
I've tried to come up with a case where it mattered and I can't seem to find one


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

https://reviews.llvm.org/D144081