Skip to content

Move per-function debuginfo methods from codegen to builder methods#158560

Open
bjorn3 wants to merge 5 commits into
rust-lang:mainfrom
bjorn3:ssa_refactor_debuginfo
Open

Move per-function debuginfo methods from codegen to builder methods#158560
bjorn3 wants to merge 5 commits into
rust-lang:mainfrom
bjorn3:ssa_refactor_debuginfo

Conversation

@bjorn3

@bjorn3 bjorn3 commented Jun 29, 2026

Copy link
Copy Markdown
Member

In cg_clif the debuginfo for the function we are currently codegening is stored in FunctionDebugContext, which is stored in the FunctionCx, not in the per-cgu DebugContext.

@rustbot

rustbot commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

rustc_codegen_gcc is developed in its own repository. If possible, consider making this change to rust-lang/rustc_codegen_gcc instead.

cc @antoyo, @GuillaumeGomez

@rustbot rustbot added A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jun 29, 2026
@rustbot

rustbot commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

r? @Kivooeo

rustbot has assigned @Kivooeo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler, debuginfo
  • compiler, debuginfo expanded to 73 candidates
  • Random selection from 17 candidates

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants