Fix a bug in the property-based serialization of

dependent template names.

Apparently we didn't test this in the test suite because we have
a lot of redundant ways of representing this situation that kick
in in the more common situations.  For example, DependentTST stores
a qualifier + identifier pair rather than a TemplateName.

Author: rjmccall

clang/include/clang/AST/PropertiesBase.td

@@ -362,7 +362,7 @@ let Class = PropertyTypeCase<TemplateName, "DependentTemplate"> in {
                       : nullptr) }];
   }
   def : Property<"operatorKind", OverloadedOperatorKind> {
-    let Conditional = [{ identifier }];
+    let Conditional = [{ !identifier }];
     let Read = [{ dtn->getOperator() }];
   }
   def : Creator<[{

clang/test/PCH/cxx-templates.cpp

@@ -165,3 +165,13 @@ namespace DependentMemberExpr {
   static_assert(A<int>().f() == 1); // expected-error {{static_assert failed}}
 #endif
 }
+
+namespace DependentTemplateName {
+  struct HasMember {
+    template <class T> struct Member;
+  };
+
+  void test() {
+    getWithIdentifier<HasMember>();
+  }
+}

clang/test/PCH/cxx-templates.h

@@ -448,3 +448,11 @@ namespace DependentMemberExpr {
     constexpr int f() { return Base::setstate(); }
   };
 }
+
+namespace DependentTemplateName {
+  template <template <class> class Template>
+  struct TakesClassTemplate {};
+
+  template <class T>
+  TakesClassTemplate<T::template Member> getWithIdentifier();
+}