Skip to content

Handle all modules being serialized during LTO#880

Draft
bjorn3 wants to merge 1 commit intorust-lang:masterfrom
bjorn3:lto_all_serialized
Draft

Handle all modules being serialized during LTO#880
bjorn3 wants to merge 1 commit intorust-lang:masterfrom
bjorn3:lto_all_serialized

Conversation

@bjorn3
Copy link
Copy Markdown
Member

@bjorn3 bjorn3 commented Apr 30, 2026

No description provided.

@rustbot

This comment has been minimized.

@bjorn3 bjorn3 force-pushed the lto_all_serialized branch from 1818556 to 1959b7f Compare April 30, 2026 14:12
@bjorn3
Copy link
Copy Markdown
Member Author

bjorn3 commented Apr 30, 2026

Blocked on rust-lang/rust#156003

@antoyo
Copy link
Copy Markdown
Contributor

antoyo commented Apr 30, 2026

I forgot what's the status regarding LTO.
Can you please remind me where we're at?
Is this for ThinLTO?

@bjorn3 bjorn3 force-pushed the lto_all_serialized branch from 1959b7f to e5622d5 Compare April 30, 2026 14:43
@bjorn3
Copy link
Copy Markdown
Member Author

bjorn3 commented Apr 30, 2026

This is for fat LTO. This currently only fixes something that can happen when you combine incremental compilation with fat LTO and all codegen units are unchanged.

I want to make some changes to cg_llvm and cg_gcc that would cause this code path to be unconditionally hit for fat LTO. I don't know yet if the performance of that would be acceptable though. I wanted to land the fix in this PR first to make sure I don't regress anything in cg_gcc.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants