<div dir="ltr">Hi Brad,<br><br>Here's some commit archaeology. I don't actually see any soft float support in the llvm sparc backend. Is this code actually used or needed?<div><br></div><div>-eric</div></div><br><div class="gmail_quote">On Tue, Aug 19, 2014 at 3:03 PM Brad Smith <<a href="mailto:brad@comstyle.com">brad@comstyle.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: brad<br>
Date: Tue Aug 19 16:50:15 2014<br>
New Revision: 216029<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=216029&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=216029&view=rev</a><br>
Log:<br>
Handle SPARC float command line parameters for SPARCv9.<br>
<br>
Modified:<br>
    cfe/trunk/lib/Driver/Tools.cpp<br>
    cfe/trunk/test/Driver/sparc-float.c<br>
<br>
Modified: cfe/trunk/lib/Driver/Tools.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=216029&r1=216028&r2=216029&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=216029&r1=216028&r2=216029&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/lib/Driver/Tools.cpp (original)<br>
+++ cfe/trunk/lib/Driver/Tools.cpp Tue Aug 19 16:50:15 2014<br>
@@ -1298,7 +1298,7 @@ static std::string getR600TargetGPU(cons<br>
 }<br>
<br>
 static void getSparcTargetFeatures(const ArgList &Args,<br>
-                                   std::vector<const char *> Features) {<br>
+                                   std::vector<const char *> &Features) {<br>
   bool SoftFloatABI = true;<br>
   if (Arg *A =<br>
           Args.getLastArg(options::OPT_msoft_float, options::OPT_mhard_float)) {<br>
@@ -1801,6 +1801,7 @@ static void getTargetFeatures(const Driv<br>
     getPPCTargetFeatures(Args, Features);<br>
     break;<br>
   case llvm::Triple::sparc:<br>
+  case llvm::Triple::sparcv9:<br>
     getSparcTargetFeatures(Args, Features);<br>
     break;<br>
   case llvm::Triple::aarch64:<br>
<br>
Modified: cfe/trunk/test/Driver/sparc-float.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/sparc-float.c?rev=216029&r1=216028&r2=216029&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/sparc-float.c?rev=216029&r1=216028&r2=216029&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/test/Driver/sparc-float.c (original)<br>
+++ cfe/trunk/test/Driver/sparc-float.c Tue Aug 19 16:50:15 2014<br>
@@ -5,6 +5,7 @@<br>
 // RUN: %clang -c %s -### -o %t.o 2>&1 \<br>
 // RUN:     -target sparc-linux-gnu \<br>
 // RUN:   | FileCheck --check-prefix=CHECK-DEF %s<br>
+// CHECK-DEF: "-target-feature" "+soft-float"<br>
 // CHECK-DEF: "-msoft-float"<br>
 //<br>
 // -mhard-float<br>
@@ -17,12 +18,14 @@<br>
 // RUN: %clang -c %s -### -o %t.o 2>&1 \<br>
 // RUN:     -target sparc-linux-gnu -msoft-float \<br>
 // RUN:   | FileCheck --check-prefix=CHECK-SOFT %s<br>
+// CHECK-SOFT: "-target-feature" "+soft-float"<br>
 // CHECK-SOFT: "-msoft-float"<br>
 //<br>
 // Default sparc64<br>
 // RUN: %clang -c %s -### -o %t.o 2>&1 \<br>
 // RUN:     -target sparc64-linux-gnu \<br>
 // RUN:   | FileCheck --check-prefix=CHECK-DEF-SPARC64 %s<br>
+// CHECK-DEF-SPARC64: "-target-feature" "+soft-float"<br>
 // CHECK-DEF-SPARC64: "-msoft-float"<br>
 //<br>
 // -mhard-float<br>
@@ -35,4 +38,5 @@<br>
 // RUN: %clang -c %s -### -o %t.o 2>&1 \<br>
 // RUN:     -target sparc64-linux-gnu -msoft-float \<br>
 // RUN:   | FileCheck --check-prefix=CHECK-SOFT-SPARC64 %s<br>
+// CHECK-SOFT-SPARC64: "-target-feature" "+soft-float"<br>
 // CHECK-SOFT-SPARC64: "-msoft-float"<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">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>
</blockquote></div>