Merge pull request #4049 from rjmccall/invalid-precedencegroup-range

Fix the source range of a precedencegroup declaration that lacks a }

Author: tkremenek

lib/Parse/ParseDecl.cpp

@@ -5684,6 +5684,11 @@ Parser::parseDeclPrecedenceGroup(ParseDeclOptions flags,
     return result;
   };
   auto createInvalid = [&] {
+    // Use the last consumed token location as the rbrace to satisfy
+    // the AST invariant about a decl's source range including all of
+    // its components.
+    if (!rbraceLoc.isValid()) rbraceLoc = PreviousLoc;
+
     auto result = create();
     result->setInvalid();
     return makeParserErrorResult(result);

test/decl/precedencegroup/parsing.swift

@@ -0,0 +1,7 @@
+// RUN: %target-parse-verify-swift
+
+precedencegroup; // expected-error {{expected identifier}}
+
+precedencegroup A; // expected-error {{expected '{'}}
+
+precedencegroup B {; // expected-error {{expected operator attribute identifier}}