[PATCH] D51388: [analyzer] NFC: Legalize state manager factory injection.

[Artem Dergachev via Phabricator via cfe-commits]

NoQ added inline comments.


================
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) \
----------------
george.karpenkov wrote:
> What happened to the comment about multiple TUs?
> One still has to be very careful here, right?
D52905#1257040.


Repository:
  rL LLVM

https://reviews.llvm.org/D51388