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

CI tests for LLVM 10, 14, 15 and 16 #2754

Merged
merged 8 commits into from
Jul 27, 2024

Conversation

Vipul-Cariappa
Copy link
Contributor

@Vipul-Cariappa Vipul-Cariappa commented Jul 5, 2024

@Vipul-Cariappa
Copy link
Contributor Author

Vipul-Cariappa commented Jul 5, 2024

It looks like we broke something with LLVM 14. The build is successful, but running the tests fails with segfault.

EDITED
I am not able to reproduce the bug/error locally. All tests are passing in my local system.

@Vipul-Cariappa Vipul-Cariappa marked this pull request as draft July 6, 2024 03:09
@Vipul-Cariappa
Copy link
Contributor Author

@Shaikh-Ubaid, I am not able to replicate the error on my local computer. Could you please have a look at this.
Only LLVM 14 fails. Build is successful, but ctest fails with segfault.

@Shaikh-Ubaid
Copy link
Collaborator

Only LLVM 14 fails. Build is successful, but ctest fails with segfault.

LLVM 14 passes for me on mac. Need to check with Linux.

@Vipul-Cariappa
Copy link
Contributor Author

Looks like integration tests pass, but ctest (i.e. REPL related tests) fail.

@Vipul-Cariappa
Copy link
Contributor Author

Merging this PR without the tests for LLVM 14. I have opened an issue #2792.

@Shaikh-Ubaid, please have a look at it.

@Vipul-Cariappa Vipul-Cariappa marked this pull request as ready for review July 27, 2024 03:30
@Shaikh-Ubaid Shaikh-Ubaid merged commit 517da5c into lcompilers:main Jul 27, 2024
18 checks passed
@Vipul-Cariappa Vipul-Cariappa deleted the llvm-ci branch July 27, 2024 12:23
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