ASDenysPetrov added a comment.

@NoQ

> Even though NFC commits don't require tests, this doesn't mean they shouldn't 
> add them!

While developing this, I wasn't able to reproduce any regression or unpassed 
cases on the projects from CodeChecker list (//Bitcoin_v0.20.1, 
Curl_curl-7_66_0, Memchached_1.6.8, OpenSSL_openssl-3.0.0-alpha7, etc.//) due 
to my platform. I use Windows and those projects either don't compile or need 
hard effort to set them up.
But anyway I was able to detect some different diagnostics in existing tests 
`casts.cpp` using Z3 refutation option. That actually helped to make 
corrections.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90157

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D90157: [analyzer]... Artem Dergachev via Phabricator via cfe-commits
    • [PATCH] D90157: [anal... Denys Petrov via Phabricator via cfe-commits

Reply via email to