Skip to content

Improve rewrite framing with formula-level read tracking#948

Merged
strub merged 1 commit intomainfrom
proc-change-with-exn
Mar 23, 2026
Merged

Improve rewrite framing with formula-level read tracking#948
strub merged 1 commit intomainfrom
proc-change-with-exn

Conversation

@strub
Copy link
Member

@strub strub commented Mar 23, 2026

It adds form/exnpost folding and formula read-analysis utilities,
exposes statement-logic read extraction in ecLowPhlGoal, and
updates ecPhlRewrite to build rewrite obligations from the
actual framed precondition and written variables instead of
relying on coarser statement-wide equality assumptions.

@strub strub requested a review from Gustavo2622 March 23, 2026 11:17
@strub strub self-assigned this Mar 23, 2026
@strub strub force-pushed the proc-change-with-exn branch from 7507659 to e349bf5 Compare March 23, 2026 13:39
@strub strub changed the title Track program-logic reads precisely in proc-change rewrites Improve rewrite framing with formula-level read tracking Mar 23, 2026
@strub strub removed the bug label Mar 23, 2026
@strub strub mentioned this pull request Mar 23, 2026
@strub strub force-pushed the proc-change-with-exn branch from e349bf5 to b56f978 Compare March 23, 2026 13:58
It adds form/exnpost folding and formula read-analysis utilities,
exposes statement-logic read extraction in ecLowPhlGoal, and
updates ecPhlRewrite to build rewrite obligations from the
actual framed precondition and written variables instead of
relying on coarser statement-wide equality assumptions.
Also increases test coverage for the proc change tactic.

Co-Authored-By: Gustavo Delerue <gxdelerue@proton.me>
@Gustavo2622 Gustavo2622 force-pushed the proc-change-with-exn branch from b56f978 to 63b8965 Compare March 23, 2026 16:41
@strub strub merged commit cc03b30 into main Mar 23, 2026
16 checks passed
@strub strub deleted the proc-change-with-exn branch March 23, 2026 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants