[PATCH] D43687: Improve merging of debug locations (fixes PR 36410)

[Ulrich Weigand via Phabricator via llvm-commits]

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