[PATCH] D34968: AMDGPU/SI: Disable PostRA scheduler with SI scheduler

Axel Davy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 3 16:21:50 PDT 2017


axeldavy created this revision.
axeldavy added a project: AMDGPU.
Herald added subscribers: t-tye, tpr, dstuttard, yaxunl, nhaehnle, wdng, kzhuravl, arsenm.

When the SI scheduler is enabled, disable the PostRA scheduler.

The PostRA scheduler currently tends to worsen the scheduler.
It could be reenabled with SI scheduler when better tuned.


Repository:
  rL LLVM

https://reviews.llvm.org/D34968

Files:
  lib/Target/AMDGPU/AMDGPUSubtarget.h


Index: lib/Target/AMDGPU/AMDGPUSubtarget.h
===================================================================
--- lib/Target/AMDGPU/AMDGPUSubtarget.h
+++ lib/Target/AMDGPU/AMDGPUSubtarget.h
@@ -673,6 +673,12 @@
     return true;
   }
 
+  // The PostRA scheduler doesn't give better results with
+  // sisched.
+  bool enablePostRAScheduler() const override {
+    return !EnableSIScheduler;
+  }
+
   void overrideSchedPolicy(MachineSchedPolicy &Policy,
                            unsigned NumRegionInstrs) const override;
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34968.105124.patch
Type: text/x-patch
Size: 528 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170703/ee873526/attachment.bin>


More information about the llvm-commits mailing list