[PATCH] D83468: [Debuginfo] Fix for PR46653
Adrian Prantl via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 27 18:41:16 PDT 2020
aprantl added a comment.
Thanks for posting the line table. It appears to confirm my suspicion that this patch merely hides the issue, but it isn't correct. To be sure it may be possible to preserve the debug location in this function, but implicitly setting it to the location of the instruction before it, is not the right thing to do. It may have the desired effect in your example, but it't not generally correct.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83468/new/
https://reviews.llvm.org/D83468
More information about the llvm-commits
mailing list