[PATCH] [DebugInfo][FastISel] Prevent using debug location from previous block for local values

Sergey Dmitrouk sdmitrouk at accesssoftek.com
Tue Jun 9 01:12:06 PDT 2015


On Mon, Jun 08, 2015 at 03:00:42PM -0700, David Blaikie wrote:
> Might be worth pinging the original thread to provide the full context or
> continue the discussion there (the mailing list archives are bad at
> navigating the full context since they chunk it up into monthly pages - so
> I can't see any replies to that patch review email, for example).

The thread is not really helpful (here is its last message:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20121001/152251.html).
Comments in the PR describe what happened better:
https://llvm.org/bugs/show_bug.cgi?id=14501

> It's a bit problematic to continue to add workarounds on top of
> workarounds - makes the code harder to maintain, etc. So there's something
> to be said for considering what the "right" approach, if any, might be and
> how much it would cost before we do this.

The response by Frédéric on this thread seems to be the most detailed
source of possible solutions:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20150518/277613.html

-- 
Sergey




More information about the llvm-commits mailing list