labath accepted this revision.
labath added a comment.

So, linux actually manages to invoke the atexit handlers (through some deep 
magic, no doubt) upon shared library unload, so we are ok here (plus linux 
likes to nuke /tmp after every reboot). LGTM, with a small RAII request.


================
Comment at: source/Host/common/HostInfoBase.cpp:86
@@ +85,3 @@
+{
+    CleanupProcessSpecificLLDBTempDir();
+    delete g_fields;
----------------
I think we should move this into the destructor of `g_fields`. Then you can 
simply do `delete g_fields` (or even `g_fields.reset()`) here.


http://reviews.llvm.org/D17420



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to