<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 20, 2015 at 9:06 AM, Ed Maste <span dir="ltr"><<a href="mailto:emaste@freebsd.org" target="_blank">emaste@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi echristo,<br>
<br>
<a href="http://reviews.llvm.org/D8484" target="_blank">http://reviews.llvm.org/D8484</a><br>
<br>
Files:<br>
  lib/Driver/Tools.cpp<br></blockquote><div><br>Could do with a test case<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Index: lib/Driver/Tools.cpp<br>
===================================================================<br>
--- lib/Driver/Tools.cpp<br>
+++ lib/Driver/Tools.cpp<br>
@@ -3296,8 +3296,7 @@<br>
   // FIXME: Move backend command line options to the module.<br>
   // -gsplit-dwarf should turn on -g and enable the backend dwarf<br>
   // splitting and extraction.<br>
-  // FIXME: Currently only works on Linux.<br>
-  if (getToolChain().getTriple().isOSLinux() &&<br>
+  if (getToolChain().getTriple().isOSBinFormatELF() &&<br>
       Args.hasArg(options::OPT_gsplit_dwarf)) {<br>
     CmdArgs.push_back("-g");<br>
     CmdArgs.push_back("-backend-option");<br>
@@ -4616,7 +4615,7 @@<br>
   // Add the split debug info name to the command lines here so we<br>
   // can propagate it to the backend.<br>
   bool SplitDwarf = Args.hasArg(options::OPT_gsplit_dwarf) &&<br>
-    getToolChain().getTriple().isOSLinux() &&<br>
+    getToolChain().getTriple().isOSBinFormatELF() &&<br>
     (isa<AssembleJobAction>(JA) || isa<CompileJobAction>(JA) ||<br>
      isa<BackendJobAction>(JA));<br>
   const char *SplitDwarfOut;<br>
@@ -5115,9 +5114,8 @@<br>
<br>
   // Handle the debug info splitting at object creation time if we're<br>
   // creating an object.<br>
-  // TODO: Currently only works on linux with newer objcopy.<br>
   if (Args.hasArg(options::OPT_gsplit_dwarf) &&<br>
-      getToolChain().getTriple().isOSLinux())<br>
+      getToolChain().getTriple().isOSBinFormatELF())<br>
     SplitDebugInfo(getToolChain(), C, *this, JA, Args, Output,<br>
                    SplitDebugName(Args, Inputs));<br>
 }<br>
@@ -7420,9 +7418,8 @@<br>
<br>
   // Handle the debug info splitting at object creation time if we're<br>
   // creating an object.<br>
-  // TODO: Currently only works on linux with newer objcopy.<br>
   if (Args.hasArg(options::OPT_gsplit_dwarf) &&<br>
-      getToolChain().getTriple().isOSLinux())<br>
+      getToolChain().getTriple().isOSBinFormatELF())<br>
     SplitDebugInfo(getToolChain(), C, *this, JA, Args, Output,<br>
                    SplitDebugName(Args, Inputs));<br>
 }<br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div></div>