diff options
| author | Matt Arsenault <Matthew.Arsenault@amd.com> | 2016-07-01 18:03:46 +0000 |
|---|---|---|
| committer | Matt Arsenault <Matthew.Arsenault@amd.com> | 2016-07-01 18:03:46 +0000 |
| commit | 105c2a204c5062cbbba1ced79cc326271f594e85 (patch) | |
| tree | 117aaa5041b5ddcf528c54208773f4191265cd39 | |
| parent | 6b560f0dd977ca600eda96e03b9edb06ccb3ab89 (diff) | |
| download | bcm5719-llvm-105c2a204c5062cbbba1ced79cc326271f594e85.tar.gz bcm5719-llvm-105c2a204c5062cbbba1ced79cc326271f594e85.zip | |
AMDGPU/SI: Enable testing several variants for si scheduler
Enable testing different scheduling variants if sgpr usage
is very high. It was previously disabled because of a bug
in handleMove, but it has been fixed since.
Patch by Axel Davy
llvm-svn: 274372
| -rw-r--r-- | llvm/lib/Target/AMDGPU/SIMachineScheduler.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Target/AMDGPU/SIMachineScheduler.cpp b/llvm/lib/Target/AMDGPU/SIMachineScheduler.cpp index cff9477ae95..8de1c4a51e2 100644 --- a/llvm/lib/Target/AMDGPU/SIMachineScheduler.cpp +++ b/llvm/lib/Target/AMDGPU/SIMachineScheduler.cpp @@ -1826,7 +1826,7 @@ void SIScheduleDAGMI::schedule() SIScheduler Scheduler(this); Best = Scheduler.scheduleVariant(SISchedulerBlockCreatorVariant::LatenciesAlone, SISchedulerBlockSchedulerVariant::BlockLatencyRegUsage); -#if 0 // To enable when handleMove fix lands + // if VGPR usage is extremely high, try other good performing variants // which could lead to lower VGPR usage if (Best.MaxVGPRUsage > 180) { @@ -1865,7 +1865,7 @@ void SIScheduleDAGMI::schedule() Best = Temp; } } -#endif + ScheduledSUnits = Best.SUs; ScheduledSUnitsInv.resize(SUnits.size()); |
