[clang-tools-extra] [clang-tidy] Diagnose NULL macro expansions in implicit-bool-conversion (PR #188694)

Baranov Victor via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 26 07:24:57 PDT 2026


================
@@ -382,7 +382,8 @@ void ImplicitBoolConversionCheck::handleCastToBool(const ImplicitCastExpr *Cast,
     return;
   }
 
-  auto Diag = diag(Cast->getBeginLoc(), "implicit conversion %0 -> 'bool'")
+  auto Diag = diag(Context.getSourceManager().getFileLoc(Cast->getBeginLoc()),
----------------
vbvictor wrote:

Do we need some new tests for this? Or removing `--system-header` helped somehow?
IIRC `<cstddef>` is already included with -isystem

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


More information about the cfe-commits mailing list