Skip to content

Conversation

@melver
Copy link
Contributor

@melver melver commented Nov 18, 2025

Test that -fsanitize=alloc-token is compatible with kcfi and memtag, as these should also be possible to combine.

NFC.

Test that -fsanitize=alloc-token is compatible with kcfi and memtag, as
these should also be possible to combined.

NFC.
@llvmbot llvmbot added clang Clang issues not falling into any other category clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' labels Nov 18, 2025
@llvmbot
Copy link
Member

llvmbot commented Nov 18, 2025

@llvm/pr-subscribers-clang-driver

@llvm/pr-subscribers-clang

Author: Marco Elver (melver)

Changes

Test that -fsanitize=alloc-token is compatible with kcfi and memtag, as these should also be possible to combined.

NFC.


Full diff: https://github.com/llvm/llvm-project/pull/168600.diff

1 Files Affected:

  • (modified) clang/test/Driver/fsanitize-alloc-token.c (+1)
diff --git a/clang/test/Driver/fsanitize-alloc-token.c b/clang/test/Driver/fsanitize-alloc-token.c
index 6d8bda16dfb96..0ffe9abad8053 100644
--- a/clang/test/Driver/fsanitize-alloc-token.c
+++ b/clang/test/Driver/fsanitize-alloc-token.c
@@ -5,6 +5,7 @@
 // CHECK-NO-TOKEN-ALLOC-NOT: "-fsanitize=alloc-token"
 
 // RUN: %clang --target=x86_64-linux-gnu -flto -fvisibility=hidden -fno-sanitize-ignorelist -fsanitize=alloc-token,undefined,cfi %s -### 2>&1 | FileCheck %s --check-prefix=CHECK-COMPATIBLE
+// RUN: %clang --target=aarch64-linux-android -march=armv8-a+memtag -flto -fvisibility=hidden -fsanitize=alloc-token,kcfi,memtag %s -### 2>&1 | FileCheck %s --check-prefix=CHECK-COMPATIBLE
 // CHECK-COMPATIBLE: "-fsanitize={{.*}}alloc-token"
 
 // RUN: %clang --target=x86_64-linux-gnu -fsanitize=alloc-token -fsanitize-minimal-runtime %s -### 2>&1 | FileCheck %s --check-prefix=CHECK-MINIMAL

@melver melver merged commit 8aca6c3 into llvm:main Nov 18, 2025
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants