tomoaki
fa00ed3018
Split 1,239-line contract_checks.rs into responsibility-based modules
for better maintainability and clarity.
## Changes
1. **New file: debug_assertions.rs** (440 lines)
- 6 debug-only verification functions (panic! on violation)
- All functions guarded with #[cfg(debug_assertions)]
- Excluded from release builds
- Functions:
* verify_loop_header_phis()
* verify_exit_line()
* verify_exit_phi_no_collision()
* verify_valueid_regions()
* verify_condition_bindings_consistent()
* verify_header_phi_dsts_not_redefined()
2. **Updated: contract_checks.rs** (1,239 → 848 lines, -391 lines)
- Kept 6 Fail-Fast functions (Result<(), String>)
- Kept all 13 unit tests
- Removed debug-only functions and imports
3. **Updated: mod.rs**
- Added `mod debug_assertions;` declaration
## Responsibility Split
- **contract_checks.rs**: Fail-Fast contracts (production)
- Return errors with diagnostic messages
- Run in both debug and release builds
- **debug_assertions.rs**: Debug-only assertions (development)
- Panic on contract violations
- Excluded from release builds (#[cfg(debug_assertions)])
## Benefits
- Single Responsibility Principle (each file <850 lines)
- Clear separation: Fail-Fast vs Debug-only
- Improved maintainability (localized changes)
- Better build performance (debug code stripped in release)
## Test Results
- ✅ Build: 0 errors
- ✅ Smoke tests: 45/46 PASS (no regression)
- ❌ core_direct_array_oob_set_rc_vm: FAIL (existing known issue)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
MIR Builder (src/mir/builder/)
このディレクトリは Rust 側の MIR 生成（AST → MIR）を担う。
JoinIR の merge もここ（control_flow/joinir/merge/）が入口。
原則（SSOT / Box-First）
- 状態は Context が SSOT:
MirBuilderの状態は Context（箱）に分割され、二重管理をしない。 - ValueId 発行は SSOT: 関数内の ValueId は
MirBuilder::next_value_id()を唯一入口にする。 - 境界は Fail-Fast: JoinIR merge は
contract_checks.rsで契約違反を早期検出する（debug-only）。
Context 構成（責務マップ）
core_context.rs- ID 生成器（ValueId/BlockId/BindingId 等）と最小の共通コア状態。
type_context.rs- ValueId → 型/種別/起源（NewBox 由来など）の追跡。
scope_context.rs- lexical scope / loop/if/try のスタックと、
current_function/current_blockの実行文脈。
- lexical scope / loop/if/try のスタックと、
binding_context.rs- 変数名 ↔ BindingId の対応（shadowing の復元を含む）。
variable_context.rs- 変数解決（variable_map 等）。
metadata_context.rs- span/source_hint/region（観測）などのメタ情報。
compilation_context.rs- コンパイル全体のレジストリ（Box/型レジストリ、reserved ids 等）。
context.rs- 上記 Context を束ねる入れ物（
MirBuilderはここを介して状態へアクセスする）。
- 上記 Context を束ねる入れ物（
主要エントリポイント
- ValueId/BlockId
src/mir/builder/utils.rs（MirBuilder::next_value_id()など）
- AST → MIR の基本道
src/mir/builder/stmts.rssrc/mir/builder/exprs.rs
- JoinIR merge（契約検証を含む）
src/mir/builder/control_flow/joinir/merge/mod.rssrc/mir/builder/control_flow/joinir/merge/contract_checks.rs
追加ルール（将来の変更者向け）
- 新しい状態を追加する場合は、まず「どの Context の責務か」を決めてから追加する（
MirBuilder直下に増やさない）。 - 変更後に最低限確認する:
tools/smokes/v2/profiles/integration/apps/phase135_trim_mir_verify.sh（MIR verify の回帰防止）