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

Test MatrixTransposeWithF is unstable #1246

Open
bobismijnnaam opened this issue Sep 25, 2024 · 2 comments
Open

Test MatrixTransposeWithF is unstable #1246

bobismijnnaam opened this issue Sep 25, 2024 · 2 comments

Comments

@bobismijnnaam
Copy link
Contributor

Test MatrixTransposeWithF in suite SYCLFullProgramsSpec is unstable. It is disabeld for now until a solution is found.

@OmerSakar
Copy link
Member

To give my two cents:

The problem seems to be that the example times out due to the parsing taking too long. The C++ grammar from ANTLR is inefficient with the way we are parsing things in VerCors.

I guess the problem would be fixed if we rewrite the C++ grammar to have a similar style to the grammar of the other languages.

@superaxander
Copy link
Member

I think the parsing is not the problem for the CI because what times out is some assert which it then concludes can't be proven (probably because the CI is slower than your local machine) On my branches with changes to the C++ struct encoding things are a bit slower and that causes this example to always fail, even locally. So I think it might just be an example that is on the edge of failing locally and then fails on a slower machine more often. Maybe adding the extra set of curly braces such that the buffer is definitely accessible can help in some cases but I don't have a consistent solution that will always work.

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

3 participants