From e74d6269259e28225a494ce98bd1762cdac1fe89 Mon Sep 17 00:00:00 2001 From: Simon Pilgrim Date: Thu, 11 Mar 2021 10:29:02 +0000 Subject: [PATCH] [IPO] Fix EXPENSIVE_CHECKS assert added at D83744. NFCI. It wasn't taking into account that QueryingAA was a pointer. --- llvm/include/llvm/Transforms/IPO/Attributor.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/llvm/include/llvm/Transforms/IPO/Attributor.h b/llvm/include/llvm/Transforms/IPO/Attributor.h index 86f08ac..cbf33bc 100644 --- a/llvm/include/llvm/Transforms/IPO/Attributor.h +++ b/llvm/include/llvm/Transforms/IPO/Attributor.h @@ -1127,11 +1127,11 @@ struct Attributor { DepClassTy DepClass, bool ForceUpdate = false) { #ifdef EXPENSIVE_CHECKS // Don't allow callbase information to leak. - if (auto CBContext = IRP.getCallBaseContext()) { + if (auto *CBContext = IRP.getCallBaseContext()) { assert( ((CBContext->getCalledFunction() == IRP.getAnchorScope() || - QueryingAA || - !QueryingAA.getIRPosition().isAnyCallSitePosition())) && + !QueryingAA || + !QueryingAA->getIRPosition().isAnyCallSitePosition())) && "non callsite positions are not allowed to propagate CallBaseContext " "across functions"); } -- 2.7.4