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

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 8 11:43:22 PST 2018


dblaikie added a subscriber: uweigand.
dblaikie added a comment.

Yeah, I still need to look more closely at this - my hunch/feeling is that,
while/if this approach is necessary for dbg.value intrinsics, it'd still
produce wrong or at least suboptimal locations in general. :/ (& I'm not
even sure it's the best thing for the dbg.values either... not sure what
should be done about them - perhaps they shouldn't be merged, instead
duplicated & keep their respective locations)


Repository:
  rL LLVM

https://reviews.llvm.org/D43687





More information about the llvm-commits mailing list