<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Jun 6, 2018 at 1:29 PM Teresa Johnson via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tejohnson created this revision.<br>
tejohnson added a reviewer: pcc.<br>
Herald added a subscriber: inglorion.<br>
<br>
I discovered when writing the summary parsing support that the<br>
per-module index builder and writer are computing the GUID from the<br>
value name alone (ignoring the linkage type). This was ok since those<br>
GUID were not emitted in the bitcode, and there are never multiple<br>
conflicting names in a single module.<br>
<br>
However, I don't see a reason for making the GUID computation different<br>
for the per-module case. It also makes things simpler on the parsing<br>
side to have the GUID computation. </blockquote><div>"to have the GUID computation consistent." (fixed in my git log)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">So this patch changes the summary<br>
analysis phase and the per-module summary writer to compute the GUID<br>
using the facility on the GlobalValue.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D47844" rel="noreferrer" target="_blank">https://reviews.llvm.org/D47844</a><br>
<br>
Files:<br>
  include/llvm/IR/ModuleSummaryIndex.h<br>
  lib/Analysis/ModuleSummaryAnalysis.cpp<br>
  lib/Bitcode/Writer/BitcodeWriter.cpp<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><span style="font-family:Times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Teresa Johnson |</td><td style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td><td style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"> 408-460-2413</td></tr></tbody></table></span></div></div>