This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Drop the GCNRegBankReassign pass
ClosedPublic

Authored by bsaleil on Apr 26 2021, 11:14 AM.

Details

Summary

Experiments show that the GCNRegBankReassign pass significantly impacts the compilation time and there is no case for which we see any improvement in performance.
This patch removes this pass and its associated test cases from the tree.

Diff Detail

Unit TestsFailed

Event Timeline

bsaleil created this revision.Apr 26 2021, 11:14 AM
bsaleil requested review of this revision.Apr 26 2021, 11:14 AM
rampitec accepted this revision.Apr 26 2021, 11:20 AM
This revision is now accepted and ready to land.Apr 26 2021, 11:20 AM
bsaleil updated this revision to Diff 340599.Apr 26 2021, 11:41 AM

Update patch so it applies upstream.

bsaleil retitled this revision from Drop the GCNRegBankReassign pass to [AMDGPU] Drop the GCNRegBankReassign pass.Apr 26 2021, 2:01 PM
This revision was landed with ongoing or failed builds.Apr 26 2021, 2:22 PM
This revision was automatically updated to reflect the committed changes.
llvm/test/CodeGen/AMDGPU/memory_clause.ll