[PATCH] D36018: Make .file directive to have basename only
Peter Collingbourne via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 9 18:27:24 PDT 2017
On Wed, Aug 9, 2017 at 6:19 PM, Rafael Avila de Espindola <
rafael.espindola at gmail.com> wrote:
> Peter Collingbourne <peter at pcc.me.uk> writes:
>
> > ThinLTO uses the source filename to compute GUIDs for globals, so this
> > change will increase the likelihood of GUID collisions.
>
> Note that this change only the asm printer. Is that the code path that
> is used for computing GUID?
>
Apologies, I misunderstood your earlier mail and didn't look at the change
itself. The GUID computation is done elsewhere.
This change LGTM.
Thanks,
--
--
Peter
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170809/c3fe6c22/attachment.html>
More information about the llvm-commits
mailing list