[PATCH] D61600: [DebugInfo] More precise variable range for stack locations

Jeremy Morse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 17 09:10:48 PDT 2019


jmorse accepted this revision.
jmorse added a comment.
This revision is now accepted and ready to land.

LGTM (I think I can approve?). The source IR needing no-frame-pointer-elim seems like enough reason to keep that attribute around.

As mentioned above, r360301 needs to be reverted before this is committed, and then D61940 <https://reviews.llvm.org/D61940> is needed to fix the compile time regression that will cause. Best to wait for D61940 <https://reviews.llvm.org/D61940> to be approved before committing, so that we can do that all at once.


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

https://reviews.llvm.org/D61600





More information about the llvm-commits mailing list