Build the Clang plugin on Windows.

tools/clang/plugins/ChromeClassTester.cpp

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // A general interface for filtering and only acting on classes in Chromium C++
 // code.
 
 #include "ChromeClassTester.h"
 
+#include <algorithm>
+
 #include "clang/AST/AST.h"
 #include "clang/Basic/FileManager.h"
 #include "clang/Basic/SourceManager.h"
 
 #ifdef LLVM_ON_UNIX
 #include <sys/param.h>
 #endif
 
 using namespace clang;
 
@@ skipping 102 matching lines @@
 
   if (ends_with(filename, ".cc") || ends_with(filename, ".cpp") ||
       ends_with(filename, ".mm")) {
     return true;
   }
 
   return false;
 }
 
 void ChromeClassTester::BuildBannedLists() {
-  banned_namespaces_.push_back("std");
-  banned_namespaces_.push_back("__gnu_cxx");
+  banned_namespaces_.emplace_back("std");

[Nico, 2015/04/12 04:19:14]

nit: I'd keep push_back since everybody knows what that does, this isn't
perf-critical, and it keeps the diff shorter

[dcheng, 2015/04/12 06:29:30]

But but but C++11!

I'll probably revisit this in another CL in the future =)

+  banned_namespaces_.emplace_back("__gnu_cxx");
 
-  banned_namespaces_.push_back("blink");
-  banned_namespaces_.push_back("WTF");
+  banned_namespaces_.emplace_back("blink");
+  banned_namespaces_.emplace_back("WTF");
 
-  banned_directories_.push_back("/third_party/");
-  banned_directories_.push_back("/native_client/");
-  banned_directories_.push_back("/breakpad/");
-  banned_directories_.push_back("/courgette/");
-  banned_directories_.push_back("/pdf/");
-  banned_directories_.push_back("/ppapi/");
-  banned_directories_.push_back("/usr/include/");
-  banned_directories_.push_back("/usr/lib/");
-  banned_directories_.push_back("/usr/local/include/");
-  banned_directories_.push_back("/usr/local/lib/");
-  banned_directories_.push_back("/testing/");
-  banned_directories_.push_back("/v8/");
-  banned_directories_.push_back("/dart/");
-  banned_directories_.push_back("/sdch/");
-  banned_directories_.push_back("/icu4c/");
-  banned_directories_.push_back("/frameworks/");
+  // System headers:
+#if defined(LLVM_ON_UNIX)
+  banned_directories_.emplace_back("/usr/include/");
+  banned_directories_.emplace_back("/usr/lib/");
+  banned_directories_.emplace_back("/usr/local/include/");
+  banned_directories_.emplace_back("/usr/local/lib/");
+#endif
+
+  banned_directories_.emplace_back("/third_party/");
+  banned_directories_.emplace_back("/native_client/");
+  banned_directories_.emplace_back("/breakpad/");
+  banned_directories_.emplace_back("/courgette/");
+  banned_directories_.emplace_back("/pdf/");
+  banned_directories_.emplace_back("/ppapi/");
+  banned_directories_.emplace_back("/testing/");
+  banned_directories_.emplace_back("/v8/");
+  banned_directories_.emplace_back("/dart/");
+  banned_directories_.emplace_back("/sdch/");
+  banned_directories_.emplace_back("/icu4c/");
+  banned_directories_.emplace_back("/frameworks/");
 
   // Don't check autogenerated headers.
   // Make puts them below $(builddir_name)/.../gen and geni.
   // Ninja puts them below OUTPUT_DIR/.../gen
   // Xcode has a fixed output directory for everything.
-  banned_directories_.push_back("/gen/");
-  banned_directories_.push_back("/geni/");
-  banned_directories_.push_back("/xcodebuild/");
+  banned_directories_.emplace_back("/gen/");
+  banned_directories_.emplace_back("/geni/");
+  banned_directories_.emplace_back("/xcodebuild/");
 
   // You are standing in a mazy of twisty dependencies, all resolved by
   // putting everything in the header.
-  banned_directories_.push_back("/automation/");
+  banned_directories_.emplace_back("/automation/");
 
   // Don't check system headers.
-  banned_directories_.push_back("/Developer/");
+  banned_directories_.emplace_back("/Developer/");
 
   // Used in really low level threading code that probably shouldn't be out of
   // lined.
-  ignored_record_names_.insert("ThreadLocalBoolean");
+  ignored_record_names_.emplace("ThreadLocalBoolean");
 
   // A complicated pickle derived struct that is all packed integers.
-  ignored_record_names_.insert("Header");
+  ignored_record_names_.emplace("Header");
 
   // Part of the GPU system that uses multiple included header
   // weirdness. Never getting this right.
-  ignored_record_names_.insert("Validators");
+  ignored_record_names_.emplace("Validators");
 
   // Has a UNIT_TEST only constructor. Isn't *terribly* complex...
-  ignored_record_names_.insert("AutocompleteController");
-  ignored_record_names_.insert("HistoryURLProvider");
+  ignored_record_names_.emplace("AutocompleteController");
+  ignored_record_names_.emplace("HistoryURLProvider");
 
   // Because of chrome frame
-  ignored_record_names_.insert("ReliabilityTestSuite");
+  ignored_record_names_.emplace("ReliabilityTestSuite");
 
   // Used over in the net unittests. A large enough bundle of integers with 1
   // non-pod class member. Probably harmless.
-  ignored_record_names_.insert("MockTransaction");
+  ignored_record_names_.emplace("MockTransaction");
 
   // Enum type with _LAST members where _LAST doesn't mean last enum value.
-  ignored_record_names_.insert("ServerFieldType");
+  ignored_record_names_.emplace("ServerFieldType");
 
   // Used heavily in ui_base_unittests and once in views_unittests. Fixing this
   // isn't worth the overhead of an additional library.
-  ignored_record_names_.insert("TestAnimationDelegate");
+  ignored_record_names_.emplace("TestAnimationDelegate");
 
   // Part of our public interface that nacl and friends use. (Arguably, this
   // should mean that this is a higher priority but fixing this looks hard.)
-  ignored_record_names_.insert("PluginVersionInfo");
+  ignored_record_names_.emplace("PluginVersionInfo");
 
   // Measured performance improvement on cc_perftests. See
   // https://codereview.chromium.org/11299290/
-  ignored_record_names_.insert("QuadF");
+  ignored_record_names_.emplace("QuadF");
 
   // Enum type with _LAST members where _LAST doesn't mean last enum value.
-  ignored_record_names_.insert("ViewID");
+  ignored_record_names_.emplace("ViewID");
 }
 
 std::string ChromeClassTester::GetNamespaceImpl(const DeclContext* context,
                                                 const std::string& candidate) {
   switch (context->getDeclKind()) {
     case Decl::TranslationUnit: {
       return candidate;
     }
     case Decl::Namespace: {
       const NamespaceDecl* decl = dyn_cast<NamespaceDecl>(context);
       std::string name_str;
       llvm::raw_string_ostream OS(name_str);
       if (decl->isAnonymousNamespace())
         OS << "<anonymous namespace>";
       else
         OS << *decl;
       return GetNamespaceImpl(context->getParent(),
                               OS.str());
     }
     default: {
       return GetNamespaceImpl(context->getParent(), candidate);
     }
   }
 }
 
 bool ChromeClassTester::InBannedDirectory(SourceLocation loc) {
+  if (instance().getSourceManager().isInSystemHeader(loc))
+    return true;
+
   std::string filename;
   if (!GetFilename(loc, &filename)) {
     // If the filename cannot be determined, simply treat this as a banned
     // location, instead of going through the full lookup process.
     return true;
   }
 
   // We need to special case scratch space; which is where clang does its
   // macro expansion. We explicitly want to allow people to do otherwise bad
   // things through macros that were defined due to third party libraries.
   if (filename == "<scratch space>")
     return true;
 
   // Don't complain about autogenerated protobuf files.
   if (ends_with(filename, ".pb.h")) {
     return true;
   }
 
-#ifdef LLVM_ON_UNIX
+#if defined(LLVM_ON_UNIX)
   // We need to munge the paths so that they are relative to the repository
   // srcroot. We first resolve the symlinktastic relative path and then
   // remove our known srcroot from it if needed.
   char resolvedPath[MAXPATHLEN];
   if (realpath(filename.c_str(), resolvedPath)) {
     filename = resolvedPath;
   }
 #endif
 
+#if defined(LLVM_ON_WIN32)
+  std::replace(filename.begin(), filename.end(), '\\', '/');
+#endif
+
   for (const std::string& banned_dir : banned_directories_) {
     // If any of the banned directories occur as a component in filename,
     // this file is rejected.
     assert(banned_dir.front() == '/' && "Banned dir must start with '/'");
     assert(banned_dir.back() == '/' && "Banned dir must end with '/'");
 
     if (filename.find(banned_dir) != std::string::npos)
       return true;
   }
 
@@ skipping 11 matching lines @@
   PresumedLoc ploc = source_manager.getPresumedLoc(spelling_location);
   if (ploc.isInvalid()) {
     // If we're in an invalid location, we're looking at things that aren't
     // actually stated in the source.
     return false;
   }
 
   *filename = ploc.getFilename();
   return true;
 }