Skip to content

Eliminate gold linker usage#35112

Merged
mlukasze merged 1 commit intoopenvinotoolkit:masterfrom
olpipi:remove_gold
Apr 3, 2026
Merged

Eliminate gold linker usage#35112
mlukasze merged 1 commit intoopenvinotoolkit:masterfrom
olpipi:remove_gold

Conversation

@olpipi
Copy link
Copy Markdown
Contributor

@olpipi olpipi commented Apr 1, 2026

Details:

  • Eliminate gold linker usage due to its obsolescence and conflict with new cmake versions
  • ...

Tickets:

AI Assistance:

  • *AI assistance used: no *

@olpipi olpipi requested review from a team as code owners April 1, 2026 14:58
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: build OpenVINO cmake script / infra category: ONNX FE OpenVINO ONNX FrontEnd labels Apr 1, 2026
@mlukasze mlukasze requested a review from Copilot April 1, 2026 19:10
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the project’s optional usage of the GNU gold linker, aligning the build system with newer CMake behavior and avoiding reliance on an obsolete linker.

Changes:

  • Removed ov_try_use_gold_linker() invocations from ONNX frontend and core test CMake targets.
  • Deleted the ov_try_use_gold_linker() CMake helper function that injected -fuse-ld=gold.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/frontends/onnx/tests/CMakeLists.txt Stops attempting to enable gold linker for ONNX frontend test targets.
src/core/tests/CMakeLists.txt Stops attempting to enable gold linker for core unit test targets.
cmake/developer_package/compile_flags/functions.cmake Removes the gold-linker selection helper that added -fuse-ld=gold.

@praasz praasz added this to the 2026.2 milestone Apr 2, 2026
@mlukasze mlukasze enabled auto-merge April 2, 2026 08:21
@mlukasze mlukasze added this pull request to the merge queue Apr 3, 2026
Merged via the queue into openvinotoolkit:master with commit 1b0ed65 Apr 3, 2026
213 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: build OpenVINO cmake script / infra category: Core OpenVINO Core (aka ngraph) category: ONNX FE OpenVINO ONNX FrontEnd

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants