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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
