diff options
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCppSemantics.java | 23 |
1 files changed, 12 insertions, 11 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCppSemantics.java b/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCppSemantics.java index 0e1f2f63d5..9dc7fc98ba 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCppSemantics.java +++ b/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCppSemantics.java @@ -103,17 +103,6 @@ public class ObjcCppSemantics implements CppSemantics { actionBuilder.addTransitiveMandatoryInputs(objcProvider.get(STATIC_FRAMEWORK_FILE)); actionBuilder.addTransitiveMandatoryInputs(objcProvider.get(DYNAMIC_FRAMEWORK_FILE)); - ImmutableSet.Builder<Artifact> generatedHeaders = ImmutableSet.builder(); - - // TODO(b/62060839): Identify the mechanism used to add generated headers in c++, and recycle - // it here. - for (Artifact header : objcProvider.get(HEADER)) { - if (!header.isSourceArtifact()) { - generatedHeaders.add(header); - } - } - actionBuilder.addMandatoryInputs(generatedHeaders.build()); - if (isHeaderThinningEnabled) { Artifact sourceFile = actionBuilder.getSourceFile(); if (!sourceFile.isTreeArtifact() @@ -121,6 +110,18 @@ public class ObjcCppSemantics implements CppSemantics { actionBuilder.addMandatoryInputs( ImmutableList.of(intermediateArtifacts.headersListFile(sourceFile))); } + } else { + // Header thinning feature will make all generated files mandatory inputs to the + // ObjcHeaderScanning action so this is only required when that is disabled + // TODO(b/62060839): Identify the mechanism used to add generated headers in c++, and recycle + // it here. + ImmutableSet.Builder<Artifact> generatedHeaders = ImmutableSet.builder(); + for (Artifact header : objcProvider.get(HEADER)) { + if (!header.isSourceArtifact()) { + generatedHeaders.add(header); + } + } + actionBuilder.addMandatoryInputs(generatedHeaders.build()); } } |