[PATCH] D122862: gn build: Fix support for building the builtins for baremetal.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 15 15:52:01 PDT 2022


pcc added a comment.

I don't think I would expect the stage 2 runtimes to be built with a sanitizer even if `use_asan` is set. The sanitizer runtimes themselves don't support being built with sanitizers, and they might be compiled to depend on the builtins, which would create a circular dependency. So I think `use_asan` and such should only affect the stage 1 compiler. I think I would favor adding `use_asan = false` to the `stage2_unix_toolchain` template.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122862/new/

https://reviews.llvm.org/D122862



More information about the llvm-commits mailing list