Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

/trace output is less helpful than it could be due to buffering #10

Closed
0xabu opened this issue Jul 15, 2016 · 5 comments
Closed

/trace output is less helpful than it could be due to buffering #10

0xabu opened this issue Jul 15, 2016 · 5 comments

Comments

@0xabu
Copy link

0xabu commented Jul 15, 2016

Just a minor nitpick. When running Dafny with /trace, it produces output such as:

Verifying CheckWellformed$$_.foo ...
  [0.716 s, 2 proof obligations]  verified

Verifying Impl$$_.foo ...
  [187.421 s, 225 proof obligations]  verified

However, the line for "Verifying" only appears in the console window at the same time that the second line with the result information is printed, so you're never sure what Dafny is currently attempting to verify. I assume this could be fixed by adding an fflush()-equivalent between the two print statements.

@parno
Copy link
Collaborator

parno commented Jul 15, 2016

I've been annoyed by that too. However, I think that the trace is coming from Boogie, not Dafny, so you may need to file this on the Boogie repo.

@0xabu
Copy link
Author

0xabu commented Jul 15, 2016

Done. Feel free to close this, if that's appropriate.

@parno
Copy link
Collaborator

parno commented Jul 15, 2016

Okay, I think it makes sense to close this, given that it needs to be fixed on the Boogie side.

@parno parno closed this as completed Jul 15, 2016
@RustanLeino
Copy link
Collaborator

The behavior you're seeing indeed comes from Boogie. It's a feature! Boogie may actually do verifications in different orders or using multiple cores. It then sorts these to come out in the program order.

@cpitclaudel
Copy link
Member

Do we really want trace to run things in parallel? Isn't this a profiling tool primarily?

camrein added a commit that referenced this issue Apr 8, 2021
Add dot completion integration tests.
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

No branches or pull requests

4 participants