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