[llvm] r302469 - Make it illegal for two Functions to point to the same DISubprogram

[Hans Wennborg via llvm-commits]

On Mon, May 8, 2017 at 2:17 PM, Adrian Prantl via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: adrian
> Date: Mon May  8 16:17:08 2017
> New Revision: 302469
>
> URL: http://llvm.org/viewvc/llvm-project?rev=302469&view=rev
> Log:
> Make it illegal for two Functions to point to the same DISubprogram
>
> As recently discussed on llvm-dev [1], this patch makes it illegal for
> two Functions to point to the same DISubprogram and updates
> FunctionCloner to also clone the debug info of a function to conform
> to the new requirement. To simplify the implementation it also factors
> out the creation of inlineAt locations from the Inliner into a
> general-purpose utility in DILocation.
>
> [1] http://lists.llvm.org/pipermail/llvm-dev/2017-May/112661.html
> <rdar://problem/31926379>
>
> Differential Revision: https://reviews.llvm.org/D32975

This caused assertions when building Chromium on Linux. See
https://bugs.llvm.org/show_bug.cgi?id=32977

I have reverted in r302533.