We had couple places which still return 10 as a maximum
occupancy. Fixed.
Also print comment about occupancy as compiler see it.
Paths
| Differential D65423
[AMDGPU] Fix high occupancy calculation and print it ClosedPublic Authored by rampitec on Jul 29 2019, 3:51 PM.
Details Summary We had couple places which still return 10 as a maximum Also print comment about occupancy as compiler see it.
Diff Detail Event TimelineHerald added subscribers: t-tye, tpr, dstuttard and 4 others. · View Herald TranscriptJul 29 2019, 3:51 PM
rampitec added inline comments.
rampitec added inline comments.
Comment Actions Removed "GCNSubtarget::getMaxWavesPerEU()" in favor of using "AMDGPUSubtarget::getMaxWavesPerEU;" This revision is now accepted and ready to land.Jul 30 2019, 5:08 PM Closed by commit rL367381: [AMDGPU] Fix high occupancy calculation and print it (authored by rampitec). · Explain WhyJul 30 2019, 6:06 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 212258 lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp
lib/Target/AMDGPU/AMDGPUSubtarget.h
lib/Target/AMDGPU/AMDGPUSubtarget.cpp
lib/Target/AMDGPU/SIMachineFunctionInfo.cpp
lib/Target/AMDGPU/SIProgramInfo.h
test/CodeGen/AMDGPU/hsa-metadata-kernel-code-props-v3.ll
test/CodeGen/AMDGPU/nsa-reassign.ll
test/CodeGen/AMDGPU/occupancy-levels.ll
test/CodeGen/AMDGPU/wave32.ll
|
Should just set a field in the constructor rather than introducing a virtual