This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Rename llvm.amdgcn.msaa.load to llvm.amdgcn.msaa.load.x
ClosedPublic

Authored by critson on Mar 2 2021, 8:33 PM.

Details

Summary

While the underlying instruction is called image_msaa_load,
the resource must be x component only.
Rename the intrinsic for clarity.

Diff Detail

Event Timeline

critson created this revision.Mar 2 2021, 8:33 PM
critson requested review of this revision.Mar 2 2021, 8:33 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 2 2021, 8:33 PM
foad added a comment.Mar 2 2021, 10:21 PM

Are you aware of any front ends using the intrinsic, that would be broken by this change?

Are you aware of any front ends using the intrinsic, that would be broken by this change?

I am not aware of any front-ends using this intrinsic yet.
LLPC and Mesa do not use it.
ROCm have also indicated that they are fine with this change (I assume they are not using it either).

foad accepted this revision.Mar 2 2021, 11:57 PM

Then do we need an intrinsic at all? Looks fine to me anyway.

This revision is now accepted and ready to land.Mar 2 2021, 11:57 PM
This revision was landed with ongoing or failed builds.Mar 3 2021, 12:31 AM
This revision was automatically updated to reflect the committed changes.