[PATCH] D36866: [ARM] Add PostRAScheduling option

[Matthias Braun via Phabricator via llvm-commits]

MatzeB added inline comments.


================
Comment at: llvm/trunk/lib/Target/ARM/ARMSubtarget.cpp:360-368
+  if (usePostRAScheduler())
+    return true;
+  if (SchedModel.PostRAScheduler)
+    return true;
   // No need for PostRA scheduling on subtargets where we use the
   // MachineScheduler.
   if (useMachineScheduler())
----------------
We now have 3 different ways in which the PostRA scheduler could be enabled on ARM:
- A subtarget feature.
- In the scheduling model.
- With the `if (!useMachineSched) && (!Thumb || Thumb2)` criterion.

I think just one of those options (preferably the first) would be enough.


Repository:
  rL LLVM

https://reviews.llvm.org/D36866