[PATCH] D36018: Make .file directive to have basename only
Taewook Oh via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 9 20:58:37 PDT 2017
Thanks for the review. Could you please accept it from phabricator as well? Thanks!
From: Peter Collingbourne <peter at pcc.me.uk>
Date: Wednesday, August 9, 2017 at 6:27 PM
To: Rafael Avila de Espindola <rafael.espindola at gmail.com>
Cc: Teresa Johnson <tejohnson at google.com>, Mehdi AMINI <joker.eph at gmail.com>, "reviews+D36018+public+24bc47bb4db179c8 at reviews.llvm.org" <reviews+D36018+public+24bc47bb4db179c8 at reviews.llvm.org>, Taewook Oh via Phabricator <reviews at reviews.llvm.org>, Taewook Oh <twoh at fb.com>, llvm-commits <llvm-commits at lists.llvm.org>
Subject: Re: [PATCH] D36018: Make .file directive to have basename only
On Wed, Aug 9, 2017 at 6:19 PM, Rafael Avila de Espindola <rafael.espindola at gmail.com<mailto:rafael.espindola at gmail.com>> wrote:
Peter Collingbourne <peter at pcc.me.uk<mailto: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.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits