-
Notifications
You must be signed in to change notification settings - Fork 584
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
Move ASAN over to new CI system #9958
Conversation
7d3b116
to
fe4a8b7
Compare
@@ -121,6 +121,9 @@ label_exclude_regex="($(IFS="|" ; echo "${label_exclude_args[*]?}"))" | |||
# TODO(#5715): Fix these | |||
declare -a excluded_tests=( | |||
"iree/samples/simple_embedding/simple_embedding_vulkan_test" | |||
"iree/tools/test/iree-benchmark-module.mlir.test" |
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.
Hmm I'm confused why an error here would only be popping up now?
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.
So I found the same tests marked as failure in the attached PR. Were we still running asan in the old CI?
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.
Yeah we still are. Recent passing run: https://source.cloud.google.com/results/invocations/181f49e5-b0e6-4652-93fb-a79a2b06b694
I'm guessing this is a vulkan thing. It's disabled entirely for the current asan build:
export IREE_VULKAN_DISABLE=${IREE_VULKAN_DISABLE:-1} |
Co-authored-by: Geoffrey Martin-Noble <gcmn@google.com>
Co-authored-by: Geoffrey Martin-Noble <gcmn@google.com>
Part of #9855