[clang] Skip over std namespace in WebKit checkers. (PR #90552)

2024-05-04 Thread Ryosuke Niwa via cfe-commits
rniwa wrote: Closing this in favor of https://github.com/llvm/llvm-project/pull/91103. https://github.com/llvm/llvm-project/pull/90552 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Skip over std namespace in WebKit checkers. (PR #90552)

2024-05-04 Thread Ryosuke Niwa via cfe-commits
https://github.com/rniwa closed https://github.com/llvm/llvm-project/pull/90552 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Skip over std namespace in WebKit checkers. (PR #90552)

2024-05-01 Thread Artem Dergachev via cfe-commits
haoNoQ wrote: Hmm it doesn't look like we actually have that other suppression. It looks like the checker simply never warned in the standard library because there aren't WebKit classes in the standard library. But it looks like you've found a few cases where it actually happens? How does

[clang] Skip over std namespace in WebKit checkers. (PR #90552)

2024-04-30 Thread Ryosuke Niwa via cfe-commits
rniwa wrote: > Ok am I reading this right: this is about testing code that lives directly in > namespace `std::` but somehow still lives in your project right? Such as your > custom modifications to standard classes, various overloads and > specializations of standard things. No, this is

[clang] Skip over std namespace in WebKit checkers. (PR #90552)

2024-04-30 Thread Artem Dergachev via cfe-commits
haoNoQ wrote: (If yes, LGTM!) https://github.com/llvm/llvm-project/pull/90552 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Skip over std namespace in WebKit checkers. (PR #90552)

2024-04-30 Thread Artem Dergachev via cfe-commits
haoNoQ wrote: Ok am I reading this right: this is about testing code that lives directly in namespace `std::` but somehow still lives in your project right? Such as your custom modifications to standard classes, various overloads and specializations of standard things. This patch does not

[clang] Skip over std namespace in WebKit checkers. (PR #90552)

2024-04-29 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Ryosuke Niwa (rniwa) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/90552.diff 4 Files Affected: - (modified) clang/lib/StaticAnalyzer/Checkers/WebKit/UncountedCallArgsChecker.cpp (+6) - (modified)

[clang] Skip over std namespace in WebKit checkers. (PR #90552)

2024-04-29 Thread Ryosuke Niwa via cfe-commits
https://github.com/rniwa created https://github.com/llvm/llvm-project/pull/90552 None >From fa7a6e376b07ae6262dd06920c87dc69705a646d Mon Sep 17 00:00:00 2001 From: Ryosuke Niwa Date: Mon, 29 Apr 2024 20:24:24 -0700 Subject: [PATCH] Skip over std namespace in WebKit checkers. ---