diff options
author | Pavel Labath <labath@google.com> | 2018-04-05 15:17:51 +0000 |
---|---|---|
committer | Pavel Labath <labath@google.com> | 2018-04-05 15:17:51 +0000 |
commit | 441460dc9173bc10892ee9f53e5382f5768d4c8f (patch) | |
tree | 43a447a7e529976993d99b17f431d8370ac6c51b | |
parent | 878e96f26637d0ca3194289796304177c93d1da3 (diff) | |
download | bcm5719-llvm-441460dc9173bc10892ee9f53e5382f5768d4c8f.tar.gz bcm5719-llvm-441460dc9173bc10892ee9f53e5382f5768d4c8f.zip |
Remove unused NativeProcessProtocol.h include from Platform.h
After removing it, I got a couple of compile errors because we were
missing some symbols (SIGKILL and such), as their definitions were not
transitively included anymore. I fix this by including csignal from
PosixApi.h, as it's windows version provides a stub definitions of these
symbols. This should make the result of #including PosixApi.h more
consistent across platforms (although in the long run, we should just
get rid of this header).
llvm-svn: 329296
-rw-r--r-- | lldb/include/lldb/Host/PosixApi.h | 1 | ||||
-rw-r--r-- | lldb/include/lldb/Target/Platform.h | 4 |
2 files changed, 1 insertions, 4 deletions
diff --git a/lldb/include/lldb/Host/PosixApi.h b/lldb/include/lldb/Host/PosixApi.h index d5c48dd6d17..ceb41ee040f 100644 --- a/lldb/include/lldb/Host/PosixApi.h +++ b/lldb/include/lldb/Host/PosixApi.h @@ -18,6 +18,7 @@ #include "lldb/Host/windows/PosixApi.h" #else #include <unistd.h> +#include <csignal> #endif #endif diff --git a/lldb/include/lldb/Target/Platform.h b/lldb/include/lldb/Target/Platform.h index 14a27928c01..fcb9712e89c 100644 --- a/lldb/include/lldb/Target/Platform.h +++ b/lldb/include/lldb/Target/Platform.h @@ -30,10 +30,6 @@ #include "lldb/lldb-private-forward.h" #include "lldb/lldb-public.h" -// TODO pull NativeDelegate class out of NativeProcessProtocol so we -// can just forward ref the NativeDelegate rather than include it here. -#include "lldb/Host/common/NativeProcessProtocol.h" - namespace lldb_private { class ModuleCache; |