Treat std::atomic_int as a trivial class member when checking for non-trivial classes.

tools/clang/plugins/FindBadConstructsConsumer.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.
 
 #include "FindBadConstructsConsumer.h"
 
 #include "clang/Frontend/CompilerInstance.h"
 #include "clang/AST/Attr.h"
 #include "clang/Lex/Lexer.h"
 #include "clang/Sema/Sema.h"
@@ skipping 635 matching lines @@
         (*non_trivial_member)++;
       break;
     }
     case Type::TemplateSpecialization: {
       TemplateName name =
           dyn_cast<TemplateSpecializationType>(type)->getTemplateName();
       bool whitelisted_template = false;
 
       // HACK: I'm at a loss about how to get the syntax checker to get
       // whether a template is externed or not. For the first pass here,
-      // just do retarded string comparisons.
+      // just do simple string comparisons.
+      //
+      // The check for "atomic_int" is a quick hack; The real fix will be to
+      // check the class after template instantiation. TODO(dcheng): Do the real

[dcheng, 2016/11/08 21:06:20]

=P

+      // fix. https://bugs.chromium.org/p/chromium/issues/detail?id=663463
       if (TemplateDecl* decl = name.getAsTemplateDecl()) {
         std::string base_name = decl->getNameAsString();
-        if (base_name == "basic_string")
+        if (base_name == "basic_string" || base_name == "atomic_int")
           whitelisted_template = true;
       }
 
       if (whitelisted_template)
         (*non_trivial_member)++;
       else
         (*templated_non_trivial_member)++;
       break;
     }
     case Type::Elaborated: {
@@ skipping 344 matching lines @@
       }
     } else if (non_reference_type->isPointerType()) {
       non_reference_type = non_reference_type->getPointeeType();
       continue;
     }
     break;
   }
 }
 
 }  // namespace chrome_checker