Skip to content

Extract FunctionControlFlowInfo from SSA and decouple CFG building#62

Open
RealA10N wants to merge 1 commit intomainfrom
claude/cfg-extraction-l9a1P
Open

Extract FunctionControlFlowInfo from SSA and decouple CFG building#62
RealA10N wants to merge 1 commit intomainfrom
claude/cfg-extraction-l9a1P

Conversation

@RealA10N
Copy link
Copy Markdown
Owner

No description provided.

Extracts the basic-block linearisation and CFG construction into a new,
standalone opt.FunctionControlFlowInfo struct so that any opt pass (e.g.
constant propagation, liveness analysis) can obtain the CFG without
constructing a full SSA info object.

FunctionSsaInfo is updated to embed opt.FunctionControlFlowInfo instead
of duplicating the same fields; no behaviour changes.

https://claude.ai/code/session_0165cPFCURfazKMvWB6yPZBa
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 16, 2026

Codecov Report

❌ Patch coverage is 0% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 38.44%. Comparing base (501b2a4) to head (5e2130c).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
opt/function_control_flow_info.go 0.00% 16 Missing ⚠️
opt/ssa/function_ssa_info.go 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #62      +/-   ##
==========================================
- Coverage   38.45%   38.44%   -0.02%     
==========================================
  Files         122      123       +1     
  Lines        3789     3790       +1     
==========================================
  Hits         1457     1457              
- Misses       2247     2248       +1     
  Partials       85       85              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

RealA10N pushed a commit that referenced this pull request Mar 16, 2026
Tests cover NewFunctionControlFlowInfo for:
- single-block functions (no forward edges)
- BasicBlocksToIndex consistency with BasicBlocks positions
- unconditional jump (j) produces exactly one forward edge
- conditional branch (jz) produces fall-through + branch edges (3 blocks)

https://claude.ai/code/session_01NbcU4m68EVYiLAxXeFqfDL
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.

2 participants