[PATCH] D109185: [gn build] Add build files for LLDB
Raphael Isemann via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 8 01:30:17 PDT 2021
teemperor added a comment.
Thanks! Out of curiosity, is there a public GN bot that is testing this?
> LLDB has many dependency cycles, something GN doesn't allow. For
that reason, I've omitted some dependency edges. Hopefully we can
clean up the cycles one day.
No idea about GN, but I would assume this would break the build in some form? Or does this just mean ninja doesn't know the dependencies and the build process has a race condition?
> LLDB has a public/private header distinction, but mostly ignores it.
> Many libraries include private headers from other modules.
Is this referring to some `private-*.h` `public-*.h` files or what are those public/private headers that are incorrectly included?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109185/new/
https://reviews.llvm.org/D109185
More information about the llvm-commits
mailing list