arphaman accepted this revision.
arphaman added a comment.
This revision is now accepted and ready to land.

Nice, LGTM

Please keep in mind that LLVM follows a weekly ping rate: 
https://llvm.org/docs/DeveloperPolicy.html#code-reviews .



================
Comment at: lib/Sema/SemaCodeComplete.cpp:751
+    auto DeclNameKind = ND->getDeclName().getNameKind();
+    if (DeclNameKind == DeclarationName::NameKind::CXXOperatorName ||
+        DeclNameKind == DeclarationName::NameKind::CXXLiteralOperatorName ||
----------------
Nit: `NameKind::` is redundant.


https://reviews.llvm.org/D38081



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to