Page MenuHomePhabricator

extend binop folds for selects to include true and false binops flag intersection
ClosedPublic

Authored by mcberg2017 on Aug 20 2018, 2:34 PM.

Diff Detail

Event Timeline

mcberg2017 created this revision.Aug 20 2018, 2:34 PM

The select_meta test has a directly related case in it that maps to the bug test case in https://bugs.llvm.org/show_bug.cgi?id=38641

spatel accepted this revision.Aug 20 2018, 2:44 PM

Add a test with FMF too since that was the motivation for rL339938?
Code change LGTM.

This revision is now accepted and ready to land.Aug 20 2018, 2:44 PM
mcberg2017 added a comment.EditedAug 20 2018, 2:47 PM

Actually the other test in rL339938 shows the positive side, where both binops are fast to show the intersection in /test/Transforms/LoopVectorize/reduction.ll

This revision was automatically updated to reflect the committed changes.