[PATCH] Move Post RA Scheduling flag bit into SchedMachineModel

hfinkel at anl.gov hfinkel at anl.gov
Tue Jun 24 13:50:45 PDT 2014


================
Comment at: include/llvm/Target/TargetSchedule.td:91
@@ -90,1 +90,3 @@
 
+  bit PostRAScheduler = 0;
+
----------------
This needs a comment.

================
Comment at: lib/Target/X86/X86Subtarget.cpp:364
@@ -369,1 +363,3 @@
+  CriticalPathRCs.clear();  
+  return getSchedModel()->PostRAScheduler && OptLevel >= CodeGenOpt::Default;
 }
----------------
I'd like to see the logic moved into PostRAScheduler::runOnMachineFunction (so that every target does not have to explicitly check the scheduler model) along with the call to this enablePostRAScheduler function and the corresponding command-line-argument check. The interface may not really be setup correctly for this currently, but we can change it.

http://reviews.llvm.org/D4217






More information about the llvm-commits mailing list