=?utf-8?q?Balázs_Kéri?= <[email protected]>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/[email protected]>


https://github.com/zeyi2 commented:

I ran this check over `cppcheck`, `curl` and `clang`. Report is here: 
https://broken.life/CTIT/20260208_173913_PR180219_bugproneassignmentinselec/index.html

(The testing infra is still buggy and unstable, so there are some unrelated 
errors..)

The warnings seem consistent with the documentation, but I'm not sure whether 
common patterns like `if ((x = dyn_cast<...>(...)) && ...)` should be 
considered as a FP. IMO they are safe and the extra parentheses signal that the 
assignment is intentional.

https://github.com/llvm/llvm-project/pull/180219
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to