[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