This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU][MC][GFX10] Corrected constant bus checks to exclude null
ClosedPublic

Authored by dp on Jul 24 2019, 10:35 AM.

Details

Summary

Currently null operand is incorrectly counted against 2 scalar value limit. null should be free to use.

AMD SWDEV-157286

Diff Detail

Repository
rL LLVM

Event Timeline

dp created this revision.Jul 24 2019, 10:35 AM

Interesting question. Does null SGPR reads occupy constant bus?

Interesting question. Does null SGPR reads occupy constant bus?

I see. “Null” is an inline.

This revision is now accepted and ready to land.Aug 30 2019, 7:51 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptSep 2 2019, 7:18 AM