Skip to content

Commit

Permalink
Include argsfile field in [linker.command] JSON
Browse files Browse the repository at this point in the history
Summary:
Previously the argsfile was included in the command field of the JSON output,
but it is easier to write tests against the argsfile if it has it's own field in the produced JSON.

This diff makes that change.

Reviewed By: rmaz

Differential Revision: D59840769

fbshipit-source-id: 2986995a3f69974fcd6bd219757a7f4f2844906e
  • Loading branch information
NuriAmari authored and facebook-github-bot committed Jul 18, 2024
1 parent f6e658f commit e6c68cd
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions prelude/linking/link_info.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -983,6 +983,7 @@ def make_link_command_debug_output_json_info(ctx: AnalysisContext, debug_outputs
associated_artifacts.extend([debug_output.dist_thin_lto_codegen_argsfile, debug_output.dist_thin_lto_index_argsfile])
else:
json_info.append({
"argsfile": debug_output.argsfile,
"command": debug_output.command,
"filename": debug_output.filename,
})
Expand Down

0 comments on commit e6c68cd

Please sign in to comment.