[Lldb-commits] [PATCH] D35113: Clean up lldb-types.h

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Jul 10 04:43:43 PDT 2017

labath added a comment.

Comment at: include/lldb/Host/MainLoop.h:16
 #include "llvm/ADT/DenseMap.h"
+#include <signal.h>
Eugene.Zelenko wrote:
> I think will be good idea to include csignal instead. Same in other files. See http://clang.llvm.org/extra/clang-tidy/checks/modernize-deprecated-headers.html.
Good idea. I wasn't aware <csignal> existed.


More information about the lldb-commits mailing list