Skip to content

Conversation

@celinval
Copy link
Contributor

Description of changes:

Add a few more options to compiletest which I'm planning to use on our benchmarks. #2442

  • "--report-time": Will report how long it took to run a test. This includes the time taken to process the expected file.
  • "--kani-flag": Allow us to pass extra arguments to all tests.

Resolved issues:

Related to #2442

Related RFC:

Optional #ISSUE-NUMBER.

Call-outs:

Testing:

  • How is this change tested? Manually tested

  • Is this a refactor change? No

Checklist

  • Each commit message has a non-empty body, explaining why the change was made
  • Methods or procedures are documented
  • Regression or unit tests are included, or existing tests cover the modified code
  • My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

  - "--report-time": Will report how long it took to run a test.
  - "--kani-flag": Allow us to pass extra arguments to all tests.
@celinval celinval requested a review from a team as a code owner July 14, 2023 22:12
@celinval
Copy link
Contributor Author

@jaisnan for my use case, I'm planning to using the --only-codegen option to allow me to run benchmarks up to the codegen stage. Another option I think would be super helpful is to assess unstable features.

@celinval celinval enabled auto-merge (squash) July 14, 2023 22:37
@celinval celinval merged commit 3d8cedd into model-checking:main Jul 14, 2023
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