[PATCH] Change clang driver to pass -target-abi option

Daniel Sanders daniel.sanders at imgtec.com
Fri Jan 30 08:08:59 PST 2015


LGTM. The passing of -target-abi ought to be generic given that the backend handling is generic but this is still good as-is since it will silence the unknown feature warnings coming from the backend and fixing that is fairly urgent.


================
Comment at: lib/Driver/Tools.cpp:4868
@@ +4867,3 @@
+void ClangAs::AddMIPSTargetArgs(const ArgList &Args,
+                              ArgStringList &CmdArgs) const {
+  StringRef CPUName;
----------------
Nit: Indentation

================
Comment at: lib/Driver/Tools.cpp:4985
@@ -4984,1 +4984,3 @@
 
+    // Add target specific flags.
+  switch(getToolChain().getArch()) {
----------------
Nit: Indentation

http://reviews.llvm.org/D6091

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






More information about the llvm-commits mailing list