brucem added a subscriber: brucem.
brucem added a comment.

Unless someone else (@emaste?) objects, I think this looks fine to land and 
iterate on trunk as discussed in the mailing list thread.  Just one minor 
comment based on cleanups that are on-going on trunk now.


================
Comment at: source/Plugins/Platform/NetBSD/PlatformNetBSD.h:50
@@ +49,3 @@
+        virtual
+        ~PlatformNetBSD();
+
----------------
Might as well change this to `~PlatformNetBSD () override = default;` I think.


Repository:
  rL LLVM

http://reviews.llvm.org/D13334



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

Reply via email to