Author: jlebar Date: Fri Jan 15 21:30:08 2016 New Revision: 257983 URL: http://llvm.org/viewvc/llvm-project?rev=257983&view=rev Log: Respect bound archs, even when they don't alter the toolchain.
Summary: It's possible to BindArch without changing the toolchain at all. For example, armv7 and armv7s have exactly the same toolchain triple. Therefore the code in the Driver that checks that we're not creating a job for the same Action twice needs to consider (Action, Toolchain, BoundArch) tuples. Reviewers: tra Subscribers: aemerson, echristo, beanz, cfe-commits Differential Revision: http://reviews.llvm.org/D16250 Added: cfe/trunk/test/Driver/darwin-multiarch-arm.c Modified: cfe/trunk/include/clang/Driver/Driver.h cfe/trunk/lib/Driver/Driver.cpp Modified: cfe/trunk/include/clang/Driver/Driver.h URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Driver.h?rev=257983&r1=257982&r2=257983&view=diff ============================================================================== --- cfe/trunk/include/clang/Driver/Driver.h (original) +++ cfe/trunk/include/clang/Driver/Driver.h Fri Jan 15 21:30:08 2016 @@ -380,9 +380,9 @@ public: const llvm::opt::ArgList &Args, phases::ID Phase, Action *Input) const; - /// BuildJobsForAction - Construct the jobs to perform for the - /// action \p A and return an InputInfo for the result of running \p A. - /// Will only construct jobs for a given (Action, ToolChain) pair once. + /// BuildJobsForAction - Construct the jobs to perform for the action \p A and + /// return an InputInfo for the result of running \p A. Will only construct + /// jobs for a given (Action, ToolChain, BoundArch) tuple once. InputInfo BuildJobsForAction(Compilation &C, const Action *A, const ToolChain *TC, const char *BoundArch, bool AtTopLevel, bool MultipleArchs, Modified: cfe/trunk/lib/Driver/Driver.cpp URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Driver.cpp?rev=257983&r1=257982&r2=257983&view=diff ============================================================================== --- cfe/trunk/lib/Driver/Driver.cpp (original) +++ cfe/trunk/lib/Driver/Driver.cpp Fri Jan 15 21:30:08 2016 @@ -1803,8 +1803,15 @@ InputInfo Driver::BuildJobsForAction( bool AtTopLevel, bool MultipleArchs, const char *LinkingOutput, std::map<std::pair<const Action *, std::string>, InputInfo> &CachedResults) const { - std::pair<const Action *, std::string> ActionTC = { - A, TC->getTriple().normalize()}; + // The bound arch is not necessarily represented in the toolchain's triple -- + // for example, armv7 and armv7s both map to the same triple -- so we need + // both in our map. + std::string TriplePlusArch = TC->getTriple().normalize(); + if (BoundArch) { + TriplePlusArch += "-"; + TriplePlusArch += BoundArch; + } + std::pair<const Action *, std::string> ActionTC = {A, TriplePlusArch}; auto CachedResult = CachedResults.find(ActionTC); if (CachedResult != CachedResults.end()) { return CachedResult->second; Added: cfe/trunk/test/Driver/darwin-multiarch-arm.c URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/darwin-multiarch-arm.c?rev=257983&view=auto ============================================================================== --- cfe/trunk/test/Driver/darwin-multiarch-arm.c (added) +++ cfe/trunk/test/Driver/darwin-multiarch-arm.c Fri Jan 15 21:30:08 2016 @@ -0,0 +1,18 @@ +// Check that we compile correctly with multiple ARM -arch options. +// +// RUN: %clang -target arm7-apple-darwin10 -### \ +// RUN: -arch armv7 -arch armv7s %s 2>&1 | FileCheck %s + +// CHECK: "-cc1" "-triple" "thumbv7-apple-ios5.0.0" +// CHECK-SAME: "-o" "[[CC_OUT1:[^"]*]]" +// CHECK:ld +// CHECK-SAME: "-o" "[[LD_OUT1:[^"]*]]" +// CHECK-SAME: "[[CC_OUT1]]" +// CHECK:"-cc1" "-triple" "thumbv7s-apple-ios5.0.0" +// CHECK-SAME: "-o" "[[CC_OUT2:[^"]*]]" +// CHECK:ld +// CHECK-SAME: "-o" "[[LD_OUT2:[^"]*]]" +// CHECK-SAME: "[[CC_OUT2]]" +// CHECK:lipo +// CHECK-DAG: "[[LD_OUT1]]" +// CHECK-DAG: "[[LD_OUT2]]" _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits