oss-fuzz icon indicating copy to clipboard operation
oss-fuzz copied to clipboard

infra: use LLD for all configurations

Open kasper93 opened this issue 9 months ago • 5 comments

This makes it consistent to use LLD across all configurations. It also moves away from using gold linker, which has recently been deprecated.

On top of that LLD is faster and has better compatibility with LLVM toolchain that is already used for building.

This commit also fixes warnings that can sometimes occur with BFD linker that doesn't fully support DWARF 5, at least the version on the builder: DWARF: invalid or unhandled FORM value: 0x22

Some projects already replaced linker in thier build scripts for this reason.

kasper93 avatar Apr 09 '25 13:04 kasper93

kasper93 has previously contributed to projects/ffmpeg. The previous PR was #13175
kasper93 is either the primary contact or is in the CCs list of projects/mpv.
kasper93 has previously contributed to projects/mpv. The previous PR was #13158
kasper93 is a new contributor to projects/dbus-broker. The PR must be approved by known contributors before it can be merged. The past contributors are: DonggeLiu, evverx
kasper93 is a new contributor to projects/ntp. The PR must be approved by known contributors before it can be merged. The past contributors are: joycebrum, devtty1er, Dor1s, catenacyber (unverified)
kasper93 is a new contributor to projects/halide. The PR must be approved by known contributors before it can be merged. The past contributors are: nathaniel-brough
kasper93 is a new contributor to projects/tdengine. The PR must be approved by known contributors before it can be merged. The past contributors are: DonggeLiu, sangshuduo
kasper93 is a new contributor to projects/ecc-diff-fuzzer. The PR must be approved by known contributors before it can be merged. The past contributors are: catenacyber, gilles-peskine-arm, guidovranken, daverodgman, devtty1er, inferno-chromium, Dor1s
kasper93 is a new contributor to projects/libiec61850. The PR must be approved by known contributors before it can be merged. The past contributors are: DonggeLiu
kasper93 is a new contributor to projects/llhttp. The PR must be approved by known contributors before it can be merged. The past contributors are: DaveLak, DonggeLiu
kasper93 is a new contributor to projects/thrift. The PR must be approved by known contributors before it can be merged. The past contributors are: fishy, catenacyber
kasper93 is a new contributor to projects/connectedhomeip. The PR must be approved by known contributors before it can be merged. The past contributors are: Alami-Amine, andy31415, abcSup

github-actions[bot] avatar Apr 09 '25 13:04 github-actions[bot]

/gcbrun trial_build.py all --sanitizer address --fuzzing-engine libfuzzer

jonathanmetzman avatar Apr 09 '25 16:04 jonathanmetzman

/gcbrun trial_build.py all --sanitizer address --fuzzing-engine libfuzzer

Added missing -Wno-unused-command-line-argument, should fix most failures.

kasper93 avatar Apr 11 '25 13:04 kasper93

Are we using lld by default? I'm not sure we are, but I haven't looked.

jonathanmetzman avatar Apr 11 '25 14:04 jonathanmetzman

Are we using lld by default? I'm not sure we are, but I haven't looked.

The proposal is to use LLD by default. I've outlined the main reason in commit message. Currently BFD linker is used, with some jobs using GOLD. Idea is to unify and modernize to use LLD.

kasper93 avatar Apr 11 '25 14:04 kasper93