rtrieu would be a more appropriate reviewer for this.
On Wed, Sep 11, 2013 at 1:15 PM, Ismail Pazarbasi < [email protected]> wrote: > > Ping! This also fixes PR17053. > > > ================ > Comment at: lib/Lex/PPLexerChange.cpp:280 > @@ +279,3 @@ > + << ControllingMacro > + << FixItHint::CreateReplacement( > + > CurPPLexer->MIOpt.GetDefinedLocation(), > ---------------- > I haven't tried thoroughly, but I guess it'd be better to print the fix-it > on the warning instead of the note, because clang-check will ignore the > note (and I need clang-check to apply the fixit). > > > http://llvm-reviews.chandlerc.com/D1506 >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
