<div dir="ltr">Fixing up the affected tests is taking longer than I anticipated.  I'll have to finish this after vacation.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 30, 2016 at 1:01 PM, Adrian McCarthy <span dir="ltr"><<a href="mailto:amccarth@google.com" target="_blank">amccarth@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">amccarth added inline comments.<br>
<br>
<br>
> rnk wrote in CodeViewDebug.cpp:2100<br>
<span class="">> I don't think we want to repeat the compile record in every symbol subsection. We have one symbol subsection per function. MSVC only emits the compiler flags once up front, like this:<br>
><br>
>   CodeViewDebugInfo [<br>
>     Section: .debug$S (2)<br>
>     Magic: 0x4<br>
>     Subsection [<br>
>       SubSectionType: Symbols (0xF1)<br>
>       SubSectionSize: 0x5C<br>
>       ObjectName {<br>
>         Signature: 0x0<br>
>         ObjectName: C:\src\llvm\build\t.obj<br>
>       }<br>
>       CompilerFlags3 {<br>
>         Language: Cpp (0x1)<br>
>         Flags [ (0x6000)<br>
>           HotPatch (0x4000)<br>
>           SecurityChecks (0x2000)<br>
>         ]<br>
>         Machine: X64 (0xD0)<br>
>         FrontendVersion: 19.0.24210.0<br>
>         BackendVersion: 19.0.24210.0<br>
>         VersionName: Microsoft (R) Optimizing Compiler<br>
>       }<br>
>     ]<br>
>     Subsection [<br>
>       SubSectionType: Symbols (0xF1)<br>
>       SubSectionSize: 0x337<br>
>       UDT {<br>
<br>
</span>Good catch.  I'll fix that this afternoon and add a test for it.<br>
<div class="HOEnZb"><div class="h5"><br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D24317" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D24317</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>