diff options
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/cpp')
6 files changed, 2 insertions, 39 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCommon.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCommon.java index 1ddb027b6c..451d40020c 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCommon.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCommon.java @@ -115,7 +115,6 @@ public final class CcCommon { CppActionNames.C_COMPILE, CppActionNames.CPP_COMPILE, CppActionNames.CPP_HEADER_PARSING, - CppActionNames.CPP_HEADER_PREPROCESSING, CppActionNames.CPP_MODULE_COMPILE, CppActionNames.CPP_MODULE_CODEGEN, CppActionNames.ASSEMBLE, @@ -829,7 +828,6 @@ public final class CcCommon { // TODO(bazel-team): Remove once supports_header_parsing has been removed from the // cc_toolchain rule. unsupportedFeaturesBuilder.add(CppRuleClasses.PARSE_HEADERS); - unsupportedFeaturesBuilder.add(CppRuleClasses.PREPROCESS_HEADERS); } if (toolchain.getCcCompilationInfo().getCcCompilationContext().getCppModuleMap() == null) { unsupportedFeaturesBuilder.add(CppRuleClasses.MODULE_MAPS); diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcToolchainProvider.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcToolchainProvider.java index f8cd4d6b46..b20c61a632 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcToolchainProvider.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcToolchainProvider.java @@ -293,8 +293,7 @@ public final class CcToolchainProvider extends ToolchainInfo implements CcToolch // If parse_headers_verifies_modules is switched on, we verify that headers are // self-contained by building the module instead. return !cppConfiguration.getParseHeadersVerifiesModules() - && (featureConfiguration.isEnabled(CppRuleClasses.PREPROCESS_HEADERS) - || featureConfiguration.isEnabled(CppRuleClasses.PARSE_HEADERS)); + && featureConfiguration.isEnabled(CppRuleClasses.PARSE_HEADERS); } /** diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java index e933ae382c..f030f7cbbf 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java @@ -142,19 +142,6 @@ public class CppActionConfigs { " implies: 'compiler_output_flags'", "}", "action_config {", - " config_name: 'c++-header-preprocessing'", - " action_name: 'c++-header-preprocessing'", - " tool {", - " tool_path: '" + gccToolPath + "'", - " }", - " implies: 'legacy_compile_flags'", - " implies: 'user_compile_flags'", - " implies: 'sysroot'", - " implies: 'unfiltered_compile_flags'", - " implies: 'compiler_input_flags'", - " implies: 'compiler_output_flags'", - "}", - "action_config {", " config_name: 'c++-module-compile'", " action_name: 'c++-module-compile'", " tool {", @@ -193,7 +180,6 @@ public class CppActionConfigs { " action: 'c-compile'", " action: 'c++-compile'", " action: 'c++-header-parsing'", - " action: 'c++-header-preprocessing'", " action: 'c++-module-compile'", " action: 'c++-module-codegen'", " action: 'lto-backend'", @@ -230,7 +216,6 @@ public class CppActionConfigs { " action: 'c++-module-compile'", " action: 'objc-compile'", " action: 'objc++-compile'", - " action: 'c++-header-preprocessing'", " action: 'c++-header-parsing'", " action: 'clif-match'", " expand_if_all_available: 'dependency_file'", @@ -305,7 +290,6 @@ public class CppActionConfigs { " action: 'c-compile'", " action: 'c++-compile'", " action: 'c++-header-parsing'", - " action: 'c++-header-preprocessing'", " action: 'c++-module-compile'", " action: 'clif-match'", " flag_group {", @@ -325,7 +309,6 @@ public class CppActionConfigs { " action: 'c-compile'", " action: 'c++-compile'", " action: 'c++-header-parsing'", - " action: 'c++-header-preprocessing'", " action: 'c++-module-compile'", " action: 'clif-match'", " action: 'objc-compile'", @@ -348,7 +331,6 @@ public class CppActionConfigs { " action: 'c-compile'", " action: 'c++-compile'", " action: 'c++-header-parsing'", - " action: 'c++-header-preprocessing'", " action: 'c++-module-compile'", " action: 'clif-match'", " action: 'objc-compile'", @@ -1052,7 +1034,6 @@ public class CppActionConfigs { " action: 'c-compile'", " action: 'c++-compile'", " action: 'c++-header-parsing'", - " action: 'c++-header-preprocessing'", " action: 'c++-module-compile'", " action: 'c++-module-codegen'", " action: 'lto-backend'", @@ -1074,7 +1055,6 @@ public class CppActionConfigs { " action: 'c-compile'", " action: 'c++-compile'", " action: 'c++-header-parsing'", - " action: 'c++-header-preprocessing'", " action: 'c++-module-compile'", " action: 'c++-link-executable'", " action: 'c++-link-dynamic-library'", @@ -1104,7 +1084,6 @@ public class CppActionConfigs { " action: 'c-compile'", " action: 'c++-compile'", " action: 'c++-header-parsing'", - " action: 'c++-header-preprocessing'", " action: 'c++-module-compile'", " action: 'c++-module-codegen'", " action: 'lto-backend'", @@ -1151,7 +1130,6 @@ public class CppActionConfigs { " action: 'c++-module-codegen'", " action: 'objc-compile'", " action: 'objc++-compile'", - " action: 'c++-header-preprocessing'", " action: 'c++-header-parsing'", " action: 'lto-backend'", " expand_if_all_available: 'source_file'", @@ -1176,7 +1154,6 @@ public class CppActionConfigs { " action: 'c++-module-codegen'", " action: 'objc-compile'", " action: 'objc++-compile'", - " action: 'c++-header-preprocessing'", " action: 'c++-header-parsing'", " action: 'lto-backend'", " flag_group {", diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionNames.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionNames.java index 1f2c749a5a..6547219ff2 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionNames.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionNames.java @@ -34,8 +34,6 @@ public class CppActionNames { public static final String OBJCPP_COMPILE = "objc++-compile"; /** A string constant for the c++ header parsing. */ public static final String CPP_HEADER_PARSING = "c++-header-parsing"; - /** A string constant for the c++ header preprocessing. */ - public static final String CPP_HEADER_PREPROCESSING = "c++-header-preprocessing"; /** * A string constant for the c++ module compilation action. Note: currently we don't support C * module compilation. diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileActionBuilder.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileActionBuilder.java index 994e8ba999..b2be52f504 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileActionBuilder.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileActionBuilder.java @@ -193,13 +193,9 @@ public class CppCompileActionBuilder { if (!cppConfiguration.getParseHeadersVerifiesModules() && featureConfiguration.isEnabled(CppRuleClasses.PARSE_HEADERS)) { return CppActionNames.CPP_HEADER_PARSING; - } else if (!cppConfiguration.getParseHeadersVerifiesModules() - && featureConfiguration.isEnabled(CppRuleClasses.PREPROCESS_HEADERS)) { - return CppActionNames.CPP_HEADER_PREPROCESSING; } else { // CcCommon.collectCAndCppSources() ensures we do not add headers to - // the compilation artifacts unless either 'parse_headers' or - // 'preprocess_headers' is set. + // the compilation artifacts unless 'parse_headers' is set. throw new IllegalStateException(); } } else if (CppFileTypes.C_SOURCE.matches(sourcePath)) { diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppRuleClasses.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppRuleClasses.java index fc62f040b7..3cb9f29ea9 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppRuleClasses.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppRuleClasses.java @@ -122,11 +122,6 @@ public class CppRuleClasses { public static final String PARSE_HEADERS = "parse_headers"; /** - * A string constant for the preprocess_headers feature. - */ - public static final String PREPROCESS_HEADERS = "preprocess_headers"; - - /** * A string constant for the module_maps feature; this is a precondition to the layering_check and * header_modules features. */ |