This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC] Set setMaxAtomicSizeInBitsSupported appropriately for 32-bit PowerPC in PPCTargetLowering
ClosedPublic

Authored by brad on Aug 18 2020, 12:19 PM.

Details

Summary

Set setMaxAtomicSizeInBitsSupported appropriately for 32-bit PowerPC in PPCTargetLowering.

Diff Detail

Event Timeline

brad created this revision.Aug 18 2020, 12:19 PM
brad requested review of this revision.Aug 18 2020, 12:19 PM
brad updated this revision to Diff 287544.Aug 24 2020, 6:42 PM

Drop the tests I had copied and modified from X86 as it looks like the existing tests are probably good enough and update the failing tests with update_llc_test_checks.py.

brad updated this revision to Diff 287817.Aug 25 2020, 7:52 PM

Nothing changed. Just want to rebuild as an unrelated Windows test was broken.

brad added a subscriber: hfinkel.Aug 27 2020, 8:15 PM
brad retitled this revision from [PowerPC] setMaxAtomicSizeInBitsSupported appropriately for 32-bit PowerPC in PPCTargetLowering to [PowerPC] Set setMaxAtomicSizeInBitsSupported appropriately for 32-bit PowerPC in PPCTargetLowering.Aug 30 2020, 3:40 PM
brad edited the summary of this revision. (Show Details)
brad added a comment.Sep 1 2020, 3:08 PM

Any chance of a review of the auto generated tests. Do they look appropriate?

brad added a reviewer: jrtc27.Sep 6 2020, 2:40 PM
nemanjai accepted this revision.Sep 8 2020, 8:46 AM

LGTM. Sorry for the delay.

This revision is now accepted and ready to land.Sep 8 2020, 8:46 AM