From 42b44fef7611cb1bfe851533f27e4be67e55571f Mon Sep 17 00:00:00 2001 From: OCHyams Date: Fri, 31 Mar 2023 14:17:52 +0100 Subject: [PATCH] [Assignment Tracking] Remove assertion from DbgAssignIntrinsic::setAddress Follow up to https://reviews.llvm.org/D146987. Remove assertion that the Value must be a pointer type. This fires in real-world examples e.g. by codegenprepare introducing ptrtoint conversions. The buildbots have not caught up yet but without this change the test compiler-rt/test/ubsan/TestCases/TypeCheck/vptr.cpp fails with an ICE. --- llvm/lib/IR/IntrinsicInst.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/llvm/lib/IR/IntrinsicInst.cpp b/llvm/lib/IR/IntrinsicInst.cpp index e120491..1c78d4b 100644 --- a/llvm/lib/IR/IntrinsicInst.cpp +++ b/llvm/lib/IR/IntrinsicInst.cpp @@ -213,8 +213,6 @@ void DbgAssignIntrinsic::setAssignId(DIAssignID *New) { } void DbgAssignIntrinsic::setAddress(Value *V) { - assert(V->getType()->isPointerTy() && - "Destination Component must be a pointer type"); setOperand(OpAddress, MetadataAsValue::get(getContext(), ValueAsMetadata::get(V))); } -- 2.7.4