BRevzin added inline comments.
================ Comment at: llvm/include/llvm/ADT/DirectedGraph.h:97 + } + friend bool operator!=(const NodeType &M, const NodeType &N) { !(M == N); } ---------------- jfb wrote: > davidstone wrote: > > Missing `return` > 😱 > > Did this not trigger a diagnostic when building? I wonder if it's just not on? Yeah I was surprised too. I'm compiling with `-Wall -Wextra`... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78938/new/ https://reviews.llvm.org/D78938 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits