george.karpenkov accepted this revision. george.karpenkov added inline comments. This revision is now accepted and ready to land.
================ Comment at: include/clang/StaticAnalyzer/Core/PathSensitive/ProgramStateTrait.h:33 /// introduce a type named \c NameTy. - /// The macro should not be used inside namespaces, or for traits that must - /// be accessible from more than one translation unit. + /// The macro should not be used inside namespaces. #define REGISTER_TRAIT_WITH_PROGRAMSTATE(Name, Type) \ ---------------- What happened to the comment about multiple TUs? One still has to be very careful here, right? Repository: rC Clang https://reviews.llvm.org/D51388 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
