[compiler-rt] [hwasan][test] Fix regex so deep-recursion.c is unsupported on aarch64 targets (PR #69254)

Paul Kirth via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 16 14:41:49 PDT 2023


https://github.com/ilovepi created https://github.com/llvm/llvm-project/pull/69254

After 144c5b6d58803a2d4a0fe92a0fe331ff0347dc3b, we still see this test running in CI for aarch64-linux targets. This appears to be related to the triple being `aarch64-unknown-linux-gnu`, or similar.

The bot link below includes 144c5b6d58803a2d4a0fe92a0fe331ff0347dc3b, and fails the deep-recursion.c test, which should have been disabled. https://luci-milo.appspot.com/ui/p/fuchsia/builders/toolchain.ci/clang-linux-arm64/b8767065085790662609/overview

>From 8bd4319a74f745fc3ad2b16e6d61f4f7cddaf949 Mon Sep 17 00:00:00 2001
From: Paul Kirth <paulkirth at google.com>
Date: Mon, 16 Oct 2023 21:33:31 +0000
Subject: [PATCH] [hwasan][test] Update regex so the test is ignored

After 144c5b6d58803a2d4a0fe92a0fe331ff0347dc3b, we still see this test
running in CI for aarch64-linux targets. This appears to be related to
the triple being `aarch64-unknown-linux-gnu`, or similar.

The bot link below includes 144c5b6d58803a2d4a0fe92a0fe331ff0347dc3b,
and fails the deep-recursion.c test, which should have been disabled.
https://luci-milo.appspot.com/ui/p/fuchsia/builders/toolchain.ci/clang-linux-arm64/b8767065085790662609/overview
---
 compiler-rt/test/hwasan/TestCases/deep-recursion.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/compiler-rt/test/hwasan/TestCases/deep-recursion.c b/compiler-rt/test/hwasan/TestCases/deep-recursion.c
index 39902d072a0d34a..bf390d051d472e7 100644
--- a/compiler-rt/test/hwasan/TestCases/deep-recursion.c
+++ b/compiler-rt/test/hwasan/TestCases/deep-recursion.c
@@ -18,7 +18,7 @@
 // XFAIL: target=x86_64{{.*}}
 
 // Flaky on AArch64 Linux, see https://github.com/llvm/llvm-project/issues/69221.
-// UNSUPPORTED: target=aarch64-linux{{.*}}
+// UNSUPPORTED: target=aarch64{{.*}}
 
 #include <stdlib.h>
 // At least -O1 is needed for this function to not have a stack frame on



More information about the llvm-commits mailing list