[Lldb-commits] [PATCH] D107338: [lldb] Get rid of HAVE_SYS_TYPES_H

Raphael Isemann via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Aug 3 04:24:31 PDT 2021

teemperor accepted this revision.
teemperor added a comment.
This revision is now accepted and ready to land.

Thanks! This LGTM itself, but I would prefer if we could remove that define in this commit and remove the redundant Config.h includes in another (so, the first version of this patch LGTM).

Removing Config.h is always a bit icky as it will just silently disable all the `#if feature` code in case it's actually needed. IIUC you checked all the including files, but there are also downstream folks who will have probably an easier time bisecting potential fallout from this when it's a dedicated commit with a clear '[lldb] Remove redundant Config.h includes' as a title.

(Also unrelated: I think quite a few of the files here don't actually need anything from sys/types.h? We could probably just remove a bunch of this stuff all-together as a followup)



More information about the lldb-commits mailing list