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

[llvm][llvm-lit] Enable internal shell for llvm test suite #102697

Open
Harini0924 opened this issue Aug 9, 2024 · 0 comments · May be fixed by #106324
Open

[llvm][llvm-lit] Enable internal shell for llvm test suite #102697

Harini0924 opened this issue Aug 9, 2024 · 0 comments · May be fixed by #106324
Labels
llvm Umbrella label for LLVM issues test-suite

Comments

@Harini0924
Copy link
Contributor

Harini0924 commented Aug 9, 2024

Testing in lit is currently inconsistent across different platforms, as many RUN line commands are written in a way that only works on POSIX-compliant shells. Lit has an internal shell implementation to ensure uniform execution of these commands that can be used regardless of platform. We should aim to turn on the internal shell for the llvm test suite to eliminate reliance on external shells.

The internal shell has missing capabilities that need to be addressed before doing so. Here is a list of the issues that block us from turning on the internal shell for llvm.

#102374
#102383
#102395
#106627

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
llvm Umbrella label for LLVM issues test-suite
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants