[clang] [clang] Fix 71315698c9 in presence of incomplete types (PR #114095)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 29 10:43:23 PDT 2024


================
@@ -8900,7 +8900,8 @@ void Sema::CheckMemaccessArguments(const CallExpr *Call,
           << Call->getCallee()->getSourceRange());
     else if (const auto *RT = PointeeTy->getAs<RecordType>()) {
 
-      bool IsTriviallyCopyableCXXRecord =
+      bool MayBeTriviallyCopyableCXXRecord =
+          RT->isIncompleteType() ||
----------------
erichkeane wrote:

I guess this just makes this a 'not foolproof' diagnostic here in C++ mode, which is somewhat unfortunate.  We could perhaps provide a second diagnostic (in a different group?) about using incomplete `CXXRecordType` types, but IDK how that use case works.

https://github.com/llvm/llvm-project/pull/114095


More information about the cfe-commits mailing list