<div dir="ltr">Test case?<br><br><div class="gmail_quote"><div dir="ltr">On Fri, Sep 29, 2017 at 8:08 AM Teresa Johnson via Phabricator via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.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>
Herald added a subscriber: inglorion.<br>
<br>
Demanglers such as libiberty know how to strip suffixes of the form<br>
\.[a-zA-Z]+\.\d+, but our current promoted value suffixes are<br>
.llvm.${modulehash}, where the module hash is in hex. Change the<br>
module hash to decimal to allow demanglers to handle this.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D38405" rel="noreferrer" target="_blank">https://reviews.llvm.org/D38405</a><br>
<br>
Files:<br>
  include/llvm/IR/ModuleSummaryIndex.h<br>
<br>
<br>
Index: include/llvm/IR/ModuleSummaryIndex.h<br>
===================================================================<br>
--- include/llvm/IR/ModuleSummaryIndex.h<br>
+++ include/llvm/IR/ModuleSummaryIndex.h<br>
@@ -743,7 +743,7 @@<br>
   static std::string getGlobalNameForLocal(StringRef Name, ModuleHash ModHash) {<br>
     SmallString<256> NewName(Name);<br>
     NewName += ".llvm.";<br>
-    NewName += utohexstr(ModHash[0]); // Take the first 32 bits<br>
+    NewName += utostr(ModHash[0]); // Take the first 32 bits<br>
     return NewName.str();<br>
   }<br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div>