Page MenuHomePhabricator

[InstCombine] Test cases added for pattern "(~A & B) ^ A -> (A | B)"
AbandonedPublic

Authored by spatel on Sep 30 2020, 3:28 AM.

Details

Summary

Test cases are added for pattern "(~A & B) ^ A -> (A | B)".

And Fix for these pattern is https://reviews.llvm.org/D86395.

Diff Detail

Event Timeline

Jac1494 requested review of this revision.Sep 30 2020, 3:28 AM
Jac1494 created this revision.
Jac1494 updated this revision to Diff 295232.EditedSep 30 2020, 4:09 AM

Updated full patch to solve "content not available" error.

dexonsmith resigned from this revision.Oct 6 2020, 3:18 PM
spatel added inline comments.Oct 12 2020, 6:09 AM
llvm/test/Transforms/InstCombine/xor.ll
1182

I don't understand what this patch is proposing. Why is the output changing for an unmodified test?

RKSimon added a subscriber: RKSimon.

@Jac1494 Please can you rebase this patch against trunk - from the CHECK changes in existing tests it looks like you have some local changes as well.

spatel commandeered this revision.Oct 15 2020, 11:43 AM
spatel edited reviewers, added: Jac1494; removed: spatel.

Commandeering to close the review. I made test changes with rG77fb8cbd60cf that should provide the coverage I requested.

spatel abandoned this revision.Oct 15 2020, 11:44 AM