[PATCH] Fixit for -Wreinterpret-case-class

[Alexander Zinenko]

SemaDecl.cpp comment says

#include "clang/Lex/Preprocessor.h" // FIXME: Sema shouldn't depend on Lex

that's why I didn't use it.  Is it ok to use Preprocessor in Sema?



On 28 March 2013 18:47, Jordan Rose <jordan_rose at apple.com> wrote:

> I don't think you actually need to pass the range around.
> PP.getLocForEndOfToken should work fine. (Sorry I didn't notice this the
> first time around.)
>
> On Mar 28, 2013, at 9:35 , Alexander Zinenko <ftynse at gmail.com> wrote:
>
> > Hello
> >
> > This patch adds fixits to -Wreinterpret-base-class proposing to replace
> reinterpret_cast with static_cast. It required passing SourceRange instead
> of SourceLocation through some calls in Sema.
> >
> > Review please!
> >
> > --
> > Alex
> >
> > <13824fixit.patch>_______________________________________________
> > cfe-commits mailing list
> > cfe-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130328/16b4690f/attachment.html>