[PATCH] D30178: Do not leak OpenedHandles

Frederich Munch via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 20 18:50:20 PST 2017


marsupial added a comment.

I get what ManagedStatic is supposed to be doing, but versus using a std::unique_ptr the cost seems more with it (both in terms of size and startup time).
But I guess that conversation would add a bit of noise and doesn't have much to do with this patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D30178





More information about the llvm-commits mailing list