[PATCH] Parse: Don't crash on trailing whitespace before EOF

David Majnemer david.majnemer at gmail.com
Wed Jul 23 13:15:01 PDT 2014


  - Forgot to add the test

http://reviews.llvm.org/D4646

Files:
  lib/Parse/ParseDecl.cpp
  test/Parser/eof2.cpp

Index: lib/Parse/ParseDecl.cpp
===================================================================
--- lib/Parse/ParseDecl.cpp
+++ lib/Parse/ParseDecl.cpp
@@ -2450,7 +2450,7 @@
                                         LateParsedAttrList *LateAttrs) {
   if (DS.getSourceRange().isInvalid()) {
     DS.SetRangeStart(Tok.getLocation());
-    DS.SetRangeEnd(Tok.getLocation());
+    DS.SetRangeEnd(SourceLocation());
   }
 
   bool EnteringContext = (DSContext == DSC_class || DSContext == DSC_top_level);
Index: test/Parser/eof2.cpp
===================================================================
--- /dev/null
+++ test/Parser/eof2.cpp
@@ -0,0 +1,15 @@
+// RUN: not %clang_cc1 %s -fsyntax-only 2>&1 | FileCheck %s
+
+// CHECK: error: expected expression
+// CHECK: error: expected member name or ';' after declaration specifiers
+// CHECK: error: expected '}'
+// CHECK: note: to match this '{'
+// CHECK: error: expected ';' after class
+// CHECK: 4 errors generated.
+
+// Do not add anything to the end of this file.  This requires the whitespace
+// plus EOF after the '<' token.
+
+template <typename T>
+class a {
+  a<
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4646.11819.patch
Type: text/x-patch
Size: 1129 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140723/5a44d29f/attachment.bin>


More information about the cfe-commits mailing list