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

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 22 20:37:50 PDT 2018


MatzeB added a comment.

The code in `PEI::replaceFrameIndices` also seems sketchy to me as in case of nested callframes it would set `InsideCallSequence` to false after the inner ADJCALLSTACKUP (though at a first glance I wasn't sure yet why that variable exists at all, and why we would not track sp adjustments outside of callframe setups...)


Repository:
  rL LLVM

https://reviews.llvm.org/D53485





More information about the llvm-commits mailing list