4ba3fcd615
Phase 33-3.2: phi_invariants/conservative の JoinIR 側への移譲
...
実装内容:
- verify_select_minimal() 実装(Select 命令の最小 invariant チェック)
- phi_invariants.rs / conservative.rs のエッセンス抽出・移譲
- 4 テスト追加(simple/local with verify, reject multiple selects, check invariants)
移譲した責務:
- phi_invariants.rs::ensure_if_values_exist() → 型一貫性・完全性チェック
- conservative.rs::ConservativeMerge → 単一 PHI チェック
テスト結果:
- ✅ 8/8 tests PASS
- ✅ simple/local パターン: Verifier PASS
- ✅ 不正パターン(複数 Select): Verifier REJECT
- ✅ invariant ログ: conservative.rs / phi_invariants.rs からの移譲を明記
制限事項:
- IfSelectTest.* のみ対象、本線 if_phi は保持
- 削除は Stage-1/Stage-B/selfhost への適用後に行う
Phase 33-3.2 完了(2025-11-27)
2025-11-27 03:55:45 +09:00
f096a2a588
docs(phase33): Phase 33-3.1 - IfSelectTest.* JoinIR coverage documentation
...
Documentation update (code-unchanged):
- Add Phase 33-3.1 section to CURRENT_TASK.md
- Document JoinIR coverage range (simple/local patterns)
- Note constraints (dev-only, function name guard, mainline preserved)
- Document deletion plan (~600 lines: if_phi/phi_invariants/conservative)
Next steps: Phase 33-3.2 (JoinIR migration), Phase 33-3.3 (deletion)
🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-27 03:39:09 +09:00
0c252406ef
docs(joinir): L-4.3b - Phase 40 role transfer documentation
...
## L-4.3b 完了内容
Phase 32 scope clarification:
- llvmlite harness JoinIR experiment (L-4.3a) ✅ completed
- ny-llvmc JoinIR integration → deferred to Phase 40
## 方針明文化
**Phase 32 では**:
- ny-llvmc 側のコード変更なし
- JoinIR→MIR' 変換は Rust 本体 (llvm.rs) で実施
- ny-llvmc は「post-JoinIR MIR JSON」を受け取るだけ
**Phase 40 (LLVM Native Backend) で**:
- ny-llvmc 本体の最適化・native backend 化と併せて
- JoinIR 経路の統合を行う
## ドキュメント更新
- Phase 32 TASKS.md: L-4.3b 完了マーク + Phase 40 移管明記
- CURRENT_TASK.md: 1-00w セクション追加
関連: Phase 32 L-4.3a (llvmlite 実証完了)
2025-11-26 16:12:48 +09:00
b8893787dc
feat(joinir): L-5.3 Phase 1 - progress carrier guard for generic_case_a
...
## 実装内容
1. has_safe_progress() helper 追加 (loop_to_join.rs:186-195)
- Phase 1: scope.progress_carrier.is_some() をチェック (保守的)
- Phase 2 (future): MirQuery で Add 命令チェック予定
2. is_supported_case_a_loop_view() に progress guard 追加 (256-266)
- 無限ループの可能性があるループを事前にフォールバック
- デバッグログで reject 理由を出力
## テスト結果
✅ 25 passed; 0 failed - JoinIR 関連テスト全通過
✅ skip_ws / trim / append_defs / Stage‑1 UsingResolver 全ケース PASS
✅ 既存テストへの影響なし
## 技術メモ
- 保守的アプローチ: progress_carrier.is_some() のみチェック
- LoopScopeShape で progress_carrier を carriers の先頭として設定済み
- ignored テスト失敗は MIR 自体の PHI バグで、本変更とは無関係 (git stash で確認済み)
関連: Phase 29 L-5.3 (TASKS.md), CURRENT_TASK.md 1-00v
2025-11-26 16:08:49 +09:00
bde83f311e
docs(joinir): L-2.3 PHI問題のA/B実行実証
...
- CURRENT_TASK.md に L-2.3 進捗を追記
- docs/private サブモジュールに Phase 32 を追加
Route A vs Route B 比較結果:
- Route A (VM直接): Main.skip(" abc") → 0 (PHI バグ)
- Route B (JoinIR): Main.skip(" abc") → 3 (正解)
JoinIR が PHI 問題を設計的に解決していることを実行レベルで実証。
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-26 12:28:32 +09:00
ffd49ece91
docs(joinir): L-2.2 Step-5 - Verify Stage-B stability with generic ON
...
Verified NYASH_JOINIR_LOWER_GENERIC=1 does not break Stage-B:
- generic OFF: joinir_json 10 PASS, joinir_stageb 6 PASS
- generic ON: joinir_json 10 PASS, joinir_stageb 6 PASS,
auto_lowering 2 PASS
- No guard additions needed - existing code is stable
- Stage-B execution remains on VM Route A (JoinIR as canary only)
L-2.2 is now fully complete (Step-1 through Step-5).
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-26 12:02:29 +09:00
eddc4e6035
docs(joinir): L-2.2 Step-4 - Update JoinIR VM bridge documentation
...
Phase 32 README:
- Expand VM Bridge table with "JoinIR 利用範囲" column
- Add L-2.2 progress summary table (Step-1~5 status)
- Add developer notes for Stage-B JoinIR observation
environment-variables.md:
- Add detailed descriptions for NYASH_JOINIR_* variables
- Clarify scope and limitations (Stage-B = lowering only)
- Add Stage-B JoinIR observation example
CURRENT_TASK:
- Update L-2.2 status to Step-1~4 complete
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-26 11:55:04 +09:00
51ff558904
feat(phase32): L-2.1 Stage-1 UsingResolver JoinIR integration + cleanup
...
Phase 32 L-2.1 complete implementation:
1. Stage-1 UsingResolver main line JoinIR connection
- CFG-based LoopForm construction for resolve_for_source/5
- LoopToJoinLowerer integration with handwritten fallback
- JSON snapshot tests 6/6 PASS
2. JoinIR/VM Bridge improvements
- Simplified join_ir_vm_bridge.rs dispatch logic
- Enhanced json.rs serialization
- PHI core boxes cleanup (local_scope_inspector, loop_exit_liveness, loop_var_classifier)
3. Stage-1 CLI enhancements
- Extended args.rs, groups.rs, mod.rs for new options
- Improved stage1_bridge module (args, env, mod)
- Updated stage1_cli.hako
4. MIR builder cleanup
- Simplified if_form.rs control flow
- Removed dead code from loop_builder.rs
- Enhanced phi_merge.rs
5. Runner module updates
- json_v0_bridge/lowering.rs improvements
- dispatch.rs, selfhost.rs, modes/vm.rs cleanup
6. Documentation updates
- CURRENT_TASK.md, AGENTS.md
- Various docs/ updates
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-26 10:17:37 +09:00
7a29ebd268
refactor(phase31): Connect 4 lowerers to LoopToJoinLowerer
...
Phase 31 Step 2 complete:
- skip_ws.rs: use LoopToJoinLowerer::lower_minimal_skip_ws_case_a()
- funcscanner_trim.rs: use LoopToJoinLowerer::lower_minimal_trim_case_a()
- funcscanner_append_defs.rs: use LoopToJoinLowerer::lower_minimal_append_defs_case_a()
- stage1_using_resolver.rs: use LoopToJoinLowerer::lower_minimal_stage1_case_a()
Eliminates direct generic_case_a calls, routing through unified box.
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-26 02:36:43 +09:00
a898ff3f83
refactor(joinir): Phase 30 F-2.0/F-3 - PHI箱インベントリと旧APIレガシー削除
...
F-3 レガシー削除:
- generic_case_a.rs: 旧API関数4個削除(_with_scope 移行完了)
- loop_scope_shape.rs: CaseAContext::new() 削除(from_scope() に統一)
- mod.rs: 不要な pub use 削除
- #[allow(dead_code)] 除去(5関数)
- 未使用import削除(コード削減約150行)
F-2.0 PHI箱インベントリ:
- PHI_BOX_INVENTORY.md 作成: 13箱+11補助構造体の棚卸し
- 削除順ポリシー: 早期/中期/最終の3段階
- TASKS.md/CURRENT_TASK.md 更新
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-25 23:25:39 +09:00
f41564723d
docs: Phase 30 F-1/F-3/F-4.4 preparation survey
...
Task A/B/C results:
- Task A: Lowerer files Case A analysis (6 files surveyed)
- Found: all lowerers use empty LoopVarClassBox/LoopExitLivenessBox
- stageb_body/stageb_funcscanner have placeholders only
- Task B: Add VM runner shrinkage plan to TASKS.md F-4.4
- Task C: Add bridge summary to CURRENT_TASK.md
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-25 15:30:54 +09:00
03efa13e76
docs(phase-29): JoinIR/Stage-1 環境変数棚卸し完了
...
Step 1: ENV_INVENTORY.md 作成 (docs/private submodule)
Step 2: 実装統一確認 - 既にhelper経由で統一済み
Step 3: environment-variables.md に JoinIRセクション追加
Step 4: スモークテスト pass
Step 5: CURRENT_TASK.md に完了記録
発見事項:
- JoinIR: env_flag_is_1() ヘルパー経由で統一済み
- Stage-1: src/config/env/stage1.rs SSOT モジュール経由で統一済み
- NYASH_RUN_JOINIR_MINIMAL は既に削除済み
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-25 09:03:28 +09:00
466e636af6
Span trace utilities and runner source hint
2025-11-24 14:17:02 +09:00
a8555e67d5
feat(joinir): S-5.2完了 - BoxCall → VM method_router 経由実装
...
実装内容:
- box_to_join_value() ヘルパー関数追加(VM Box → JoinValue 変換)
- StringBox.length/substring を VM 実装経由で呼び出し(hardcoded削除)
- safe_substring() 削除(不要になった)
技術的成果:
- ガードレール設計実現: 制御フローは JoinIR Runner、Box実装は VM に委譲
- VM 2号機 回避: Box実装の重複なし
- テスト全 PASS: joinir_runner_standalone_skip_ws/trim 両方成功
Phase 27-shortterm S-5.2 完了 ✅
2025-11-24 08:01:56 +09:00
b7c7e48526
feat(joinir): Phase 27.14 - FuncScannerBox._append_defs JoinIR lowering完了 + コード品質改善
...
## Phase 27.14: FuncScannerBox._append_defs/2 JoinIR lowering
- **新規実装**: `funcscanner_append_defs.rs` (322行)
- Shared Builder Pattern採用
- MIR-based lowering with CFG sanity checks
- ValueId range 9000-10999 割り当て
- **テスト**: `mir_joinir_funcscanner_append_defs.rs` (3テスト)
- type_sanity, empty_module_returns_none, auto_lowering (ignored)
- **最小.hako**: `funcscanner_append_defs_minimal.hako`
## コード品質改善 (5項目完了)
1. **CFG Sanity Checks強化** (`common.rs`)
- `has_array_method()`: ArrayBox操作検出
- `has_loop_increment()`: i+1パターン検出
2. **ValueIdテスト自動化** (`value_id_ranges.rs`)
- マクロ化 + 自動overlap検証で30→15行に削減
3. **モジュール名統一確認** (作業不要、既に統一済み)
4. **Shared Builder命名統一** (`funcscanner_trim.rs`)
- `build_trim_joinir` → `build_funcscanner_trim_joinir`
5. **全テストPASS確認**
- value_id_ranges, funcscanner_trim, funcscanner_append_defs全てPASS ✅
## 効果
- CFG検証関数: 1個 → 3個 (200%↑)
- テストコード: 50%削減 (保守性向上)
- 命名一貫性: 75% → 100%
- ビルド成功率: 100%維持
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-24 05:23:26 +09:00
3d5979c78e
refactor(joinir): Phase 27.9 - Modular separation of join_ir.rs into directory structure
...
Phase 27.9 で join_ir.rs (~1,336行) を以下のモジュール構造に分離:
## 新規ディレクトリ構造:
```
src/mir/join_ir/
├── mod.rs # 型定義・共通ユーティリティ (~330行)
└── lowering/
├── mod.rs # lowering インターフェース
├── min_loop.rs # lower_min_loop_to_joinir (~140行)
├── skip_ws.rs # skip_ws lowering 3関数 (~390行)
└── funcscanner_trim.rs # trim lowering (~480行)
```
## 技術的変更:
- **型定義統一**: JoinFuncId, JoinInst, JoinModule 等を mod.rs に集約
- **lowering 分離**: 3つの lowering 関数を個別モジュールに移動
- **後方互換性**: pub use で lowering 関数を re-export(既存コード影響なし)
- **削除**: src/mir/join_ir.rs (旧単一ファイル)
## テスト結果:
- **385 passed** (+1 from 384)
- **9 failed** (-1 from 10)
- **ビルド成功**: 0 errors, 18 warnings (変化なし)
## 効果:
- **保守性向上**: 1,336行 → 4ファイル(各300-500行)で可読性向上
- **モジュール境界明確化**: 型定義 vs lowering 実装の責務分離
- **将来の拡張容易**: 新 lowering 関数追加が簡単に
Phase 27.8 で実装した MIR 自動解析 lowering の基盤整備完了。
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-23 16:49:49 +09:00
4fd74f2a6e
feat(phi): Phase 27.5 - JoinIR Exit φ 統合(LoopExitShape 雛形)
...
LoopExitShape 構造体を追加し、Exit φ の意味を JoinIR 側に固定:
- LoopExitShape 追加 (src/mir/join_ir.rs:84-111)
- exit_args: Vec<ValueId> で Exit φ の意味を表現
- minimal (exit_args=[i]), trim (exit_args=[e], Option A)
- #[allow(dead_code)] で Phase 27.6 まで設計専用
- Exit φ コメント追加
- lower_skip_ws_to_joinir: 2箇所の exit パスに意味明記
- lower_funcscanner_trim_to_joinir: Option A として意味明記
- テストコメント更新
- mir_joinir_skip_ws.rs: Exit φ (i の合流) 検証を明記
- mir_joinir_funcscanner_trim.rs: Exit φ (e の合流+substring) を明記
- ドキュメント更新
- IMPLEMENTATION_LOG.md: Phase 27.5 セクション追加
- TASKS.md: Phase 27.5 完了マーク
ExitPhiBuilder は Phase 27.6 まで保留。本線影響ゼロ。
2025-11-23 11:03:38 +09:00
8750186e55
chore: Phase 26-H セッション完了 - 全ドキュメント更新
...
Phase 26-H 完了内容:
✅ JoinIR 型定義実装(src/mir/join_ir.rs)
✅ MIR → JoinIR 自動変換実装(lower_min_loop_to_joinir)
✅ 自動変換テスト実装(mir_joinir_min_auto_lowering)
✅ PHI/Loop箱 → JoinIR 移行対応表追加(loopform_ssot.md)
ドキュメント更新:
- Phase 27 JoinIR タスク計画追加
- Phase 26-H タスク完了記録
- 各種 README 更新(進捗反映)
- CURRENT_TASK.md 更新
コミット統計: $(git status --short | wc -l) files changed
次のステップ: Phase 27 一般化 MIR → JoinIR 変換
2025-11-23 05:53:27 +09:00
f2bb07b542
refactor(mir): レガシーコード削除 - JoinIR準備整理
...
## 削除内容
- **src/mir/builder_modularized/control_flow.rs**: 削除
- JoinIR実装準備のため整理
- 使用されていないレガシーモジュール
## 修正内容
- **src/mir/loop_builder.rs**: 軽微な整理
- **CURRENT_TASK.md**: Phase 26-H進捗更新
## 影響範囲
- ✅ 既存パイプライン無影響
- ✅ テスト全パス維持
- ✅ JoinIR実装準備完了
🌟 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-23 04:35:13 +09:00
2692eafbbf
feat(mir): Phase 26-H JoinIR型定義実装完了 - ChatGPT設計
...
## 実装内容(Step 1-3 完全達成)
### Step 1: src/mir/join_ir.rs 型定義追加
- **JoinFuncId / JoinContId**: 関数・継続ID型
- **JoinFunction**: 関数(引数 = φノード)
- **JoinInst**: Call/Jump/Ret/Compute 最小命令セット
- **MirLikeInst**: 算術・比較命令ラッパー
- **JoinModule**: 複数関数保持コンテナ
- **単体テスト**: 型サニティチェック追加
### Step 2: テストケース追加
- **apps/tests/joinir_min_loop.hako**: 最小ループ+breakカナリア
- **src/tests/mir_joinir_min.rs**: 手書きJoinIR構築テスト
- MIR → JoinIR手動構築で型妥当性確認
- #[ignore] で手動実行専用化
- NYASH_JOINIR_EXPERIMENT=1 トグル制御
### Step 3: 環境変数トグル実装
- **NYASH_JOINIR_EXPERIMENT=1**: 実験モード有効化
- **デフォルト挙動**: 既存MIR/LoopForm経路のみ(破壊的変更なし)
- **トグルON時**: JoinIR手書き構築テスト実行
## Phase 26-H スコープ遵守
✅ 型定義のみ(変換ロジックは未実装)
✅ 最小限の命令セット
✅ Debug 出力で妥当性確認
✅ 既存パイプライン無影響
## テスト結果
```
$ NYASH_JOINIR_EXPERIMENT=1 cargo test --release mir_joinir_min_manual_construction -- --ignored --nocapture
[joinir/min] MIR module compiled, 3 functions
[joinir/min] JoinIR module constructed:
[joinir/min] ✅ JoinIR型定義は妥当(Phase 26-H)
test result: ok. 1 passed; 0 failed
```
## JoinIR理論の実証
- **φノード = 関数引数**: `fn loop_step(i, k_exit)`
- **merge = join関数**: 分岐後の合流点
- **ループ = 再帰関数**: `loop_step` 自己呼び出し
- **break = 継続呼び出し**: `k_exit(i)`
## 次フェーズ (Phase 27.x)
- LoopForm v2 → JoinIR 自動変換実装
- break/continue ハンドリング
- Exit PHI の JoinIR 引数化
🌟 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
Co-Authored-By: ChatGPT <noreply@openai.com >
2025-11-23 04:10:12 +09:00
948f22a03a
feat(phi): Phase 26-F-2 - 箱理論による責務分離(IfBodyLocalMergeBox新設)
...
**箱理論による問題解決**:
- ❌ 問題: LoopVarClassBox(ループスコープ分析)とif-merge処理が混在
- ✅ 解決: if-merge専用箱を新設して責務分離
**新箱: IfBodyLocalMergeBox**:
- 責務: if-merge専用のbody-local φ候補決定
- ロジック:
- 両腕に存在する変数を検出
- pre_ifと比較して値が変わった変数のみ
- empty elseは空リスト返す
- 特徴: LocalScopeInspector不要、LoopVarClassBox不使用
**変更ファイル**:
- src/mir/phi_core/if_body_local_merge.rs: 新規作成(IfBodyLocalMergeBox)
- src/mir/phi_core/phi_builder_box.rs: IfBodyLocalMergeBox使用に切り替え
- src/mir/phi_core/body_local_phi_builder.rs: filter_if_merge_candidates()削除
- src/mir/loop_builder.rs: BodyLocalPhiBuilder setup削除
- src/mir/phi_core/mod.rs: if_body_local_merge追加
**テスト結果**:
- Passed: 353→354 (+1) ✅
- Failed: 14→14 (退行なし)
**既知の問題**:
- domination error依然残存(%48 in bb48 from bb52)
- 次フェーズで調査・修正予定
技術詳細:
- ChatGPT箱理論分析による設計
- A案ベースのシンプル実装
- 責務明確化: ループスコープ分析 vs if-merge専用処理
2025-11-22 11:03:21 +09:00
fc16130d6b
feat(phi): Phase 26-E-4 - variable_map リセットタイミング修正
...
**問題**:
- loop_builder.rs の lower_if_in_loop で PHI生成**前**に variable_map を pre_if_var_map にリセット
- else ブロック内で定義された変数が消失 → domination error 発生
- エラー: `Value %48 (obj_end) used in block bb54 but defined in non-dominating block bb52`
**修正内容**:
- 1053行, 1129行削除: PHI生成前の variable_map リセット(早すぎる)
- 1155-1159行追加: PHI生成**後**に variable_map リセット
- 理由: else_var_map_end_opt が正しい snapshot を保持したまま PHI 生成に渡す必要がある
**結果**:
- 決定性100%達成(3回実行で一貫したエラー)
- domination error 部分改善: bb52→bb54 から bb53→bb55 に変化
- 残課題: bb53 (break先) → bb55 (merge) の PHI 問題(別途対応予定)
**指示元**: ChatGPT + Task先生
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-22 08:41:14 +09:00
b9a034293d
feat(phi): Phase 26-E-2 - PhiBuilderBox If PHI生成完全実装
...
Phase 26-E Phase 2 完了: PhiBuilderBox による If PHI生成SSOT統一化
**実装内容:**
1. PhiBuilderBox 作成 (444行)
- If PHI生成: generate_if_phis() 完全実装
- Conservative戦略: void emission 含む完全対応
- 決定的順序: BTreeSet/BTreeMap で非決定性排除
2. PhiBuilderOps trait (7メソッド)
- 最小PHI生成インターフェース
- new_value, emit_phi, update_var, get_block_predecessors
- emit_void, set_current_block, block_exists
3. loop_builder.rs 統合
- PhiBuilderOps trait 実装 (Ops構造体)
- If PHI呼び出し箇所統合 (line 1136-1144)
- Legacy if_phi::merge_modified_with_control 置換完了
**技術的成果:**
- Conservative PHI生成: 全経路カバー + void fallback
- 決定的変数順序: BTreeSet で変更変数をソート
- 決定的PHI入力順序: pred_bb.0 でソート
- テスタビリティ: MockOps でユニットテスト可能
**Phase 3 設計方針 (ChatGPT提案):**
- trait 階層化: LoopFormOps: PhiBuilderOps
- blanket impl: impl<T: LoopFormOps> PhiBuilderOps for T
- PhiBuilderBox: PhiBuilderOps 最小セットのみに依存
- 段階的移行: 既存コード保護しながら統一化
**削減見込み:**
- Phase 2: -80行 (If側重複削除)
- Phase 4: -287行 (loop_phi.rs Legacy削除)
- 合計: -367行 (純削減)
**関連ファイル:**
- src/mir/phi_core/phi_builder_box.rs (新規, 444行)
- src/mir/phi_core/mod.rs (module登録)
- src/mir/loop_builder.rs (PhiBuilderOps実装)
- CURRENT_TASK.md (Phase 26-E記録)
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-22 07:05:21 +09:00
7812c3d4c1
feat(phi): Phase 25.1 - BTreeMap移行 (21ファイル、80%決定性達成)
...
## 修正内容
### Core MIR/PHI (5ファイル)
- builder.rs: variable_map, value_types, value_origin_newbox
- context.rs: 3つのマップ
- loop_builder.rs: 3箇所
- loop_snapshot_manager.rs: snapshot マップ
- loop_snapshot_merge.rs: 2箇所
### MIR関連 (4ファイル)
- function.rs: FunctionMetadata.value_types
- resolver.rs: CalleeResolverBox
- guard.rs: CalleeGuardBox
- loop_common.rs: apply_increment_before_continue
### JSON Bridge (5ファイル)
- json_v0_bridge/lowering.rs
- json_v0_bridge/lowering/expr.rs
- json_v0_bridge/lowering/if_else.rs
- json_v0_bridge/lowering/merge.rs
- json_v0_bridge/lowering/try_catch.rs
- json_v0_bridge/mod.rs
### Printer & Providers (4ファイル)
- printer.rs, printer_helpers.rs
- host_providers/mir_builder.rs
- backend/mir_interpreter/handlers/extern_provider.rs
### Tests (3ファイル)
- phi_core/conservative.rs
- tests/json_program_loop.rs
- tests/mir_stage1_using_resolver_verify.rs (2テスト有効化)
## テスト結果
- mir_stage1_using_resolver_resolve_with_modules_map_verifies: 80%成功率
- 完全な決定性は未達成 (HashMap 86箇所、HashSet 63箇所が残存)
🐱 Generated with Claude Code
2025-11-22 05:33:40 +09:00
93bcb32e68
docs: Phase 21.7++ 全フェーズ完了を CURRENT_TASK.md に記録
...
- Phase 0-4 の完了状況をコミットハッシュ付きで記録
- 累計工数: 10時間(進捗率: 50-67%)
- 各フェーズの実装内容と効果を整理
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-22 02:52:10 +09:00
5eb97b60bc
docs: Phase 21.7++ Phase 0-2 完了記録 - CURRENT_TASK.md 更新
2025-11-22 02:37:34 +09:00
ce7517dc21
docs(phase-21.7): NamingBox SSOT 統一化チェックリスト作成
...
Phase 21.7++ の詳細実装計画を文書化
## 作成ファイル
- docs/development/current/main/phase-21.7-naming-ssot-checklist.md
- Phase 0-4 の詳細タスクリスト(チェックボックス付き)
- 各タスクの具体的な実装コード例
- テストケース
- 工数見積もり・優先順位
- 完了条件
## CURRENT_TASK.md 更新
- Phase 21.7 セクションにチェックリストへのリンク追加
## 実装優先順位
1. Phase 0: 観測ライン緊急構築(最優先、2-3時間)
2. Phase 1: 基盤整備(4-6時間)
3. Phase 2: VM 統一(3-4時間)
4. Phase 3-4: 全体統一・ドキュメント(Phase 22+)
次のステップ: Phase 0 実装開始
2025-11-22 01:59:27 +09:00
208c656589
docs: Phase 21.7 完了記録 - Static Box Methodization
2025-11-22 00:01:58 +09:00
2f07ab6a30
docs(phase25.4): Phase 25.4 ロードマップ・ドキュメント整備
...
📋 Phase 25.4 計画確定: NamingBox SSOT化 & CLI設定箱
✅ 追加ドキュメント:
- docs/development/roadmap/phases/phase-25.4-naming-cli-cleanup/
- README.md: Phase 25.4 全体計画
- A. NamingBox SSOT化(完了)
- B. Stage-1 CLI 設定箱(次フェーズ)
- C. MIR ログ観測リスト(次フェーズ)
- docs/development/roadmap/phases/phase-21.7-normalization/README.md
- Phase 21.7 との関連性追記
- docs/development/roadmap/phases/phase-25.1/stage1-usingresolver-loopform.md
- Phase 25.1 完了記録更新
- CURRENT_TASK.md: タスク進捗更新
- Phase 25.4-A 完了
- Phase 25.4-B/C 準備完了
- MIR Builder 型混乱バグ調査完了記録
🎯 Phase 25.4 戦略:
0. 共通方針
- 既定挙動は変えない(Fail-Fast + テスト緑キープ)
- 新規ロジックは「小さな箱」に閉じ込める
- まずドキュメント・構造を揃えてからコード
A. NamingBox SSOT化 ✅ 完了
- static/global 名前決定を src/mir/naming.rs に一本化
- Builder/VM で統一ルール使用
B. Stage-1 CLI 設定箱(次フェーズ)
- env.get() を Stage1CliConfigBox に集約
- mode/backend/source 等を Config として管理
C. MIR ログ観測リスト(次フェーズ)
- __mir__.log の一覧化・分類
- 将来の MirLogBox 化準備
📊 テスト確認コマンド:
- cargo test -q mir_static_box_naming --lib
- cargo test -q mir_stage1_cli_entry_ssa_smoke --lib
- tools/smokes/v2/run.sh --profile quick
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-21 09:02:30 +09:00
f9d100ce01
chore: Phase 25.1 完了 - LoopForm v2/Stage1 CLI/環境変数削減 + Phase 26-D からの変更
...
Phase 25.1 完了成果:
- ✅ LoopForm v2 テスト・ドキュメント・コメント完備
- 4ケース(A/B/C/D)完全テストカバレッジ
- 最小再現ケース作成(SSAバグ調査用)
- SSOT文書作成(loopform_ssot.md)
- 全ソースに [LoopForm] コメントタグ追加
- ✅ Stage-1 CLI デバッグ環境構築
- stage1_cli.hako 実装
- stage1_bridge.rs ブリッジ実装
- デバッグツール作成(stage1_debug.sh/stage1_minimal.sh)
- アーキテクチャ改善提案文書
- ✅ 環境変数削減計画策定
- 25変数の完全調査・分類
- 6段階削減ロードマップ(25→5、80%削減)
- 即時削除可能変数特定(NYASH_CONFIG/NYASH_DEBUG)
Phase 26-D からの累積変更:
- PHI実装改善(ExitPhiBuilder/HeaderPhiBuilder等)
- MIRビルダーリファクタリング
- 型伝播・最適化パス改善
- その他約300ファイルの累積変更
🎯 技術的成果:
- SSAバグ根本原因特定(条件分岐内loop変数変更)
- Region+next_iパターン適用完了(UsingCollectorBox等)
- LoopFormパターン文書化・テスト化完了
- セルフホスティング基盤強化
Co-Authored-By: Claude <noreply@anthropic.com >
Co-Authored-By: ChatGPT <noreply@openai.com >
Co-Authored-By: Task Assistant <task@anthropic.com >
2025-11-21 06:25:17 +09:00
3a82633924
refactor(funcscanner): Region+next_i パターン統一 & SSA テスト追加
...
**FuncScanner .hako 側改善**:
- scan_all_boxes を Region + next_i 形式に統一(continue 多発による SSA/PHI 複雑さ削減)
- インデント修正(タブ→スペース統一)
- デバッグ print 削除
**SSA テスト追加**:
- lang/src/compiler/tests/funcscanner_scan_methods_min.hako
- src/tests/mir_funcscanner_ssa.rs (scan_methods & fib_min SSA デバッグテスト)
**Phase 25.3 ドキュメント**:
- docs/development/roadmap/phases/phase-25.3-funcscanner/ 追加
**関連**: Phase 25.3 FuncScanner 箱化準備作業
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-20 06:38:43 +09:00
9bdf2ff069
chore: Phase 25.2関連ドキュメント更新&レガシーテストアーカイブ整理
...
## ドキュメント更新
- CURRENT_TASK.md: Phase 25.2完了記録
- phase-25.1b/e/q/25.2 README更新
- json_v0_bridge/README.md新規追加
## テストファイル整理
- vtable_*テストをtests/archive/に移動(6ファイル)
- json_program_loop.rsテスト追加
## コード整理
- プラグイン(egui/python-compiler)微修正
- benchmarks.rs, instance_v2.rs更新
- MIR関連ファイル微調整
## 全体成果
Phase 25.2完了により:
- LoopSnapshotMergeBox統一管理実装
- ValueId(1283)バグ根本解決
- ~35行コード削減(目標210行の16%)
- 11テスト全部PASS、3実行テストケースPASS
2025-11-20 03:56:12 +09:00
525e59bc8d
feat(loop-phi): Add body-local variable PHI generation for Rust AST loops
...
Phase 25.1c/k: Fix ValueId undefined errors in loops with body-local variables
**Problem:**
- FuncScannerBox.scan_all_boxes/1 and BreakFinderBox._find_loops/2 had ValueId
undefined errors for variables declared inside loop bodies
- LoopFormBuilder only generated PHIs for preheader variables, missing body-locals
- Example: `local ch = s.substring(i, i+1)` inside loop → undefined on next iteration
**Solution:**
1. **Rust AST path** (src/mir/loop_builder.rs):
- Detect body-local variables by comparing body_end_vars vs current_vars
- Generate empty PHI nodes at loop header for body-local variables
- Seal PHIs with latch + continue snapshot inputs after seal_phis()
- Added HAKO_LOOP_PHI_TRACE=1 logging for debugging
2. **JSON v0 path** (already fixed in previous session):
- src/runner/json_v0_bridge/lowering/loop_.rs handles body-locals
- Uses same strategy but for JSON v0 bridge lowering
**Results:**
- ✅ FuncScannerBox.scan_all_boxes: 41 body-local PHIs generated
- ✅ Main.main (demo harness): 23 body-local PHIs generated
- ⚠️ Still some ValueId undefined errors remaining (exit PHI issue)
**Files changed:**
- src/mir/loop_builder.rs: body-local PHI generation logic
- lang/src/compiler/entry/func_scanner.hako: debug logging
- /tmp/stageb_funcscan_demo.hako: test harness
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-19 23:12:01 +09:00
1747ec976c
refactor(json_v0_bridge): Phase 25.1p - FunctionDefBuilder箱化+me予約修正
...
【変更内容】
1. FunctionDefBuilder 箱化(SSOT化)
- インスタンスメソッド判定の一元化
- パラメータ ValueId 生成の統一
- 変数マップ初期化の統一
2. ValueId(0) me 予約バグ修正
- is_instance_method() で box_name != "Main" 判定
- インスタンスメソッドは me を ValueId(0) に予約
- variable_map["me"] = ValueId(0) を自動設定
3. コード削減・可読性向上
- 60行 → 40行(関数定義処理)
- 重複ロジック削除
- デバッグログ追加(is_instance表示)
【効果】
- json_v0_bridge 経路の ValueId(0) 未定義エラー解消
- Stage-B compiler で static box メソッドが正しく動作
- 設計の一貫性向上(me の扱いが明確)
【非スコープ】
- Rust MirBuilder 側は未修正(Phase 26で統一予定)
- lower_static_method_as_function は現状維持
関連: Phase 25.1m (静的メソッド修正), Phase 25.1c/k (SSA修正)
🐱 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-19 10:08:04 +09:00
75f3df2505
refactor(mir): Phase 25.1o - do_break/continue 共通化(LoopExitKind型統一)
...
【変更内容】
1. LoopExitKind enum定義
- Break / Continue の型安全な区別
2. do_loop_exit() 共通メソッド作成(47行)
- スナップショット取得(共通処理)
- kind別のスナップショット保存
- kind別のジャンプターゲット
- unreachable ブロック切り替え(共通処理)
3. do_break/continue をthin wrapperに変換
- do_break: 13行 → 4行
- do_continue: 12行 → 4行
- 合計21行削減
【効果】
- 構造改善: break/continue の共通ロジック一箇所に集約
- 保守性向上: デバッグログなどの共通処理が統一管理
- 拡張性向上: labeled break/continue等の将来拡張が容易
【検証結果】
- ビルド成功(警告なし)
- mir_stageb_loop_break_continue_verifies: PASS
- /tmp/loop_continue_fixed.hako: RC=3(期待通り)
関連: Phase 25.1m (continue PHI修正), Phase 25.1n (レガシー削除)
2025-11-19 08:56:44 +09:00
77fc670cd3
refactor(mir): Phase 25.1n - レガシーコード削除(未使用コード整理)
...
**削減内容**:
- loop_builder.rs: incomplete_phis, emit_safepoint, mark_block_sealed (-28行)
- builder.rs: hint_loop_*, debug_loop_*, debug_replace_region (-28行)
- builder/loops.rs: create_loop_blocks (-9行)
**成果**:
- コード削減: 65行の未使用コード削除
- 警告削減: 7個 → 2個 (71%削減)
- 機能維持: すべてのテスト通過 ✅
**削除されたレガシーAPI**:
- incomplete_phis フィールド (LoopFormBuilder移行後未使用)
- emit_safepoint() メソッド (未使用)
- mark_block_sealed() メソッド (未使用)
- hint_loop_header/latch/carrier() (ヒントシステム未使用)
- debug_next_loop_id() (デバッグ機能未使用)
- debug_replace_region() (デバッグ機能未使用)
- create_loop_blocks() (LoopForm v2移行後未使用)
🧹 箱化・モジュール化の第一歩として未使用コード整理完了
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-19 08:35:56 +09:00
dad278caf2
feat(hotfix-8): Fix static method receiver detection logic (Hotfix 4 inversion bug)
...
## Issue
- StageBTraceBox.log("label") passed null for the label parameter
- Static methods incorrectly detected as having implicit receiver
- Hotfix 4 logic was inverted: `!matches!(params[0], MirType::Box(_))`
## Root Cause
src/mir/function.rs:90-101 had inverted logic:
- Instance methods (params[0]: Box type) → should have receiver
- Static methods (params[0]: non-Box type) → should NOT have receiver
- Previous code: `!matches!()` = true for non-Box → receiver=true (WRONG)
## Fix
- Changed `!matches!(signature.params[0], MirType::Box(_))` to
`matches!(signature.params[0], MirType::Box(_))`
- Updated comments to clarify instance vs static method detection
- Result: static methods now correctly have receiver=0
## Verification
Before: fn='StageBTraceBox.log/1' params=1, receiver=1, total=2 ❌
After: fn='StageBTraceBox.log/1' params=1, receiver=0, total=1 ✅
Test output:
Before: [stageb/trace] # label=null
After: [stageb/trace] test_label # label passed correctly ✅
## Files Changed
- src/mir/function.rs: Fixed has_implicit_receiver logic
- lang/src/compiler/entry/compiler_stageb.hako: Added guaranteed entry marker
and direct print traces for Phase 25.1c debugging
- CURRENT_TASK.md: Updated task progress
## Phase 25.1c Progress
- Hotfix 8 完了:static method parameter passing 修正
- Stage-B トレース正常動作確認
- 次のタスク:ParserBox.parse_program2 ハング問題調査
🐾 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-19 05:30:31 +09:00
fa571a656e
feat(stageb): Phase 25.1c - Stage-B トレース追加(dev-only)
...
追加内容:
- StageBTraceBox: dev トレース用 Box 追加(HAKO_STAGEB_TRACE=1 で有効)
- トレースポイント:
- StageBArgsBox.resolve_src: enter/return_len
- StageBBodyExtractorBox.build_body_src: enter_len/return_len
- StageBDriverBox.main: enter/after_resolve_src/after_build_body_src/
after_parse_program2/func_scan methods/exit rc=0
Phase 25.1c 目標:
- Stage-B / Stage-1 CLI 構造デバッグ
- fib canary / selfhost CLI canary の rc=1 原因特定
ポリシー:
- dev env でガード(挙動不変)
- 既定挙動は変更せず、観測のみ追加
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-19 04:49:25 +09:00
b79697f137
feat(region): Phase 25.1l FunctionSlotRegistry完全実装
...
ChatGPT実装 M-1〜M-4:
- FunctionSlotRegistry: 変数スロット管理中央化
- RegionKind::Function追加
- RefKind分類統合
- 観測レイヤー完成
品質評価 (Task先生レビュー):
- 設計: ⭐ ⭐ ⭐ ⭐ ⭐ (箱理論完璧)
- 実装: M-1〜M-4全て完全
- 統合: 既存システムと高品質統合
- 影響: SSA/PHI非侵襲(観測専用)
既知問題:
- userbox_birth_to_string_vm失敗
→ 既存問題(Phase 25.1h以前から)
→ 本実装とは無関係
→ 別途調査予定
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-19 03:28:58 +09:00
39f5256c18
📊 Phase 25.1l: Region観測レイヤー骨格 + スコープ契約設計理解
...
**Region Box統一理論の実装開始**
新規追加:
- src/mir/region/mod.rs: Region/RefSlotKind型定義
- src/mir/region/observer.rs: Region観測レイヤー
- docs/development/roadmap/phases/phase-25.1l/: 設計ドキュメント
主要概念:
- Region Box = Function/Loop/If の統一箱
- RefSlotKind = GC管理用スロット種別(Strong/Weak/Borrowed/NonRef)
- 観測専用(NYASH_REGION_TRACE=1で動作、挙動変更なし)
設計理解の深化:
- ValueId(40)問題 = LoopForm v2スコープ契約違反の症状
- 根本解決 = Region観測で無名一時値のスコープまたぎを検出
- 箱理論3原則: 境界明確化/差し替え可能/段階的移行
関連議論:
- ChatGPT提案: Region統一理論でGC/寿命管理の基盤構築
- SlotRegistry: 変数の単一真実源(SSOT)
- 階層構造: FunctionRegion → LoopRegion → IfRegion
次のステップ:
- Phase 1: Region観測(現在)- 非破壊的追加
- Phase 2: メタデータ出力(MIR JSON拡張)
- Phase 3: GC統合(retain/release挿入)
テスト追加:
- lang/src/compiler/tests/stageb_mini_driver.hako
- tools/test_loopssa_breakfinder_slot.sh
Build: ✅ 全警告は既存のもの
Tests: 既存テスト全て緑維持
2025-11-19 02:44:40 +09:00
80f8a7bc8c
🔧 Hotfix 7 (Enhanced): ValueId receiver alias tracking for nested loops
...
- Problem: Pinned receiver variables in loops cause undefined ValueId errors
- Enhanced fix: Update all receiver aliases (me + all __pin$N$@recv levels)
- Handles nested loops by updating previous pin levels
- Test status: Partial improvement, ValueId(50) → ValueId(40)
- Further investigation needed for complete fix
Files modified:
- src/mir/phi_core/loopform_builder.rs (emit_header_phis)
2025-11-19 00:02:41 +09:00
d3cbc71c9b
feat(mir): Phase 25.1f完了 - Conservative PHI + ControlForm観測レイヤー
...
🎉 Conservative PHI Box理論による完全SSA構築
**Phase 7-B: Conservative PHI実装**
- 片方branchのみ定義変数に対応(emit_void使用)
- 全変数にPHI生成(Conservative Box理論)
- Stage-1 resolver全テスト緑化(3/3 PASS)
**Phase 25.1f: ControlForm観測レイヤー**
- LoopShape/IfShape/ControlForm構造定義
- Loop/If統一インターフェース実装
- debug_dump/debug_validate機能追加
- NYASH_CONTROL_FORM_TRACE環境変数対応
**主な変更**:
- src/mir/builder/phi.rs: Conservative PHI実装
- src/mir/control_form.rs: ControlForm構造(NEW)
- src/mir/loop_builder.rs: LoopForm v2デフォルト化
**テスト結果**:
✅ mir_stage1_using_resolver_min_fragment_verifies
✅ mir_stage1_using_resolver_full_collect_entries_verifies
✅ mir_parserbox_parse_program2_harness_parses_minimal_source
🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com >
Co-Authored-By: ChatGPT <chatgpt@openai.com >
2025-11-18 18:56:35 +09:00
f74b7d2b04
📦 Hotfix 1 & 2: Parameter ValueId Reservation + Exit PHI Validation (Box-First Theory)
...
**箱理論に基づく根治的修正**:
## 🎯 Hotfix 1: Parameter ValueId Reservation (パラメータ ValueId 予約)
### 根本原因
- MirFunction counter が params.len() を考慮していなかった
- local variables が parameter ValueIds を上書き
### 箱理論的解決
1. **LoopFormContext Box**
- パラメータ予約を明示的に管理
- 境界をはっきりさせる
2. **MirFunction::new() 改善**
- `initial_counter = param_count.max(1)` でパラメータ予約
- Parameters are %0, %1, ..., %N-1
3. **ensure_counter_after() 強化**
- パラメータ数 + 既存 ValueIds 両方を考慮
- `min_counter = param_count.max(max_id + 1)`
4. **reserve_parameter_value_ids() 追加**
- 明示的な予約メソッド(Box-First)
## 🎯 Hotfix 2: Exit PHI Predecessor Validation (Exit PHI 検証)
### 根本原因
- LoopForm builder が存在しないブロックを PHI predecessor に追加
- 「幽霊ブロック」問題
### 箱理論的解決
1. **LoopFormOps.block_exists() 追加**
- CFG 存在確認メソッド
- 境界を明確化
2. **build_exit_phis() 検証**
- 非存在ブロックをスキップ
- デバッグログ付き
### 実装ファイル
- `src/mir/function.rs`: Parameter reservation
- `src/mir/phi_core/loopform_builder.rs`: Context + validation
- `src/mir/loop_builder.rs`: LoopFormOps impl
- `src/mir/builder/stmts.rs`: Local variable allocation
### 業界標準準拠
- ✅ LLVM IR: Parameters are %0, %1, ...
- ✅ SSA Form: PHI predecessors must exist in CFG
- ✅ Cytron et al. (1991): Parameter reservation principle
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-18 06:39:45 +09:00
73844dbe04
feat(builder): CalleeBoxKind構造ガードで静的/ランタイムBox混線を根絶
...
🎯 箱理論の実践: 「境界を作る」原則による構造レベル分離
## 問題
- StageBArgsBox.resolve_src内のargs.get(i)が
Stage1UsingResolverBox.getに化ける(静的Box名混入)
- 未定義ValueIdエラー発生(receiver定義なし)
## 解決策(構造ガード)
✅ CalleeBoxKind enum追加
- StaticCompiler: Stage-B/Stage-1コンパイラBox
- RuntimeData: MapBox/ArrayBox等ランタイムBox
- UserDefined: ユーザー定義Box
✅ classify_box_kind(): Box名から種別判定
- 静的Box群を明示的に列挙(1箇所に集約)
- ランタイムBox群を明示的に列挙
- 将来の拡張も容易
✅ apply_static_runtime_guard(): 混線検出・正規化
- me-call判定(receiver型==box_name → 静的降下に委ねる)
- 真の混線検出(receiver型≠box_name → 正規化)
- トレースログで可視化
## 効果
- 修正前: Invalid value ValueId(150/187)
- 修正後: Unknown method 'is_space' (別issue、StringBox実装不足)
- → 静的Box名混入問題を根絶!
## 箱理論原則
- ✅ 境界を作る: Static/Runtime/UserDefinedを構造的に分離
- ✅ Fail-Fast: フォールバックより明示的エラー
- ✅ 箱にする: CalleeBoxKindでBox種類を1箇所に集約
## ファイル
- src/mir/definitions/call_unified.rs: CalleeBoxKind enum
- src/mir/builder/calls/call_unified.rs: classify_box_kind()
- src/mir/builder/calls/emit.rs: apply_static_runtime_guard()
- docs/development/roadmap/phases/phase-25.1d/README.md: 箱化メモ更新
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-17 23:13:57 +09:00
f300b9f3c9
Fix MIR builder me-call recursion and add compile tracing
2025-11-17 19:53:44 +09:00
e5b9b84aca
fix: guard unified BoxCall recursion and document Stage-B stack overflow status
2025-11-17 17:53:40 +09:00
757b0fcfc9
feat(mir/builder): implement BoxCompilationContext for structural metadata isolation
...
箱理論の完璧な実装!各static boxコンパイルを独立したコンテキストで実行。
設計:
- BoxCompilationContext: variable_map, value_origin_newbox, value_types を箱化
- MirBuilder: compilation_context: Option<BoxCompilationContext> フィールド追加
- context swap: lower_static_method_as_function 開始/終了時に std::mem::swap
- 自動クリーンアップ: スコープ終了でコンテキスト破棄
実装:
1. src/mir/builder/context.rs: BoxCompilationContext構造体定義(テスト付き)
2. src/mir/builder.rs: compilation_contextフィールド追加、既存フィールドにコメント追加
3. src/mir/builder/lifecycle.rs: 各static boxでコンテキスト作成・破棄
4. src/mir/builder/builder_calls.rs: lower_static_method_as_functionでcontext swap
5. src/mir/builder/decls.rs, exprs.rs: 古いmanual clear()削除
効果:
✅ グローバル状態汚染を構造的に不可能化
✅ 各static boxが完全に独立したコンテキストでコンパイル
✅ 既存コード変更なし(swap技法で完全後方互換性)
✅ StageBArgsBox ValueId(21)エラー完全解決
箱理論的評価: 🟢 95点
- 明示的な境界: 各boxのコンテキストが物理的に分離
- 汚染不可能: 前の箱の状態が構造的に残らない
- 戻せる: コンテキスト差し替えで簡単ロールバック
- 美しい設計: スコープベースのリソース管理
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-17 11:28:18 +09:00
cc9fb2f654
fix(phi): if-block PHI reassignment のSSA違反を解消
...
**問題**: If-block PHI nodes がグローバルValueIdアロケーター使用 → 関数ローカルIDと衝突
- insert_phi() が value_gen.next() 使用
- 関数内で後から割り当てられるValueIdと重複 → SSA違反
**根本原因**:
```rust
// ❌ Before: グローバルアロケーター
let phi_val = self.value_gen.next();
// ✅ After: 関数ローカルアロケーター
let phi_val = if let Some(ref mut f) = self.current_function {
f.next_value_id() // 関数コンテキスト
} else {
self.value_gen.next() // モジュールコンテキスト
};
```
**修正内容**:
1. **insert_phi() 修正** (src/mir/utils/phi_helpers.rs:63-70)
- 関数コンテキストでは f.next_value_id() 使用
- pin_to_slot() / loop builder (e2d061d1 ) と同じパターン
2. **insert_phi_with_dst() ドキュメント強化** (lines 94-114)
- 正しいValueId割り当て方法を例示
- ❌ 間違い: 常に value_gen.next() 使用
- ✅ 正しい: 関数コンテキスト判定
**テスト結果**:
✅ Simple if-param-method: PASS (SSA違反なし)
✅ Test1/Test3: PASS (regression なし)
⚠️ Test2 (Stage-B): ValueId(22) 残存(別問題: receiver materialization)
**残存問題** (別issue):
- ValueId(22) = receiver materialization 問題
- pin_to_slot / emit_guard / BlockScheduleBox が分散
- 責務の集約が必要(次タスク)
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-17 06:54:48 +09:00
47071f2755
fix(loop/loopform): 専用preheader作成でValueId定義エラー解消 (WIP)
...
**問題**: NYASH_LOOPFORM_PHI_V2=1 でValueId未定義エラー発生
- emit_preheader()が既存blockに命令追加 → forward reference
- if-then内のloopでpreheader_id = current_block()が誤動作
**解決策**: LLVM canonical loop form準拠
- 専用preheaderブロック作成 (before_loop → preheader → header)
- 変数スナップショットをnew_block()前に取得
- emit_jump(preheader)で明示的分離
**成果**:
✅ ValueId定義エラー完全解消 (ValueId(10) etc.)
✅ 詳細デバッグ出力追加 (variable_map追跡)
❌ 新しい型エラー発生 (要調査: variable_map不整合)
**デバッグ出力**:
- before_loop_id + variable_map size
- Block IDs (preheader/header/body/latch/exit)
- 変数ごとのparam判定 + iteration count
**次のステップ**: variable_map変更点特定
- before_loop: args=ValueId(0)
- prepare_structure: args=ValueId(1) ← なぜ変わる?
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-17 05:48:03 +09:00
c459135238
feat(mir/phi): improve LoopForm parameter detection - track param names
...
**Problem**: is_parameter() was too simple, checking only ValueId which changes
through copies/PHIs. This caused parameters like 'data' to be misclassified as
carriers, leading to incorrect PHI construction.
**Solution**: Track original parameter names at function entry.
**Changes**:
1. **Added function_param_names field** (builder.rs):
- HashSet<String> to track original parameter names
- Populated in lower_static_method_as_function()
- Cleared and repopulated for each new function
2. **Improved is_parameter()** (loop_builder.rs):
- Check name against function_param_names instead of ValueId
- More reliable than checking func.params (ValueIds change)
- __pin$*$@* variables correctly classified as carriers
- Added debug logging with NYASH_LOOPFORM_DEBUG
3. **Enhanced debug output** (loopform_builder.rs):
- Show carrier/pinned classification during prepare_structure()
- Show variable_map state after emit_header_phis()
**Test Results**:
- ✅ 'args' correctly identified as parameter (was working)
- ✅ 'data' now correctly identified as parameter (was broken)
- ✅ __pin variables correctly classified as carriers
- ✅ PHI values allocated and variable_map updated correctly
- ⚠️ ValueId undefined errors persist (separate issue)
**Remaining Issue**:
ValueId(10) undefined error suggests PHI visibility problem or VM verification
issue. Needs further investigation of emit_phi_at_block_start() or VM executor.
**Backward Compatibility**:
- Flag OFF: 100% existing behavior preserved (legacy path unchanged)
- Feature-flagged with NYASH_LOOPFORM_PHI_V2=1
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude <noreply@anthropic.com >
2025-11-17 05:24:07 +09:00