lebedev.ri added a subscriber: rjmccall.
lebedev.ri added a comment.

In https://reviews.llvm.org/D33826#856619, @Eugene.Zelenko wrote:

> In https://reviews.llvm.org/D33826#856610, @lebedev.ri wrote:
>
> > Any status update here? :)
> >  I generally do see a benefit in this check, because `-Wcast-align` (at 
> > least currently?) does not warn on `reinterpret_cast<>()`.
>
>
> I think will be good idea to extend -Wcast-align.


Hm, are you *sure* `reinterpret_cast<>()` is not allowed to increase alignment?
There was a commit that intentionally removed that warning: (by @rjmccall, i 
believe)
https://github.com/llvm-mirror/clang/commit/35a38d95da89d48778019c37b5f8c9a20f7e309c


Repository:
  rL LLVM

https://reviews.llvm.org/D33826



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to