This is an archive of the discontinued LLVM Phabricator instance.

[X86] Add negative test for D145930
ClosedPublic

Authored by n-omer on Mar 13 2023, 6:22 AM.

Details

Summary

This patch adds a negative test for the issue discovered in D145930.

Diff Detail

Event Timeline

n-omer created this revision.Mar 13 2023, 6:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 13 2023, 6:22 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
n-omer requested review of this revision.Mar 13 2023, 6:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 13 2023, 6:22 AM
n-omer updated this revision to Diff 504624.Mar 13 2023, 6:24 AM

Update diff.

This comment was removed by n-omer.
pengfei added inline comments.Mar 13 2023, 6:36 AM
llvm/test/CodeGen/X86/atomic-rm-bit-test-64.ll
2 ↗(On Diff #504624)

Why adding it? Doesn't look necessary.

pengfei added inline comments.Mar 13 2023, 6:39 AM
llvm/test/CodeGen/X86/atomic-rm-bit-test-64.ll
2 ↗(On Diff #504624)

Sorry, just see the reason in D145930.

n-omer added inline comments.Mar 13 2023, 6:39 AM
llvm/test/CodeGen/X86/atomic-rm-bit-test-64.ll
2 ↗(On Diff #504624)

I tried removing it but that eliminates the # fixup A - offset: 5, value: a-5, kind: reloc_riprel_4byte line which demonstrates the issue from my test case.

RKSimon added inline comments.Mar 13 2023, 7:13 AM
llvm/test/CodeGen/X86/atomic-rm-bit-test-64.ll
1668 ↗(On Diff #504624)

If this is the only line that is being changed by D145930 - it'd be better to move this to its own pr61384.ll test file

n-omer updated this revision to Diff 504703.Mar 13 2023, 9:20 AM

Address review comments.

This revision is now accepted and ready to land.Mar 13 2023, 10:02 AM
This revision was automatically updated to reflect the committed changes.