[Lldb-commits] [PATCH] Move full initialization to lldb/API

[Zachary Turner]

I checked and I saw the warning with a different class but not with ManagedStatic<>.  Since ManagedStatic<> is POD it only relies on the linker for initialization, so it looks like it doesn't have this issue.  I think we don't have to worry about that changing in the future either.  A comment in ManagedStaticBase.h says this:

  // This should only be used as a static variable, which guarantees that this
  // will be zero initialized.

So relying on the linker for initialization is part of the guarantee of the class, so it should never get a global constructor in the future either.  Seems like we're ok then?


http://reviews.llvm.org/D8462

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/