This is an archive of the discontinued LLVM Phabricator instance.

[InstCombine] Baseline tests for reducing icmp x, ~x and icmp ~x, x
AcceptedPublic

Authored by tianz on Sep 5 2022, 4:07 PM.

Details

Reviewers
spatel
Summary

[InstCombine] Baseline tests for reducing icmp x, ~x and icmp ~x, x

Baseline tests for this patch https://reviews.llvm.org/D133328

Diff Detail

Event Timeline

tianz created this revision.Sep 5 2022, 4:07 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 5 2022, 4:07 PM
tianz requested review of this revision.Sep 5 2022, 4:07 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 5 2022, 4:07 PM
spatel added inline comments.Sep 6 2022, 4:48 AM
llvm/test/Transforms/InstCombine/icmp-not.ll
21

As suggested in the other review, it would be better to give each test a descriptive name. I'd remove the "_1" suffixes and replace "_2" with "_swapped" or "_commuted".

tianz updated this revision to Diff 463417.Sep 27 2022, 10:21 PM

Updated according to review comments

tianz updated this revision to Diff 463420.Sep 27 2022, 10:28 PM

Added the test to the wrong place in the previous update.

spatel accepted this revision.Sep 28 2022, 7:48 AM

LGTM - if you do not have commit access, please specify name and email to use as patch author.

llvm/test/Transforms/InstCombine/icmp-not.ll
31

Remove extra underscore after "icmp".

This revision is now accepted and ready to land.Sep 28 2022, 7:48 AM