[DebugInfo] Remove an unnecessary cast (NFC) (llvm#146809)

kazutakahirata · web-flow · commit 4cd9c894d1ef · 2025-07-03T09:46:02.000-07:00
Mem is already of uint8_t *.
diff --git a/llvm/lib/DebugInfo/PDB/Native/GSIStreamBuilder.cpp b/llvm/lib/DebugInfo/PDB/Native/GSIStreamBuilder.cpp
@@ -119,7 +119,7 @@ static CVSymbol serializePublic(uint8_t *Mem, const BulkPublic &Pub) {
   memcpy(NameMem, Pub.Name, NameLen);
   // Zero the null terminator and remaining bytes.
   memset(&NameMem[NameLen], 0, Size - sizeof(PublicSym32Layout) - NameLen);
-  return CVSymbol(ArrayRef(reinterpret_cast<uint8_t *>(Mem), Size));
+  return CVSymbol(ArrayRef(Mem, Size));
 }
 
 uint32_t GSIHashStreamBuilder::calculateSerializedLength() const {

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,7 @@ static CVSymbol serializePublic(uint8_t *Mem, const BulkPublic &Pub) {`
`119`	`119`	`memcpy(NameMem, Pub.Name, NameLen);`
`120`	`120`	`// Zero the null terminator and remaining bytes.`
`121`	`121`	`memset(&NameMem[NameLen], 0, Size - sizeof(PublicSym32Layout) - NameLen);`
`122`		`- return CVSymbol(ArrayRef(reinterpret_cast<uint8_t *>(Mem), Size));`
	`122`	`+ return CVSymbol(ArrayRef(Mem, Size));`
`123`	`123`	`}`
`124`	`124`
`125`	`125`	`uint32_t GSIHashStreamBuilder::calculateSerializedLength() const {`