[PATCH] D104381: [analyzer] Added a test case for PR46264

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 17 11:11:18 PDT 2021


NoQ added a comment.

Thanks!

I guess there might be some value in building an older Clang from around when the bug was reported so that to see if the crash was indeed there and we're not just reproducing it wrong. It would also allow us to understand where we fixed it through bisecting. But I don't insist.



================
Comment at: clang/test/Analysis/diagnostics/PR46264.cpp:4
+// PR46264
+// This case shall not warn about dereference of void*
+namespace ns1 {
----------------
I think you mean this case shall not crash with an assertion failure.

"Warn" exclusively means "display a warning to the user".


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104381/new/

https://reviews.llvm.org/D104381



More information about the cfe-commits mailing list