[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