[PATCH] D43687: Improve merging of debug locations (fixes PR 36410)
Ulrich Weigand via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 9 09:56:10 PST 2018
uweigand added a comment.
In https://reviews.llvm.org/D43687#1032868, @aprantl wrote:
> > OK. Maybe it is indeed a better choice to *not* merge dbg.value intrinsics, but just move over both sets from both sides of the "if" unchanged. In fact, if you have dbg.value intrinsics for *different* variables, they are currently just deleted -- moving them over should keep the (correct) information that *both* those variables now have the specfied value at the new location ...
>
> Keeping both intrinsics is always a safe choice.
This is now here: https://reviews.llvm.org/D44312
Repository:
rL LLVM
https://reviews.llvm.org/D43687
More information about the llvm-commits
mailing list