[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