[PATCH] D53485: [ScheduleDAGRRList] Do not preschedule the node has ADJCALLSTACKDOWN parent

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 2 03:24:13 PDT 2018


samparker added a comment.

The one patch I made in this area was corrected by Eli, so he's far more informed than I! The problem that time was also due to interleaving of down/up, and, as Matthias said in his email, I'm not sure why we'd need this ability. I'll take sometime today to look into the DAG builder to see if serialising these nodes isn't too much of a pain, because I'd hope that expressing this via dependencies would be better long-term.


Repository:
  rL LLVM

https://reviews.llvm.org/D53485





More information about the llvm-commits mailing list