Skip to content

Introduce -Zdisable-incr-comp-backend-caching#157435

Open
bjorn3 wants to merge 1 commit into
rust-lang:mainfrom
bjorn3:disable_backend_incr_comp
Open

Introduce -Zdisable-incr-comp-backend-caching#157435
bjorn3 wants to merge 1 commit into
rust-lang:mainfrom
bjorn3:disable_backend_incr_comp

Conversation

@bjorn3
Copy link
Copy Markdown
Member

@bjorn3 bjorn3 commented Jun 4, 2026

This disables caching of compiled objects by the codegen backend during incremental compilation. This is useful for iterating on a codegen backend without having to run the entire rustc frontend from scratch every time when recompiling the standard library. Currently cg_clif has the CG_CLIF_DISABLE_INCR_CACHE env var that does this, but once cg_clif switches to the codegen coordinator of cg_ssa, it will no longer be possible for cg_clif to implement this. It has to be implemented in cg_ssa.

This disables caching of compiled objects by the codegen backend during
incremental compilation. This is useful for iterating on a codegen
backend without having to run the entire rustc frontend from scratch
every time when recompiling the standard library. Currently cg_clif has
the CG_CLIF_DISABLE_INCR_CACHE env var that does this, but once cg_clif
switches to the codegen coordinator of cg_ssa, it will no longer be
possible for cg_clif to implement this. It has to be implemented in
cg_ssa.
@rustbot rustbot added 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 4, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Jun 4, 2026

r? @adwinwhite

rustbot has assigned @adwinwhite.
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: codegen, compiler
  • codegen, compiler expanded to 73 candidates
  • Random selection from 19 candidates

Copy link
Copy Markdown
Member

@lqd lqd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me, albeit a bit niche, it sounds like something useful to also allow cg_clif to use more of cg_ssa.

View changes since this review

@bjorn3
Copy link
Copy Markdown
Member Author

bjorn3 commented Jun 4, 2026

@bors r=lqd

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Jun 4, 2026

📌 Commit f9f9adb has been approved by lqd

It is now in the queue for this repository.

@rust-bors rust-bors Bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

4 participants