<p dir="ltr">As we've discussed on llvm-dev earlier, we'll choose the debug info format based on the triple for now (to simplify things) but may add a command line option later when needed.</p>
<p dir="ltr">Are you OK if I just add a FIXME before the condition for now?</p>
<div class="gmail_quote">09 янв. 2014 г. 18:11 пользователь "Eric Christopher" <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> написал:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
  Looks pretty good, thanks.<br>
<br>
  I'd prefer you split out the MC bit when you commit, just because it's not necessary as part of this. One other comment below.<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()).isOSWindows()) {<br>
+      Handlers.push_back(HandlerInfo(new WinCodeViewLineTables(this),<br>
----------------<br>
I'm not sure this is the right conditional for this - at the very least it should be possible to continue with dwarf information on windows and this isn't going to let it. Possibly a command line option?<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2232" target="_blank">http://llvm-reviews.chandlerc.com/D2232</a><br>
</blockquote></div>