https://github.com/sarnex updated https://github.com/llvm/llvm-project/pull/143160
>From 56f19d4f839d1c65a3f1145fcdcac5207ff61c11 Mon Sep 17 00:00:00 2001 From: "Sarnie, Nick" <nick.sar...@intel.com> Date: Fri, 6 Jun 2025 08:21:56 -0700 Subject: [PATCH 1/2] [clang][AST] Fix spaces TypePrinter for some calling convs Signed-off-by: Sarnie, Nick <nick.sar...@intel.com> --- clang/lib/AST/TypePrinter.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/clang/lib/AST/TypePrinter.cpp b/clang/lib/AST/TypePrinter.cpp index 330cfcd962825..d18723d807c6a 100644 --- a/clang/lib/AST/TypePrinter.cpp +++ b/clang/lib/AST/TypePrinter.cpp @@ -1095,13 +1095,13 @@ void TypePrinter::printFunctionAfter(const FunctionType::ExtInfo &Info, OS << " __attribute__((pcs(\"aapcs-vfp\")))"; break; case CC_AArch64VectorCall: - OS << "__attribute__((aarch64_vector_pcs))"; + OS << " __attribute__((aarch64_vector_pcs))"; break; case CC_AArch64SVEPCS: - OS << "__attribute__((aarch64_sve_pcs))"; + OS << " __attribute__((aarch64_sve_pcs))"; break; case CC_DeviceKernel: - OS << "__attribute__((device_kernel))"; + OS << " __attribute__((device_kernel))"; break; case CC_IntelOclBicc: OS << " __attribute__((intel_ocl_bicc))"; >From c99bb95b1725bc28d48c97edd4a4ad6f057210a9 Mon Sep 17 00:00:00 2001 From: "Sarnie, Nick" <nick.sar...@intel.com> Date: Fri, 6 Jun 2025 09:57:31 -0700 Subject: [PATCH 2/2] add test Signed-off-by: Sarnie, Nick <nick.sar...@intel.com> --- clang/test/AST/ast-dump-type-callingconv.cpp | 24 ++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 clang/test/AST/ast-dump-type-callingconv.cpp diff --git a/clang/test/AST/ast-dump-type-callingconv.cpp b/clang/test/AST/ast-dump-type-callingconv.cpp new file mode 100644 index 0000000000000..d7d6b823c5244 --- /dev/null +++ b/clang/test/AST/ast-dump-type-callingconv.cpp @@ -0,0 +1,24 @@ +// Verify there is a space after the parens when priting callingconv attributes. +// RUN: %clang_cc1 -DDEVICE -triple spirv64 -ast-dump -ast-dump-filter foo %s \ +// RUN: | FileCheck -check-prefix=CHECK-DEVICE --strict-whitespace %s + +// RUN: %clang_cc1 -DVECTOR -triple aarch64 -ast-dump -ast-dump-filter foo %s \ +// RUN: | FileCheck -check-prefix=CHECK-VECTOR --strict-whitespace %s + +// RUN: %clang_cc1 -DSVE -triple aarch64 -ast-dump -ast-dump-filter foo %s \ +// RUN: | FileCheck -check-prefix=CHECK-SVE --strict-whitespace %s + +#ifdef DEVICE +// CHECK-DEVICE-NOT: ()__attribute__((device_kernel)) +void foo() __attribute__((device_kernel)); +#endif + +#ifdef VECTOR +// CHECK-VECTOR-NOT: ()__attribute__((aarch64_vector_pcs)) +void foo() __attribute__((aarch64_vector_pcs)); +#endif + +#ifdef SVE +// CHECK-SVE-NOT: ()__attribute__((aarch64_sve_pcs)) +void foo() __attribute__((aarch64_sve_pcs)); +#endif _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits