vsapsai added inline comments.

================
Comment at: clang/lib/Lex/Lexer.cpp:2012-2015
+    // Skip escaped characters.  Escaped newlines will already be processed by
+    // getAndAdvanceChar.
+    if (C == '\\')
+      C = getAndAdvanceChar(CurPtr, Result);
----------------
rsmith wrote:
> You can just delete these four lines entirely.
It will make Clang reject previously accepted `#include <test\>escape.h>` 
That's what we want, right? I agree with having no support for such file names, 
just want to confirm. For the reference, proposed change would match gcc 5.4.0 
behaviour. gcc 6.1 and higher rejects such include too but in a different way.


https://reviews.llvm.org/D41423



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

Reply via email to