[PATCH] D69775: [Scheduling][ARM] Consistently enable PostRA Machine scheduling

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 4 03:20:34 PST 2019


samparker added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMTargetMachine.cpp:520
+
+  // Add both scheduling passes to give the subtarget an opertunity to pick
+  // between them.
----------------
nit: opportunity


================
Comment at: llvm/lib/Target/ARM/ARMTargetMachine.cpp:522
+  // between them.
+  addPass(&PostMachineSchedulerID);
+  addPass(&PostRASchedulerID);
----------------
Should we instead still skip this at -O0? 


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69775/new/

https://reviews.llvm.org/D69775





More information about the llvm-commits mailing list