This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Do not exapnd fp atomics on gfx940
ClosedPublic

Authored by rampitec on Feb 8 2023, 1:10 PM.

Details

Summary

FP atomics are safe on gfx940. This fixes regression after D131560.

Fixes: SWDEV-380468

Diff Detail

Event Timeline

rampitec created this revision.Feb 8 2023, 1:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 8 2023, 1:10 PM
rampitec requested review of this revision.Feb 8 2023, 1:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 8 2023, 1:10 PM
Herald added a subscriber: wdng. · View Herald Transcript
arsenm accepted this revision.Feb 8 2023, 1:13 PM

It's even safe for system scope?

This revision is now accepted and ready to land.Feb 8 2023, 1:13 PM

It's even safe for system scope?

Yes, you can read longer discussion about this in the ticket.

This revision was landed with ongoing or failed builds.Feb 8 2023, 1:22 PM
This revision was automatically updated to reflect the committed changes.