From 4e8a02e7f49dd6aa7f2a6003a2556fda639713c9 Mon Sep 17 00:00:00 2001 From: Johannes Doerfert Date: Sat, 8 Jan 2022 21:11:51 -0600 Subject: [PATCH] [Attributor][FIX] Remove assumption that doesn't have to hold There is no guarantee we strip all GEPOperators and the conservative handling doesn't even require us to. --- llvm/lib/Transforms/IPO/AttributorAttributes.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/llvm/lib/Transforms/IPO/AttributorAttributes.cpp b/llvm/lib/Transforms/IPO/AttributorAttributes.cpp index de7244a..07c16c7 100644 --- a/llvm/lib/Transforms/IPO/AttributorAttributes.cpp +++ b/llvm/lib/Transforms/IPO/AttributorAttributes.cpp @@ -7702,7 +7702,6 @@ void AAMemoryLocationImpl::categorizePtrValue( for (Value *Obj : Objects) { // TODO: recognize the TBAA used for constant accesses. MemoryLocationsKind MLK = NO_LOCATIONS; - assert(!isa(Obj) && "GEPs should have been stripped."); if (isa(Obj)) continue; if (isa(Obj)) { -- 2.7.4