[llvm] [ASan][AMDGPU] Fix Assertion Failure. (PR #78710)

Vitaly Buka via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 23 11:23:59 PST 2024


================
@@ -1255,7 +1255,8 @@ void AddressSanitizer::instrumentMemIntrinsic(MemIntrinsic *MI) {
   InstrumentationIRBuilder IRB(MI);
   if (isa<MemTransferInst>(MI)) {
     IRB.CreateCall(isa<MemMoveInst>(MI) ? AsanMemmove : AsanMemcpy,
-                   {MI->getOperand(0), MI->getOperand(1),
+                   {IRB.CreateAddrSpaceCast(MI->getOperand(0), PtrTy),
+                    IRB.CreateAddrSpaceCast(MI->getOperand(1), PtrTy),
----------------
vitalybuka wrote:

I guess never-`const` applies to the first arg of memcpy, but the patch adds CreateAddrSpaceCast?
So maybe use CreateAddrSpaceCast for all intrinsics for consistency
 

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


More information about the llvm-commits mailing list