Skip to content

CI: link icpx release builds with -ipo - #70

Merged
Lastique merged 1 commit into
boostorg:developfrom
reach2sayan:fix/ci-icpx-ipo
Oct 29, 2026
Merged

Lastique merged 1 commit into
boostorg:developfrom
reach2sayan:fix/ci-icpx-ipo

Conversation

@reach2sayan

Copy link
Copy Markdown
Contributor

Since bfgroup/b2#616, b2 compiles intel-linux release builds with -O3 -ipo, which emits LLVM bitcode, but does not pass -ipo at link time. Every icpx release link fails with file format not recognized. This breaks the icpx-2025 job on develop, master and every open PR.

Temporary workaround: add linkflags=-ipo to the icpx matrix entry. Debug links are unaffected by the extra flag. It should be reverted once b2 is fixed.

@codecov

codecov Bot commented Oct 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.60%. Comparing base (1e1f7d6) to head (b16fd45).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop      #70   +/-   ##
========================================
  Coverage    82.60%   82.60%           
========================================
  Files           23       23           
  Lines         1006     1006           
  Branches       410      410           
========================================
  Hits           831      831           
  Misses         167      167           
  Partials         8        8           

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1e1f7d6...b16fd45. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@reach2sayan

Copy link
Copy Markdown
Contributor Author

@Lastique The stopgap

@Lastique
Lastique merged commit 927321f into boostorg:develop Oct 29, 2026
63 checks passed
@Lastique

Copy link
Copy Markdown
Member

Thanks.

Did you report it to b2? It would be nice if this was fixed before 1.93.0 release.

@reach2sayan

Copy link
Copy Markdown
Contributor Author

Thanks.

Did you report it to b2? It would be nice if this was fixed before 1.93.0 release.

@Lastique Yes.

@reach2sayan
reach2sayan deleted the fix/ci-icpx-ipo branch October 30, 2026 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants