Author: Craig Topper
Date: 2020-08-18T23:44:36-07:00
New Revision: b32f203edc8579d4c0023007880293c3f9404fb7

URL: 
https://github.com/llvm/llvm-project/commit/b32f203edc8579d4c0023007880293c3f9404fb7
DIFF: 
https://github.com/llvm/llvm-project/commit/b32f203edc8579d4c0023007880293c3f9404fb7.diff

LOG: [X86][Driver] Remove code that forced a core2 mtune from 
MachO::TranslateArgs.

mtune was previously ignored by the compiler so I'm not sure this
did anything. But after D85384 we're starting to support mtune
and this code is now causing a couple test failures on MacOS.

Added: 
    

Modified: 
    clang/lib/Driver/ToolChains/Darwin.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/Driver/ToolChains/Darwin.cpp 
b/clang/lib/Driver/ToolChains/Darwin.cpp
index cf8a3e6aee27..04349ff6af98 100644
--- a/clang/lib/Driver/ToolChains/Darwin.cpp
+++ b/clang/lib/Driver/ToolChains/Darwin.cpp
@@ -2271,11 +2271,6 @@ DerivedArgList *MachO::TranslateArgs(const 
DerivedArgList &Args,
     }
   }
 
-  if (getTriple().isX86())
-    if (!Args.hasArgNoClaim(options::OPT_mtune_EQ))
-      DAL->AddJoinedArg(nullptr, Opts.getOption(options::OPT_mtune_EQ),
-                        "core2");
-
   // Add the arch options based on the particular spelling of -arch, to match
   // how the driver driver works.
   if (!BoundArch.empty()) {


        
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to