[PATCH] D146266: gn build: Fix Android build.
Peter Collingbourne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 20 13:51:36 PDT 2023
pcc added inline comments.
================
Comment at: llvm/utils/gn/secondary/llvm/lib/Support/BUILD.gn:7
"//llvm/lib/Demangle",
+ "//llvm/utils/gn/build/libs/implicit",
"//llvm/utils/gn/build/libs/pthread",
----------------
pcc wrote:
> aeubanks wrote:
> > why put this in Support rather than something like `executable` (which you had mentioned trying)?
> Unfortunately that wouldn't work, because the deps attribute is not allowed on config.
>
> https://gn.googlesource.com/gn/+/main/docs/reference.md#func_config
I just remembered how this is done in Chromium, they define template wrappers around the target types that invoke the linker: https://source.chromium.org/chromium/chromium/src/+/main:build/config/BUILDCONFIG.gn;l=512
That's a much better approach, so I'll switch to that.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146266/new/
https://reviews.llvm.org/D146266
More information about the llvm-commits
mailing list