dexonsmith added inline comments.
================ Comment at: clang/lib/Frontend/CompilerInvocation.cpp:3096 + + return Success; } ---------------- Would this avoid the changes above? ``` return Success && !Diags.hasErrorOccurred(); ``` ? Alternatively, might it be cleaner to shove the `Success = false` into a helper lambda or something? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95792/new/ https://reviews.llvm.org/D95792 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
