Skip to content

CC cleanups#571

Merged
jgarzik merged 4 commits intomainfrom
updates
Mar 24, 2026
Merged

CC cleanups#571
jgarzik merged 4 commits intomainfrom
updates

Conversation

@jgarzik
Copy link
Copy Markdown
Contributor

@jgarzik jgarzik commented Mar 24, 2026

No description provided.

jgarzik and others added 4 commits March 23, 2026 23:59
Replace flat MAX_CALLER_GROWTH with proportional growth limits
(GCC-style), hard caller size cap, and recursive-caller stack
estimate (LLVM-style). Fix Bug AL: symaddr-on-Arg during inlining
produced pointer-to-pointer for MEMORY-class struct params; convert
to copy instead. Update caller_size after each inline for accurate
mid-iteration decisions. Remove bugs.md (info in git history/memory).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace boilerplate new()/impl Default with #[derive(Default)] for
Module, Pseudo, PseudoId, PseudoKind, TranslationUnit, SymbolInfo,
and CrossRef. Use .or_default() instead of .or_insert_with(T::new).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Extract builtin dispatch from parse_primary_expr (1544→250 lines)
into parse_builtin_expr. Extract four groups from linearize_expr
(920→247 lines): linearize_compound_literal, linearize_va_op,
linearize_builtin, and linearize_c11_atomic. Pure mechanical
refactor, no logic changes.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Split linearize.rs (8,600→4,266 lines) into:
- linearize.rs: core, types, expressions, dispatch
- linearize_init.rs (1,094): initializer/declaration linearization
- linearize_stmt.rs (1,904): statement linearization
- linearize_emit.rs (1,399): IR emission helpers

Pure mechanical split — no logic changes. Linearizer fields
changed to pub(crate) so sibling modules can access them.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jgarzik jgarzik self-assigned this Mar 24, 2026
@jgarzik jgarzik merged commit e859f10 into main Mar 24, 2026
6 checks passed
@jgarzik jgarzik deleted the updates branch March 24, 2026 15:21
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.

1 participant