Hi Chad, I'm no Darwin expert but I think this change is wrong - the "driver driver" is the fat binary driver in Apple's gcc based toolchain (? It's in something non-llvm..) and this comment is saying that the -arch option acts like it. It's not a redundant "driver" token :)
Cheers, James -----Original Message----- From: [email protected] [mailto:[email protected]] On Behalf Of Chad Rosier Sent: 27 April 2012 01:36 To: [email protected] Subject: [cfe-commits] r155679 - /cfe/trunk/lib/Driver/ToolChains.cpp Author: mcrosier Date: Thu Apr 26 19:35:50 2012 New Revision: 155679 URL: http://llvm.org/viewvc/llvm-project?rev=155679&view=rev Log: Typo. Modified: cfe/trunk/lib/Driver/ToolChains.cpp Modified: cfe/trunk/lib/Driver/ToolChains.cpp URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.cpp?rev= 155679&r1=155678&r2=155679&view=diff ============================================================================ == --- cfe/trunk/lib/Driver/ToolChains.cpp (original) +++ cfe/trunk/lib/Driver/ToolChains.cpp Thu Apr 26 19:35:50 2012 @@ -886,7 +886,7 @@ DAL->AddJoinedArg(0, 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. + // how the driver works. if (BoundArch) { StringRef Name = BoundArch; const Option *MCpu = Opts.getOption(options::OPT_mcpu_EQ); _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
