[PATCH] D49238: [InstCombine] add more SPFofSPF folding

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a reviewer: RKSimon.
RKSimon added inline comments.


================
Comment at: llvm/lib/Analysis/ValueTracking.cpp:4658
+      }
     }
 
----------------
Is this better?
```
    if (match(TrueVal, MaybeSExtLHS)) {
        LHS = TrueVal;
        RHS = FalseVal;
    } else {
        LHS = FalseVal;
        RHS = TrueVal;
    }
  if (CmpUsesNegatedOp)
    std::swap(LHS, RHS);
```


https://reviews.llvm.org/D49238