Looks fine, the lexer does this, as well as other places. Can you add an -frewrite-includes test for this? I think you can rewrite, then run -cc1 -verify with // expected-no-diagnostics or something like that.
On Wed, Nov 27, 2013 at 3:50 AM, Lubos Lunak <[email protected]> wrote: > > Hello, > > could somebody please review the attached simple patch for PR15664? > > The patch intentionally doesn't do anything fancy except simply removing > the > BOM (which should be unnecessary for UTF-8 anyway). I'm not sure what > handling other UTF encodings would exactly require, and it's surely better > to > keep the compilation obviously fail for whoever will possibly run into that > one day and provide a testcase. > > -- > Lubos Lunak > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
