Closed Bug 1758781 Opened 3 years ago Closed 3 years ago

Build llvm-symbolizer separately

Categories

(Firefox Build System :: Toolchains, task)

task

Tracking

(firefox100 fixed)

RESOLVED FIXED
100 Branch
Tracking Status
firefox100 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

(Blocks 1 open bug)

Details

Attachments

(4 files)

The llvm-symbolizer tasks currently extract a llvm-symbolizer from clang tasks. Changes in clang 14 make the hack that we have in place to keep llvm-symbolizer statically linked to libllvm while clang uses a dynamic libllvm not work anymore, so it's time to bite the bullet and build llvm-symbolizer separately.

The toolchain file is a helper for cross-compilation that even deals
with case-sensivity in MSVC, which simplifies our setup.

The llvm-symbolizer tasks currently extract a llvm-symbolizer from clang
tasks. Changes in clang 14 make the hack that we have in place to keep
llvm-symbolizer statically linked to libllvm while clang uses a dynamic
libllvm not work anymore, so it's time to bite the bullet and build
llvm-symbolizer separately.

We share most of the build setup with the compiler-rt build.

Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/78117d649b5d Use the WinMsvc.cmake toolchain file to build compiler-rt. r=firefox-build-system-reviewers,mhentges https://hg.mozilla.org/integration/autoland/rev/68b936acd34c Build llvm-symbolizer separately. r=firefox-build-system-reviewers,mhentges https://hg.mozilla.org/integration/autoland/rev/4365905c4c1a Remove now unnecessary patch to link llvm-symbolizer statically. r=firefox-build-system-reviewers,mhentges

This seems to be causing build problems for me on Fedora. I'll attach error output.

Attached file Error log from mach
Regressions: 1759213
No longer regressions: 1759213
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: