[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 14:41:52 PST 2018


uweigand added a comment.

Ah, I see.   Looks like the problem is that the DIScope -> getScope() loop can actually leave the function scope and go up all the way to file scope.  This is not a good idea here.  I think this needs to be restricted to DILocalScope instead.  I'll update the patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D43687





More information about the llvm-commits mailing list