[libcxx-commits] [PATCH] D132807: [libcxx] CI: set symbolizer for bootstrapping build

[Matheus Izvekov via Phabricator via libcxx-commits]

mizvekov added inline comments.


================
Comment at: libcxx/utils/ci/run-buildbot:474
 
+    export LLVM_SYMBOLIZER_PATH="/usr/bin/llvm-symbolizer-14"
+
----------------
ldionne wrote:
> What tool picks up that environment variable? With a quick search, it looks like `clang` itself looks for it and symbolizes the crash trace when it can find it.
> 
> We are trying to keep `run-buildbot` agnostic of the versions of tools available on the CI machine, so this may belong more in` buildkite-pipeline.yml`, or perhaps we should also simply make `llvm-symbolizer` available in the `$PATH` on our CI nodes by changing `libcxx/utils/ci/Dockerfile`.
Yeah that is the case, we need that so that we can have readable crash traces in the CI output.

Setting that variable for all configurations, or just making the unversioned tool available, would also help with crash traces in configurations other than the bootstrapping-build, so I will follow one of those approaches.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132807