<div dir="ltr">Seems a bit surprising to me to implement this as on by default while it's unused and untested - I'd expect an extra flag during such an experimental phase?<br><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 6, 2017 at 12:59 PM Zachary Turner 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">zturner created this revision.<br>
Herald added subscribers: JDevlieghere, hiraditya, mgorny.<br>
<br>
This causes emission of .debug$H section unconditionally when `-gcodeview` is present.<br>
<br>
I ran some benchmarks on a self-hosted build and found a ~2% increase in overall build time (within the margin of error, so basically noise), and a ~15% increase in object file size, on average.<br>
<br>
Currently the linker doesn't use this, that will work will come in a followup patch.  For now, we're just testing that clang emits it correctly with some obj2yaml and llc based tests.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D40917" rel="noreferrer" target="_blank">https://reviews.llvm.org/D40917</a><br>
<br>
Files:<br>
  llvm/include/llvm/DebugInfo/CodeView/GlobalTypeTableBuilder.h<br>
  llvm/include/llvm/DebugInfo/CodeView/TypeHashing.h<br>
  llvm/include/llvm/MC/MCObjectFileInfo.h<br>
  llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp<br>
  llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.h<br>
  llvm/lib/DebugInfo/CodeView/CMakeLists.txt<br>
  llvm/lib/DebugInfo/CodeView/GlobalTypeTableBuilder.cpp<br>
  llvm/lib/MC/MCObjectFileInfo.cpp<br>
  llvm/lib/ObjectYAML/CodeViewYAMLTypeHashing.cpp<br>
  llvm/test/DebugInfo/COFF/global-type-hashes.ll<br>
  llvm/test/DebugInfo/COFF/globals.ll<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>