[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