From 39df40d8c2f89a084276b332b88b0633444db5ec Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Sat, 5 Aug 2017 04:04:06 +0000 Subject: [LCG] Replace an implicit bool operator with a named function. (NFC) The definition of 'false' here was already pretty vague and debatable, and I'm about to add another potential 'false' that would actually make much more sense in a bool operator. Especially given how rarely this is used, a nicely named method seems better. llvm-svn: 310165 --- llvm/lib/Analysis/LazyCallGraph.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'llvm/lib/Analysis/LazyCallGraph.cpp') diff --git a/llvm/lib/Analysis/LazyCallGraph.cpp b/llvm/lib/Analysis/LazyCallGraph.cpp index 7d38dd945ab..46e29828990 100644 --- a/llvm/lib/Analysis/LazyCallGraph.cpp +++ b/llvm/lib/Analysis/LazyCallGraph.cpp @@ -212,7 +212,7 @@ void LazyCallGraph::SCC::verify() { assert(N->LowLink == -1 && "Must set low link to -1 when adding a node to an SCC!"); for (Edge &E : **N) - assert(E.getNode() && "Can't have an unpopulated node!"); + assert(E.getNode().isPopulated() && "Can't have an unpopulated node!"); } } #endif @@ -1649,7 +1649,7 @@ void LazyCallGraph::removeDeadFunction(Function &F) { for (RefSCC &ParentRC : RC.parents()) for (SCC &ParentC : ParentRC) for (Node &ParentN : ParentC) - if (ParentN) + if (ParentN.isPopulated()) ParentN->removeEdgeInternal(N); // Now remove this RefSCC from any parents sets and the leaf list. -- cgit v1.2.3