saghir created this revision. Herald added subscribers: steven.zhang, shchenz, kbarton, nemanjai. saghir requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits.
This patch allows MMA builtins on PowerPC to accept restrict qualified pointers. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D106550 Files: clang/lib/Sema/SemaChecking.cpp clang/test/Sema/ppc-pair-mma-types.c Index: clang/test/Sema/ppc-pair-mma-types.c =================================================================== --- clang/test/Sema/ppc-pair-mma-types.c +++ clang/test/Sema/ppc-pair-mma-types.c @@ -333,3 +333,8 @@ __vector_pair vp = __builtin_vsx_lxvp(ll, v); // expected-error {{passing '__vector int' (vector of 4 'int' values) to parameter of incompatible type 'const __vector_pair *'}} __builtin_vsx_stxvp(vp, ll, s); // expected-error {{passing 'unsigned short' to parameter of incompatible type 'const __vector_pair *'}} } + +void testRestrictQualifiedPointer(__vector_quad *__restrict acc) { + vector float arr[4]; + __builtin_mma_disassemble_acc((void*)arr, acc); +} Index: clang/lib/Sema/SemaChecking.cpp =================================================================== --- clang/lib/Sema/SemaChecking.cpp +++ clang/lib/Sema/SemaChecking.cpp @@ -7303,7 +7303,8 @@ if ((ExpectedType->isVoidPointerType() && !ArgType->isPointerType()) || (!ExpectedType->isVoidPointerType() && - ArgType.getCanonicalType() != ExpectedType)) + (!ArgType.isRestrictQualified() && + ArgType.getCanonicalType() != ExpectedType))) return Diag(Arg->getBeginLoc(), diag::err_typecheck_convert_incompatible) << ArgType << ExpectedType << 1 << 0 << 0;
Index: clang/test/Sema/ppc-pair-mma-types.c =================================================================== --- clang/test/Sema/ppc-pair-mma-types.c +++ clang/test/Sema/ppc-pair-mma-types.c @@ -333,3 +333,8 @@ __vector_pair vp = __builtin_vsx_lxvp(ll, v); // expected-error {{passing '__vector int' (vector of 4 'int' values) to parameter of incompatible type 'const __vector_pair *'}} __builtin_vsx_stxvp(vp, ll, s); // expected-error {{passing 'unsigned short' to parameter of incompatible type 'const __vector_pair *'}} } + +void testRestrictQualifiedPointer(__vector_quad *__restrict acc) { + vector float arr[4]; + __builtin_mma_disassemble_acc((void*)arr, acc); +} Index: clang/lib/Sema/SemaChecking.cpp =================================================================== --- clang/lib/Sema/SemaChecking.cpp +++ clang/lib/Sema/SemaChecking.cpp @@ -7303,7 +7303,8 @@ if ((ExpectedType->isVoidPointerType() && !ArgType->isPointerType()) || (!ExpectedType->isVoidPointerType() && - ArgType.getCanonicalType() != ExpectedType)) + (!ArgType.isRestrictQualified() && + ArgType.getCanonicalType() != ExpectedType))) return Diag(Arg->getBeginLoc(), diag::err_typecheck_convert_incompatible) << ArgType << ExpectedType << 1 << 0 << 0;
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits