[PATCH] D38455: [clang-tidy] new cppcoreguidelines-narrowing-conversions check.
Alexander Kornienko via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 14 08:08:55 PDT 2018
alexfh added a comment.
Herald added subscribers: llvm-commits, klimek.
Now I finally got around to this.
Since the "Enforcement" part of the relevant C++ Core Guidelines rule is not very helpful (kind of a "hey, we know enforcing this rule will lead to tons of false positive, so we don't know what exactly makes sense to enforce"), this may well be a good first step. I personally have no concerns. Maybe someone more interested in this module will have something to say though.
Could you rebase the patch against current head?
Repository:
rL LLVM
https://reviews.llvm.org/D38455
More information about the llvm-commits
mailing list