[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