[PATCH] D14928: [PostRA scheduling] Allow a subtarget to do scheduling when it wants post RA

[hfinkel@anl.gov via llvm-commits]

hfinkel accepted this revision.
hfinkel added a reviewer: hfinkel.
hfinkel added a comment.
This revision is now accepted and ready to land.

LGTM.


================
Comment at: include/llvm/Target/TargetMachine.h:260
@@ +259,3 @@
+  /// scheduling opportunities.
+  virtual bool targetSchedulesPostRAScheduling() const { return false;};
+
----------------
Add space after first ;

================
Comment at: lib/Target/SystemZ/SystemZTargetMachine.cpp:19
@@ -18,2 +18,3 @@
 
+extern cl::opt<bool> MISchedPostRA;
 extern "C" void LLVMInitializeSystemZTarget() {
----------------
We generally frown on this, but as it is intended to be a temporary option, I suppose it is okay.

================
Comment at: lib/Target/SystemZ/SystemZTargetMachine.h:47
@@ +46,3 @@
+
+  bool targetSchedulesPostRAScheduling() const override { return true;};
+
----------------
Add a space after first ;


http://reviews.llvm.org/D14928