diff options
author | 2016-05-04 20:23:12 +0000 | |
---|---|---|
committer | 2016-05-05 19:10:12 +0000 | |
commit | 678bdeaf8ceb496cce3a3e63fbb9be6eb54c7888 (patch) | |
tree | e9228956d57993c8fe5a788ef0523f0b9fb5e447 /src/main/java/com/google/devtools | |
parent | 799a82509438572e43a9b62e19890156d721c12f (diff) |
Provide .pcm files as input to LIPO builds.
Previously, bazel would be internally inconsistent regarding whether it thought
module files were in use for LIPO builds. The merged CppCompilationContext
would not list .pcm files as inputs, because its useHeaderModules flag was
forced to false, but the CppModel would expand %module_file to a list of
.pcm files, because it thought that modules were in use. This results in
LIPO build failures due to missing files.
RELNOTES:
Fix interaction between LIPO builds and C++ header modules.
--
MOS_MIGRATED_REVID=121513092
Diffstat (limited to 'src/main/java/com/google/devtools')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompilationContext.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompilationContext.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompilationContext.java index e46452f778..691deb8c12 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompilationContext.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompilationContext.java @@ -449,8 +449,8 @@ public final class CppCompilationContext implements TransitiveInfoProvider { libContext.cppModuleMap, libContext.headerModule, libContext.picHeaderModule, - /*providesTransitiveModuleMaps=*/ false, - /*useHeaderModules=*/ false); + libContext.provideTransitiveModuleMaps, + libContext.useHeaderModules); } /** |