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.

Sign up for GitHub

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

Jump to bottom

[llvm-lit] InternalShellError in compiler-rt test using export command #102386

Closed
connieyzhu opened this issue Aug 7, 2024 · 0 comments · Fixed by #105961
Closed

[llvm-lit] InternalShellError in compiler-rt test using export command #102386

connieyzhu opened this issue Aug 7, 2024 · 0 comments · Fixed by #105961
Labels
compiler-rt:asan Address sanitizer test-suite

Comments

@connieyzhu
Copy link
Contributor

A compiler-rt test using the export command is failing when run with lit’s internal shell.

Error message:

UNRESOLVED: AddressSanitizer-x86_64-linux :: TestCases/Posix/high-address-dereference.c (129 of 9570)
******************** TEST 'AddressSanitizer-x86_64-linux :: TestCases/Posix/high-address-dereference.c' FAILED ********************
Exception during script execution:
TypeError: InternalShellError.__init__() missing 1 required positional argument: 'message'
This was referenced Aug 9, 2024
Open
Open
@connieyzhu connieyzhu mentioned this issue Aug 24, 2024
Merged
connieyzhu added a commit that referenced this issue Aug 27, 2024
@connieyzhu
[llvm-lit][test] Resolved typo in raising InternalShellError for expo…
1990d8d 
…rt command in lit's internal shell (#105961)

This patch fixes the incorrect usage of lit's built-in `export` command.
There is a typo in raising the error itself where the error being raised
had the wrong number of parameters passed in.

Fixes #102386.
5c4lar pushed a commit to 5c4lar/llvm-project that referenced this issue Aug 29, 2024
@connieyzhu @5c4lar
[llvm-lit][test] Resolved typo in raising InternalShellError for expo…
9446856 
…rt command in lit's internal shell (llvm#105961)

This patch fixes the incorrect usage of lit's built-in `export` command.
There is a typo in raising the error itself where the error being raised
had the wrong number of parameters passed in.

Fixes llvm#102386.
dmpolukhin pushed a commit to dmpolukhin/llvm-project that referenced this issue Sep 2, 2024
@connieyzhu @dmpolukhin
[llvm-lit][test] Resolved typo in raising InternalShellError for expo…
3ed9370 
…rt command in lit's internal shell (llvm#105961)

This patch fixes the incorrect usage of lit's built-in `export` command.
There is a typo in raising the error itself where the error being raised
had the wrong number of parameters passed in.

Fixes llvm#102386.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
compiler-rt:asan Address sanitizer test-suite
Projects
None yet
Milestone
No milestone
2 participants
@EugeneZelenko @connieyzhu