<p dir="ltr"><br>
On Jan 28, 2014 10:23 PM, "NAKAMURA Takumi" <<a href="mailto:geek4civic@gmail.com">geek4civic@gmail.com</a>> wrote:<br>
><br>
><br>
>   Reverted in r200375. It broke many llvm and clang tests, at least in msc17 builder.<br>
><br>
><br>
> ================<br>
> Comment at: lib/CodeGen/AsmPrinter/AsmPrinter.cpp:198<br>
> @@ -195,3 +197,3 @@<br>
>    if (MAI->doesSupportDebugInformation()) {<br>
> -    DD = new DwarfDebug(this, &M);<br>
> -    Handlers.push_back(HandlerInfo(DD, DbgTimerName, DWARFGroupName));<br>
> +    if (Triple(TM.getTargetTriple()).getOS() == Triple::Win32) {<br>
> +      Handlers.push_back(HandlerInfo(new WinCodeViewLineTables(this),<br>
> ----------------<br>
> You should tweak more other tests (also in clang/test/CodeGen/ !) when you suppressed llvm::DwarfDebug.<br>
><br>
> You can reproduce such a case with LLVM_DEFAULT_TARGET_TRIPLE:=i686-pc-win32<br>
></p>
<p dir="ltr">Wondering what we think i686-pc-win32 means if not windows?</p>
<p dir="ltr">-eric<br></p>
<p dir="ltr">><br>
> <a href="http://llvm-reviews.chandlerc.com/D2232">http://llvm-reviews.chandlerc.com/D2232</a><br>
</p>