Skip to content

Conversation

@fmayer
Copy link
Contributor

@fmayer fmayer commented Aug 6, 2025

It incorrectly stated that const char* gets normalized to ptr, while it should say that char* does.

It incorrectly stated that `const char*` gets normalized to ptr, while
it should say that `char*` does.
@fmayer fmayer requested a review from thurstond August 6, 2025 22:31
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Aug 6, 2025
@llvmbot
Copy link
Member

llvmbot commented Aug 6, 2025

@llvm/pr-subscribers-clang

Author: Florian Mayer (fmayer)

Changes

It incorrectly stated that const char* gets normalized to ptr, while it should say that char* does.


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

1 Files Affected:

  • (modified) clang/test/CodeGen/cfi-icall-generalize.c (+1-1)
diff --git a/clang/test/CodeGen/cfi-icall-generalize.c b/clang/test/CodeGen/cfi-icall-generalize.c
index 2011889fcb400..0af17e5760cc6 100644
--- a/clang/test/CodeGen/cfi-icall-generalize.c
+++ b/clang/test/CodeGen/cfi-icall-generalize.c
@@ -1,7 +1,7 @@
 // RUN: %clang_cc1 -triple x86_64-unknown-linux -fsanitize=cfi-icall -fsanitize-trap=cfi-icall -emit-llvm -o - %s | FileCheck --check-prefix=CHECK --check-prefix=UNGENERALIZED %s
 // RUN: %clang_cc1 -triple x86_64-unknown-linux -fsanitize=cfi-icall -fsanitize-trap=cfi-icall -fsanitize-cfi-icall-generalize-pointers -emit-llvm -o - %s | FileCheck --check-prefix=CHECK --check-prefix=GENERALIZED %s
 
-// Test that const char* is generalized to const ptr and that const char** is
+// Test that const char* is generalized to const ptr and that char** is
 // generalized to ptr
 
 // CHECK: define{{.*}} ptr @f({{.*}} !type [[TYPE:![0-9]+]] !type [[TYPE_GENERALIZED:![0-9]+]]

@fmayer fmayer merged commit a7f1702 into main Aug 6, 2025
11 checks passed
@fmayer fmayer deleted the users/fmayer/fmayer-patch-2 branch August 6, 2025 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants