Skip to content

Commit 826c93f

Browse files
authored
[AST] Use explicit type erasure in TypeSourceInfo constructor (#68435)
When this file is included in a project compiled with GCC 13 `-Werror`, compilation fails with the following diagnostic: <pre> /usr/lib/llvm-17.0/include/clang/AST/TypeLoc.h: In constructor 'clang::TypeSourceInfo::TypeSourceInfo(clang::QualType, size_t)': /usr/lib/llvm-17.0/include/clang/AST/TypeLoc.h:245:9: error: 'void* memset(void*, int, size_t)' clearing an object of non-trivial type 'class clang::TypeSourceInfo'; use assignment instead [-Werror=class-memaccess] 245 | memset(this + 1, 0, DataSize); | ~~~~~~^~~~~~~~~~~~~~~~~~~~~~~ </pre> To avoid this, we add an explicit type cast. The cast to `void*` makes sense, since other member functions of `TypeSourceInfo` also treat the buffer `(this + 1)` of length `DataSize` as opaque memory, and was likely left out by mistake. Fixes: 4498663 ("[AST] Initialized data after TypeSourceInfo") I also suggest to apply this to release/17.x if possible.
1 parent cb5612c commit 826c93f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

clang/include/clang/AST/TypeLoc.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -243,7 +243,7 @@ class TypeLoc {
243243

244244
inline TypeSourceInfo::TypeSourceInfo(QualType ty, size_t DataSize) : Ty(ty) {
245245
// Init data attached to the object. See getTypeLoc.
246-
memset(this + 1, 0, DataSize);
246+
memset(static_cast<void *>(this + 1), 0, DataSize);
247247
}
248248

249249
/// Return the TypeLoc for a type source info.

0 commit comments

Comments
 (0)