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

[Zachary Turner]

In http://reviews.llvm.org/D8462#143869, @zturner wrote:

> 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?


Just to be certain I saw the same warning you're talking about, I believe it said something like "Declaration requires a global destructor"


http://reviews.llvm.org/D8462

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