Skip to content

Commit 8fd848e

Browse files
hvadehracopybara-github
authored andcommitted
Automated rollback of commit f726568.
PiperOrigin-RevId: 792634757 Change-Id: I0fe46233e78e30ce1fee7818aca8ff0c991969ba
1 parent d5821ed commit 8fd848e

File tree

2 files changed

+3
-19
lines changed

2 files changed

+3
-19
lines changed

src/main/starlark/builtins_bzl/common/cc/cc_helper_internal.bzl

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -96,8 +96,6 @@ _SHARED_LIBRARY = [".so", ".dylib", ".dll", ".wasm"]
9696
_INTERFACE_SHARED_LIBRARY = [".ifso", ".tbd", ".lib", ".dll.a"]
9797
_OBJECT_FILE = [".o", ".obj"]
9898
_PIC_OBJECT_FILE = [".pic.o"]
99-
_CPP_MODULE = [".pcm", ".gcm", ".ifc"]
100-
_CPP_MODULE_MAP = [".cppmap"]
10199

102100
_CC_AND_OBJC = []
103101
_CC_AND_OBJC.extend(_CC_SOURCE)
@@ -140,8 +138,6 @@ extensions = struct(
140138
PIC_OBJECT_FILE = _PIC_OBJECT_FILE,
141139
CC_AND_OBJC = _CC_AND_OBJC,
142140
DISALLOWED_HDRS_FILES = _DISALLOWED_HDRS_FILES, # Also includes VERSIONED_SHARED_LIBRARY files.
143-
CPP_MODULE = _CPP_MODULE,
144-
CPP_MODULE_MAP = _CPP_MODULE_MAP,
145141
)
146142

147143
artifact_category = struct(

src/main/starlark/builtins_bzl/common/cc/compile/compile.bzl

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -750,10 +750,7 @@ def _create_cc_compile_actions(
750750
category = artifact_category.INCLUDED_FILE_LIST,
751751
output_name = output_name_base,
752752
),
753-
) if (
754-
_dotd_files_enabled(language, cpp_configuration, feature_configuration) and
755-
_use_dotd_file(feature_configuration, source_artifact)
756-
) else None
753+
) if _dotd_files_enabled(language, cpp_configuration, feature_configuration) else None
757754
diagnostics_file = _get_compile_output_file(
758755
action_construction_context,
759756
label,
@@ -867,15 +864,6 @@ def _create_module_action(
867864
def _get_compile_output_file(ctx, label, *, output_name):
868865
return ctx.actions.declare_file(paths.join("_objs", label.name, output_name))
869866

870-
def _use_dotd_file(feature_configuration, source_file):
871-
extension = "." + source_file.extension if source_file.extension else ""
872-
header_discover_required = extension not in (extensions.ASSEMBLER + extensions.CPP_MODULE)
873-
use_header_modules = (
874-
feature_configuration.is_enabled("use_header_modules") and
875-
extension in extensions.CC_SOURCE + extensions.CC_HEADER + extensions.CPP_MODULE_MAP
876-
)
877-
return header_discover_required and not use_header_modules
878-
879867
def _dotd_files_enabled(language, cpp_configuration, feature_configuration):
880868
enabled_in_config = (
881869
_starlark_cc_semantics.dotd_files_enabled(cpp_configuration) if language == "cpp" else cpp_configuration.objc_should_generate_dotd_files()
@@ -892,8 +880,8 @@ def _serialized_diagnostics_file_enabled(feature_configuration):
892880
_SOURCE_TYPES_FOR_CXXOPTS = set(
893881
extensions.CC_SOURCE +
894882
extensions.CC_HEADER +
895-
extensions.CLIF_INPUT_PROTO +
896-
extensions.CPP_MODULE_MAP +
883+
[".cppmap"] + # cpp module map
884+
[".pcm", ".gcm", ".ifc"] + # clif input proto
897885
[".mm"], # objc source
898886
)
899887

0 commit comments

Comments
 (0)