[PATCH] D40747: [ThinLTO] Enable importing of aliases as copy of aliasee

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 6 12:53:44 PST 2017


tejohnson added a comment.

In https://reviews.llvm.org/D40747#943859, @dblaikie wrote:

> Just a quick note:
>
> Could you check what the debug info looks like for an imported alias that is inlined, and one that isn't inlined?
>
> Specifically my concern is over whether or not this causes a second CU to be created - currentyl we're threading a needle to avoid fixing/updating GDB by not needing multiple CUs per DWO file due to importing cross-CU inlining into the calling CU, etc.
>
> Can talk more offline or through this thread.


On-list update: David and I discussing offline. But essentially the same thing happens to debug info when we import an alias vs a normal function. David is looking at whether that is correct, but we are consistent. This is expected since the imported aliases (copies of aliasee) are available_externally just like other imported functions.


https://reviews.llvm.org/D40747





More information about the llvm-commits mailing list