[PATCH] D93822: [clang][Sema] Add diagnostics for implicit widening of multiplication result
Roman Lebedev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 12 10:37:18 PST 2021
lebedev.ri added a comment.
@aaron.ballman thank you for taking a look!
@NoQ thank you for commenting.
Indeed, i very much don't want to diagnose just the cases
where the overflow can be proven to happen, doing that
i believe, would remove most of the true-positive reports.
So indeed, this is beginning to look like a coding guideline,
so let's circumvent all this FUD, and move onto clang-tidy.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93822/new/
https://reviews.llvm.org/D93822
More information about the cfe-commits
mailing list