[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