This is an archive of the discontinued LLVM Phabricator instance.

[X86][test] Add a regression test for lock cmpxchg16b on a global variable with offset
ClosedPublic

Authored by MaskRay on Oct 9 2020, 2:06 PM.

Details

Summary

Add a test for a bug (uncovered by D88808) fixed by f34bb06935aa3bab353d70d515b767fdd2f5625c.
Also delete cmpxchg16b.ll which is covered by atomic128.ll

Diff Detail

Unit TestsFailed

Event Timeline

MaskRay created this revision.Oct 9 2020, 2:06 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 9 2020, 2:06 PM
MaskRay requested review of this revision.Oct 9 2020, 2:06 PM

Thanks for the test.

llvm/test/CodeGen/X86/atomic128.ll
86

Drop dso_local and the #0?

90

Can we add nounwind to drop the cfi directives?

MaskRay updated this revision to Diff 297340.Oct 9 2020, 2:52 PM

Drop dso_local
Add nounwind

This revision is now accepted and ready to land.Oct 9 2020, 3:14 PM
This revision was landed with ongoing or failed builds.Oct 9 2020, 3:46 PM
This revision was automatically updated to reflect the committed changes.
llvm/test/CodeGen/X86/atomic128.ll