[PATCH] D27486: Correct class-template deprecation behavior

David Majnemer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 7 15:19:30 PST 2016


majnemer added inline comments.


================
Comment at: lib/Sema/SemaTemplate.cpp:2355
                                                      Converted, nullptr);
+      if (auto *attr = ClassTemplate->getTemplatedDecl()
+                           ->getAttr<clang::DeprecatedAttr>())
----------------
Please capitalize `Attr`.


================
Comment at: lib/Sema/SemaTemplate.cpp:2356
+      if (auto *attr = ClassTemplate->getTemplatedDecl()
+                           ->getAttr<clang::DeprecatedAttr>())
+        Decl->addAttr(attr->clone(Context));
----------------
I think you can remove the `clang::`


https://reviews.llvm.org/D27486





More information about the cfe-commits mailing list