[llvm] [Coroutines][LazyCallGraph] resumes are not really SCC (PR #116285)

Arthur Eubanks via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 10 21:02:27 PST 2024


================
@@ -1775,8 +1793,9 @@ void LazyCallGraph::addSplitRefRecursiveFunctions(
       if (F1 == F2)
         continue;
       Node &N2 = get(*F2);
-      assert(!N1->lookup(N2)->isCall() &&
-             "Edges between new functions must be ref edges");
+      assert(N1->lookup(N2) &&
+             (!N1->lookup(N2)->isCall() &&
----------------
aeubanks wrote:

no need for extra parentheses

is this change just so we don't crash in the assert if the edge doesn't exist?

https://github.com/llvm/llvm-project/pull/116285


More information about the llvm-commits mailing list