[PATCH] D46870: [MachineScheduler] Don't enforce some hazard checks pre-RA.

Javed Absar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 15 05:08:58 PDT 2018


javed.absar added inline comments.


================
Comment at: lib/CodeGen/MachineScheduler.cpp:2257
 
   // Bump the cycle count for issue group constraints.
   // This must be done after NextCycle has been adjust for all other stalls.
----------------
Probably the comment here needs updating if we are going to change behaviour. 
There is another problem - 
"Single Issue " relies on BeginGroup/EndGroup and that is not just PostRA related.


https://reviews.llvm.org/D46870





More information about the llvm-commits mailing list