Skip to content

Commit

Permalink
Revert "Avoid calling to_list in _get_cc_info_linker_inputs for perfo…
Browse files Browse the repository at this point in the history
…rmance (#930) (#932)

This reverts commit 77c4afa.

We're seeing some behavior changes in tests caused by this change. I
haven't figured out the underlying issue yet, but I'm reverting this
change until we can figure it out.
  • Loading branch information
amorde authored Dec 18, 2024
1 parent 77c4afa commit 36c3efc
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions rules/framework.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -562,9 +562,10 @@ def _get_cc_info_linker_inputs(*, deps):
if not CcInfo in dep:
continue

linker_inputs.append(dep[CcInfo].linking_context.linker_inputs)
for linker_input in dep[CcInfo].linking_context.linker_inputs.to_list():
linker_inputs.append(linker_input)

return depset([], transitive = linker_inputs)
return depset(linker_inputs)

def _create_swiftmodule(attrs):
kwargs = {}
Expand Down

0 comments on commit 36c3efc

Please sign in to comment.