This is an archive of the discontinued LLVM Phabricator instance.

[X86] Add test for `~x & x` -> `0` in DAG Lowering; NFC
ClosedPublic

Authored by goldstein.w.n on Mar 1 2023, 11:43 PM.

Diff Detail

Event Timeline

goldstein.w.n created this revision.Mar 1 2023, 11:43 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 1 2023, 11:43 PM
Herald added a subscriber: pengfei. · View Herald Transcript
goldstein.w.n requested review of this revision.Mar 1 2023, 11:43 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 1 2023, 11:43 PM
RKSimon accepted this revision.Mar 2 2023, 2:04 AM

LGTM - but maybe name this combine-andn.ll ? Or just add them to combine-and.ll?

This revision is now accepted and ready to land.Mar 2 2023, 2:04 AM

Move and tests, add add tests

LGTM - but maybe name this combine-andn.ll ? Or just add them to combine-and.ll?

Done (also updated with ~x + x cases in combine-add.ll as we are missing combine there as well.