[Parser] Make parser aware of the unexpected attributes

Author: StevenWong12

include/swift/AST/DiagnosticsParse.def

@@ -461,7 +461,8 @@ ERROR(operator_decl_no_fixity,none,
       "operator must be declared as 'prefix', 'postfix', or 'infix'", ())
 ERROR(operator_decl_expected_precedencegroup, none,
       "expected precedence group name after ':' in operator declaration", ())
-
+ERROR(operator_decl_should_not_contain_other_attributes, none, 
+      "unexpected attribute %0 in operator declaration", (StringRef))
 WARNING(operator_decl_remove_designated_types,none,
         "designated types are no longer used by the compiler; please remove "
         "the designated type list from this operator declaration", ())

lib/Parse/ParseDecl.cpp

@@ -4606,6 +4606,18 @@ static void diagnoseOperatorFixityAttributes(Parser &P,
     if (fixityAttrs.empty()) {
       P.diagnose(OD->getOperatorLoc(), diag::operator_decl_no_fixity);
     }
+
+    for (auto it = Attrs.begin(); it != Attrs.end(); ++it) {
+      if (isFixityAttr(*it)) {
+        continue;
+      }
+      auto *attr = *it;
+      P.diagnose(attr->getLocation(),
+                 diag::operator_decl_should_not_contain_other_attributes,
+                 attr->getAttrName())
+          .fixItRemove(attr->getRange());
+      attr->setInvalid();
+    }
   }
   // Infix operator is only allowed on operator declarations, not on func.
   else if (isa<FuncDecl>(D)) {

test/Parse/operator_decl.swift

@@ -117,3 +117,5 @@ infix operator *<*< : F, Proto
 
 // expected-error@+2 {{expected precedence group name after ':' in operator declaration}}
 postfix operator ++: // expected-error {{only infix operators may declare a precedence}} {{20-21=}}
+
+dynamic prefix operator +!+  // expected-error{{unexpected attribute 'dynamic' in operator declaration}} {{1-9=}}

test/attr/attr_dynamic.swift

@@ -7,8 +7,6 @@ struct NotObjCAble {
 
 @objc class ObjCClass {}
 
-dynamic prefix operator +!+  // expected-error{{'dynamic' modifier cannot be applied to this declaration}} {{1-9=}}
-
 class Foo {
   @objc dynamic init() {}
   @objc dynamic init(x: NotObjCAble) {} // expected-error{{method cannot be marked @objc because the type of the parameter cannot be represented in Objective-C}} expected-note{{Swift structs cannot be represented in Objective-C}}