Skip to content

Commit

Permalink
Add support for while loops with invariants.
Browse files Browse the repository at this point in the history
  • Loading branch information
parno committed Nov 15, 2023
1 parent c021b73 commit f0d6aae
Show file tree
Hide file tree
Showing 4 changed files with 95 additions and 11 deletions.
15 changes: 14 additions & 1 deletion examples/wip.rs
Original file line number Diff line number Diff line change
@@ -1,6 +1,19 @@
verus! {

fn syntactic_eq(&self, other: &Self) {
pub fn clone_vec_u8() {
let i = 0;
while i < v.len()
invariant
i <= v.len(),
i == out.len(),
forall |j| #![auto] 0 <= j < i ==> out@[j] == v@[j],
ensures
i > 0,
decreases
72,
{
i = i + 1;
}
}

} // verus!
41 changes: 34 additions & 7 deletions src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -441,7 +441,6 @@ fn to_doc<'a>(
| Rule::impl_str
| Rule::in_str
| Rule::int_str
| Rule::invariant_str
| Rule::isize_str
| Rule::let_str
| Rule::loop_str
Expand Down Expand Up @@ -497,9 +496,11 @@ fn to_doc<'a>(
.append(arena.space())
.nest(INDENT_SPACES),

Rule::decreases_str | Rule::ensures_str | Rule::recommends_str | Rule::requires_str => {
arena.hardline().append(s).nest(INDENT_SPACES)
}
Rule::decreases_str
| Rule::ensures_str
| Rule::invariant_str
| Rule::recommends_str
| Rule::requires_str => arena.hardline().append(s).nest(INDENT_SPACES),

Rule::assert_str
| Rule::assume_str
Expand Down Expand Up @@ -585,7 +586,12 @@ fn to_doc<'a>(
saw_comment_after_param_list = true;
};
// Special case where we don't want an extra newline after the possibly inline comment
comment_to_doc(ctx, arena, p, !has_qualifier || !saw_comment_after_param_list)
comment_to_doc(
ctx,
arena,
p,
!has_qualifier || !saw_comment_after_param_list,
)
}
Rule::param_list => {
saw_param_list = true;
Expand Down Expand Up @@ -742,7 +748,28 @@ fn to_doc<'a>(
Rule::if_expr => map_to_doc(ctx, arena, pair),
Rule::loop_expr => unsupported(pair),
Rule::for_expr => unsupported(pair),
Rule::while_expr => map_to_doc(ctx, arena, pair),
Rule::while_expr => {
// We need to add a newline after the very last clause,
// so that the opening brace of the loop body is on a fresh line
let mut last_clause = None;
pair.clone().into_inner().for_each(|p| match p.as_rule() {
Rule::invariant_clause => last_clause = Some(Rule::invariant_clause),
Rule::ensures_clause => last_clause = Some(Rule::ensures_clause),
Rule::decreases_clause => last_clause = Some(Rule::decreases_clause),
_ => (),
});
arena.concat(pair.into_inner().map(|p| {
if let Some(c) = last_clause {
if p.as_rule() == c {
to_doc(ctx, p, arena).append(arena.line())
} else {
to_doc(ctx, p, arena)
}
} else {
to_doc(ctx, p, arena)
}
}))
}
Rule::label => unsupported(pair),
Rule::break_expr => map_to_doc(ctx, arena, pair),
Rule::continue_expr => map_to_doc(ctx, arena, pair),
Expand Down Expand Up @@ -839,7 +866,7 @@ fn to_doc<'a>(
Rule::verus_clause_non_expr => map_to_doc(ctx, arena, pair),
Rule::requires_clause => map_to_doc(ctx, arena, pair),
Rule::ensures_clause => map_to_doc(ctx, arena, pair),
Rule::invariant_clause => unsupported(pair),
Rule::invariant_clause => map_to_doc(ctx, arena, pair),
Rule::recommends_clause => map_to_doc(ctx, arena, pair),
Rule::decreases_clause => map_to_doc(ctx, arena, pair),
Rule::assert_requires => map_to_doc(ctx, arena, pair).append(arena.line()),
Expand Down
2 changes: 0 additions & 2 deletions tests/rustfmt-tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -368,6 +368,4 @@ let (temp_owl__x607, Tracked(itree)): ( _
}
"#;
compare(file);

}

48 changes: 47 additions & 1 deletion tests/snap-tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -696,7 +696,7 @@ pub exec fn foo()
}

#[test]
fn keyword_prefixed_identifier_parsing() {
fn verus_keyword_prefixed_identifier_parsing() {
let file = r#"
verus! {
pub exec fn foo(mut_state: &mut Blah, selfie_stick: SelfieStick) {
Expand All @@ -717,3 +717,49 @@ pub exec fn foo(mut_state: &mut Blah, selfie_stick: SelfieStick) {
} // verus!
"###);
}

#[test]
fn verus_loops() {
let file = r#"
verus! {
pub fn clone_vec_u8() {
let i = 0;
while i < v.len()
invariant
i <= v.len(),
i == out.len(),
forall |j| #![auto] 0 <= j < i ==> out@[j] == v@[j],
ensures
i > 0,
decreases
72,
{
i = i + 1;
}
}
} // verus!
"#;

assert_snapshot!(parse_and_format(file).unwrap(), @r###"
verus! {
pub fn clone_vec_u8() {
let i = 0;
while i < v.len()
invariant
i <= v.len(),
i == out.len(),
forall|j| #![auto] 0 <= j < i ==> out@[j] == v@[j],
ensures
i > 0,
decreases 72,
{
i = i + 1;
}
}
} // verus!
"###);
}

0 comments on commit f0d6aae

Please sign in to comment.