[clang] [clang] Remove pointless `hasDiagnostics()` checks (PR #172705)
Qiongsi Wu via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 17 10:42:39 PST 2025
qiongsiwu wrote:
Ah thanks for catching this!
Is it also OK to remove the check for these cases?
https://github.com/llvm/llvm-project/blob/7f1a30ebd242b2a55e8393717f1e594f9cd61569/clang/lib/Interpreter/Interpreter.cpp#L115
https://github.com/llvm/llvm-project/blob/7f1a30ebd242b2a55e8393717f1e594f9cd61569/clang/tools/clang-installapi/ClangInstallAPI.cpp#L120
https://github.com/llvm/llvm-project/blob/7f1a30ebd242b2a55e8393717f1e594f9cd61569/clang/tools/driver/cc1_main.cpp#L285
https://github.com/llvm/llvm-project/pull/172705
More information about the cfe-commits
mailing list