Change behavior of 'decreases *', which can be applied to loops and methods. Now, loops that may possibly

do an infinite number of iterations (that is, loops marked with 'decreases *') and calls to methods
marked with 'decreases *' are allowed only in methods that themselves are marked with 'decreases *'.  As
before, ghost loops and ghost methods are not allowed to be marked with 'decreases *'.

Previously, 'decreases *' was allowed on a method only if the method was tail recursive;
this is no longer so.  Note, however, that if the method is not tail recursive and engages
in infinite recursion, then it will eventually run out of stack space.

Previously, a 'decreases *' was not inherited in a refining module; this is no longer so.
That is, 'decreases *' is now inherited.  To refine a possibly non-terminating method
or loop, the refining version simply provides a decreases clause that does not mention '*'.
Note that if the refined method is not recursive, it still needs to have _some_ decreases
clause in order not to inherit the 'decreases *' from the refined method, but the expression
stated in the decreases clause can be arbitrary (for example, one can write 'decreases true'
or 'decreases 7' or 'decreases x' for some 'x' in scope).

Note, in the new design, a method needs to be declared with 'decreases *' if
it may recurse forever _or_ if it contains a possibly infinite loop.

Note that the absence of 'decreases *' on a loop does not mean the loop will
terminate, but it does mean that the loop will iterate a finite number of times
(the subtle distinction here is that a loop without a 'decreases *' is allowed
to contain a nested loop that has a 'decreases *' -- provided the enclosing
method is also declared with 'decreases *', as previously mentioned).

1 files changed, 11 insertions, 12 deletions

diff --git a/Test/dafny0/TailCalls.dfy b/Test/dafny0/TailCalls.dfy
index ba683679..0bc282a4 100644
--- a/Test/dafny0/TailCalls.dfy
+++ b/Test/dafny0/TailCalls.dfy
@@ -30,25 +30,24 @@ method C(q: int) returns (x: int)
 }

 

 method D(q: int) returns (x: int)

-  decreases *;  // error: not allowed, because the method is not tail recursive

 {

   x := D(q-1);

   x := x + 1;

 }

 

-method E0(q: int) returns (x: int)

-  decreases *;  // error: not allowed, because the method is not tail recursive (since mutually recursive methods are currently not recognized as being tail recursive)

+method {:tailrecursion} E0(q: int) returns (x: int)  // error: not allowed, because the method is not

+  // tail recursive (since mutually recursive methods are currently not recognized as being tail recursive)

 {

   x := E1(q-1);

 }

-method E1(q: int) returns (x: int)

-  decreases *;  // error: not allowed, because the method is not tail recursive (since mutually recursive methods are currently not recognized as being tail recursive)

+method {:tailrecursion} E1(q: int) returns (x: int)  // error: not allowed, because the method is not

+  // tail recursive (since mutually recursive methods are currently not recognized as being tail recursive)

 {

   x := E0(q);

 }

 

 method F0(q: int) returns (x: int)

-  decreases *;  // fine, but no 'decreases' spec is needed at all here

+  decreases *;  // fine

 {

   x := D(q);

 }

@@ -63,18 +62,18 @@ method {:tailrecursion} G0(q: int) returns (x: int)
 {

   x := D(q);

 }

-method {:tailrecursion false} G1(q: int) returns (x: int)

-  decreases *;  // error: even though there is no recursion in this method's body, the annotation specifically says "not tail recursive", so (the easiest thing to do in the Resolver was to) generate an error

+method {:tailrecursion false} G1(q: int) returns (x: int)  // the annotation tells the compiler not to tail-call optimize

+  decreases *;

 {

-  x := D(q);

+  x := G1(q);

 }

 

 method H0(q: int) returns (x: int)

-  decreases *;  // fine, but no 'decreases' spec is needed at all here

+  decreases *;  // fine

 method {:tailrecursion} H1(q: int) returns (x: int)

-  decreases *;  // fine, but no 'decreases' spec is needed at all here

+  decreases *;  // fine

 method H2(q: int) returns (x: int)

-  decreases 5;  // fine, but no 'decreases' spec is needed at all here

+  decreases 5;  // fine

 

 class {:autocontracts} MyAutoContractClass {

   var left: MyAutoContractClass;