[PATCH] D29577: [PM/LCG] Teach the LazyCallGraph how to replace a function without disturbing the graph or having to update edges.

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc marked an inline comment as done.
chandlerc added inline comments.


================
Comment at: include/llvm/Analysis/LazyCallGraph.h:330-331
+
+    /// Tests whether the node has been populated with edges.
+    operator bool() const { return (bool)Edges; }
+
----------------
davide wrote:
> Not sure if `static_cast<bool>` works here, but I would prefer that, if possible.
Sure, but we can do better with Optional::hasValue.


https://reviews.llvm.org/D29577