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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to