<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 25, 2014 at 10:50 PM, David Majnemer <span dir="ltr"><<a href="mailto:david.majnemer@gmail.com" target="_blank">david.majnemer@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: majnemer<br>
Date: Fri Sep 26 00:50:45 2014<br>
New Revision: 218502<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=218502&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=218502&view=rev</a><br>
Log:<br>
llvm-vtabledump: strip trailing NUL bytes<br>
<br>
Modified:<br>
    llvm/trunk/test/tools/llvm-vtabledump/trivial.test<br>
    llvm/trunk/tools/llvm-vtabledump/llvm-vtabledump.cpp<br>
<br>
Modified: llvm/trunk/test/tools/llvm-vtabledump/trivial.test<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-vtabledump/trivial.test?rev=218502&r1=218501&r2=218502&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-vtabledump/trivial.test?rev=218502&r1=218501&r2=218502&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/tools/llvm-vtabledump/trivial.test (original)<br>
+++ llvm/trunk/test/tools/llvm-vtabledump/trivial.test Fri Sep 26 00:50:45 2014<br>
@@ -44,7 +44,7 @@ CHECK-NEXT: ??_R1A@?0A@EA@S@@8[Flags]: 6<br>
 CHECK-NEXT: ??_R1A@?0A@EA@S@@8[ClassHierarchyDescriptor]: ??_R3S@@8<br>
 CHECK-NEXT: ??_R0?AUA@@@8[VFPtr]: ??_7type_info@@6B@<br>
 CHECK-NEXT: ??_R0?AUA@@@8[AlwaysZero]: 0<br>
-CHECK-NEXT: ??_R0?AUA@@@8[MangledName]: .?AUA@@\x00<br>
+CHECK-NEXT: ??_R0?AUA@@@8[MangledName]: .?AUA@@<br>
 CHECK-NEXT: ??_R0?AUS@@@8[VFPtr]: ??_7type_info@@6B@<br>
 CHECK-NEXT: ??_R0?AUS@@@8[AlwaysZero]: 0<br>
-CHECK-NEXT: ??_R0?AUS@@@8[MangledName]: .?AUS@@\x00<br>
+CHECK-NEXT: ??_R0?AUS@@@8[MangledName]: .?AUS@@<br>
<br>
Modified: llvm/trunk/tools/llvm-vtabledump/llvm-vtabledump.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-vtabledump/llvm-vtabledump.cpp?rev=218502&r1=218501&r2=218502&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-vtabledump/llvm-vtabledump.cpp?rev=218502&r1=218501&r2=218502&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/tools/llvm-vtabledump/llvm-vtabledump.cpp (original)<br>
+++ llvm/trunk/tools/llvm-vtabledump/llvm-vtabledump.cpp Fri Sep 26 00:50:45 2014<br>
@@ -303,7 +303,9 @@ static void dumpVTables(const ObjectFile<br>
       AlwaysZero |= Data;<br>
     outs() << TDName << "[AlwaysZero]: " << AlwaysZero << '\n';<br>
     outs() << TDName << "[MangledName]: ";<br>
-    outs().write_escaped(TD.MangledName, /*UseHexEscapes=*/true) << '\n';<br>
+    outs().write_escaped(TD.MangledName.rtrim(StringRef("\0", 1)),<br>
+                         /*UseHexEscapes=*/true)<br></blockquote><div><br></div><div>I found myself using this function the other day, and I was sort of surprised that UseHexEscapes (instead of octal escapes) is not the default. Since octal string escapes have been out of style for so long, should we flip the default? It makes code using this function quite a bit less verbose.</div><div><br></div><div>-- Sean Silva</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+        << '\n';<br>
   }<br>
 }<br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div></div>