Skip to content

Conversation

@szegedy
Copy link

@szegedy szegedy commented Mar 28, 2017

This change allows to trace the parameter of CONV_TAC as a string for a first iteration of tracking conversions. The parameter of CONV_TAC is stored in the proof log.

@szegedy szegedy requested a review from girving March 28, 2017 02:32
`{x | x IN (s UNION t) /\ P x} =
{x | x IN s /\ P x} UNION {x | x IN t /\ P x}`] THEN
(CONV_TAC o GEN_SIMPLIFY_CONV TOP_DEPTH_SQCONV (basic_ss []) 5)
((CONV_TAC "o") o GEN_SIMPLIFY_CONV TOP_DEPTH_SQCONV (basic_ss []) 5)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one isn't reasonably matched, though I'm not sure how it'd be covered by this technique.

| Label_tac_log (st, th) -> Snode [Sleaf "Label_tac_log"; Sleaf st; sexp_thm th]
| Accept_tac_log th -> thm "Accept_tac_log" th
| Conv_tac_log c -> Snode [Sleaf "Conv_tac_log"; sexp_conv c]
| Conv_tac_log (s, c) -> Snode [Sleaf "Conv_tac_log"; Sleaf s; sexp_conv c]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will break any S-expr parser that we write, since the leaf strings aren't escaped. Let's leave it out for now.

@@ -0,0 +1,52 @@
def next_nonspace(s, i)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does conv_tac_params.rb differ from match_paren.rb?

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