This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Mark sendmsg hasSideEffects. NFC
ClosedPublic

Authored by Joe_Nash on May 13 2022, 10:32 AM.

Details

Summary

Address the FIXME by marking the sendmsg instructions with
hasSideEffects.

Diff Detail

Event Timeline

Joe_Nash created this revision.May 13 2022, 10:32 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 13 2022, 10:32 AM
Joe_Nash requested review of this revision.May 13 2022, 10:32 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 13 2022, 10:32 AM
foad accepted this revision.May 13 2022, 12:13 PM

LGTM since it matches the intrinsics now. Thanks!

This revision is now accepted and ready to land.May 13 2022, 12:13 PM
This revision was landed with ongoing or failed builds.May 16 2022, 7:27 AM
This revision was automatically updated to reflect the committed changes.