Skip to content

Commit 628a698

Browse files
committed
revert-ish to filtering out any already-encountered linker input
1 parent 4c241bb commit 628a698

File tree

1 file changed

+13
-15
lines changed

1 file changed

+13
-15
lines changed

apple/internal/multi_arch_binary_support.bzl

Lines changed: 13 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -44,16 +44,17 @@ def subtract_linking_contexts(owner, linking_contexts, avoid_dep_linking_context
4444
libraries = []
4545
user_link_flags = []
4646
additional_inputs = []
47-
non_library_inputs = []
4847
linkstamps = []
4948
avoid_library_set = _build_avoid_library_set(avoid_dep_linking_contexts)
49+
linker_inputs_seen = dict()
5050

5151
for linking_context in linking_contexts:
5252
for linker_input in linking_context.linker_inputs.to_list():
53-
if not linker_input.libraries:
54-
non_library_inputs.append(linker_input)
53+
if linker_input in linker_inputs_seen:
5554
continue
5655

56+
linker_inputs_seen[linker_input] = True
57+
5758
for library_to_link in linker_input.libraries:
5859
library_artifact = apple_common.compilation_support.get_library_for_linking(library_to_link)
5960

@@ -65,17 +66,14 @@ def subtract_linking_contexts(owner, linking_contexts, avoid_dep_linking_context
6566
linkstamps.extend(linker_input.linkstamps)
6667

6768
return cc_common.create_linking_context(
68-
linker_inputs = depset(
69-
[
70-
cc_common.create_linker_input(
71-
owner = owner,
72-
libraries = depset(libraries, order = "topological"),
73-
user_link_flags = user_link_flags,
74-
additional_inputs = depset(additional_inputs),
75-
linkstamps = depset(linkstamps),
76-
),
77-
],
78-
transitive = [depset(non_library_inputs)],
79-
),
69+
linker_inputs = depset([
70+
cc_common.create_linker_input(
71+
owner = owner,
72+
libraries = depset(libraries, order = "topological"),
73+
user_link_flags = user_link_flags,
74+
additional_inputs = depset(additional_inputs),
75+
linkstamps = depset(linkstamps),
76+
),
77+
]),
8078
owner = owner,
8179
)

0 commit comments

Comments
 (0)