[PATCH] [Mips] Handle -mips32r[3|5] / -mips64r[3|5] options while selecting lib/headers paths

Simon Atanasyan simon at atanasyan.com
Wed Feb 25 20:49:23 PST 2015


Thanks for review.

Closed by commit http://reviews.llvm.org/rL230611.


REPOSITORY
  rL LLVM

================
Comment at: lib/Driver/ToolChains.cpp:1786-1788
@@ -1785,3 +1785,5 @@
   addMultilibFlag(CPUName == "mips32", "march=mips32", Flags);
-  addMultilibFlag(CPUName == "mips32r2", "march=mips32r2", Flags);
+  addMultilibFlag(CPUName == "mips32r2" || CPUName == "mips32r3" ||
+                      CPUName == "mips32r5",
+                  "march=mips32r2", Flags);
   addMultilibFlag(CPUName == "mips32r6", "march=mips32r6", Flags);
----------------
dsanders wrote:
> Nit: Is this what clang-format would do? I would have expected:
> 
>   addMultilibFlag(
>       CPUName == "mips32r2" || CPUName == "mips32r3" || CPUName == "mips32r5",
>       "march=mips32r2", Flags);
At least the latest version of clang-format generates this indentation.

http://reviews.llvm.org/D7879

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list