[llvm] r297161 - Relax the conflicting function arg verifier to allow for inlined debug

[David Blaikie via llvm-commits]

Ah, it might be more obvious if one of 'f' or 'g' was left
standalone/non-inline, then? Or perhaps I'm not fully understanding the
conflict being tested for? Two args with the same name but different arg
objects in the same function?

OK, so I can see why that would require two inlinings, if that's the case.
Thanks for explaining!

On Tue, Mar 7, 2017 at 10:49 AM Adrian Prantl <aprantl at apple.com> wrote:

> Because the outermost function is nodebug and we need at least two
> function args to create a conflict.
>
> -- adrian
>
> On Mar 7, 2017, at 10:47 AM, David Blaikie <dblaikie at gmail.com> wrote:
>
> Thanks! Any idea why it requires both 'f' and 'g' to reproduce this?
>
> (& I imagine the return values probably aren't needed?)
>
> On Tue, Mar 7, 2017 at 10:39 AM Adrian Prantl <aprantl at apple.com> wrote:
>
> I replaced it with a smaller, hand-crafted testcase in r297176.
>
> -- adrian
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170307/cdb00cb0/attachment.html>