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

Thu Jun 17 15:09:27 PDT 2021

ASDenysPetrov added a comment.

@NoQ:

> 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.

Bisecting through a year+ of commits is painful. I thought about this but didn't dare. What I can do is to check the failure on a issue-date commit.

================
Comment at: clang/test/Analysis/diagnostics/PR46264.cpp:4
+// PR46264
+// This case shall not warn about dereference of void*
+namespace ns1 {
----------------
NoQ wrote:
> I think you mean this case shall not crash with an assertion failure.
> 
> "Warn" exclusively means "display a warning to the user".
I'll make changes.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104381