-
Notifications
You must be signed in to change notification settings - Fork 48
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
Induct flag only adds decreases check if type checker is enabled #657
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please explain the rationale for this change? It seems to me that this commit does not actually change the behaviour of the test suite, is that right?
EDIT: By change I mean the second commit: 936f5f2
The problem is that in the test suite, there are two contexts (one empty, and one with the correct options), and it looks like the empty context was used for the pipeline (and the type-checker option was off during |
@jad-hamza I see, thanks! |
1c3ef9f
to
77dc1bb
Compare
fixes #644 |
@Gorzen Can you please check why the build failed? |
It's because of ParialCompiler and BodyEnsuring, check-models fails on them. |
|
81f4387
to
f019648
Compare
f019648
to
55d0b70
Compare
1287cd2
to
d1c4549
Compare
This is done to ensure that the correct options are passed to the extractor (TypeCheckerEnabled)
This may break caching in ComponentTestSuite, but as of right now options aren't passed properly to the extractor. (TypeCheckerEnabled) Which causes the tests to break as now we only add |
Nice, thanks! |
No description provided.