[PATCH] D13313: [clang-tidy] new check misc-no-reinterpret-cast

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 1 06:00:54 PDT 2015


alexfh added a comment.

In http://reviews.llvm.org/D13313#256982, @vsk wrote:

> The patch lgtm. Has there been a discussion on cfe-dev about adopting these guidelines?
>
> I count well over 500 uses of reinterpret_cast in llvm+clang, so we might not all be on the same page on this.


I don't think LLVM can adopt a significant part of C++ Core Guidelines. Nevertheless, it makes sense to have these rules implemented as clang-tidy checks for the projects that decide to adopt them.


http://reviews.llvm.org/D13313





More information about the cfe-commits mailing list