Skip to content

Query binaries of an unbuilt program in the tests again - #577

Merged
maleadt merged 1 commit into
mainfrom
tb/unbuilt-program-binaries
Oct 21, 2026
Merged

maleadt merged 1 commit into
mainfrom
tb/unbuilt-program-binaries

Conversation

@maleadt

@maleadt maleadt commented Oct 20, 2026

Copy link
Copy Markdown
Member

PoCL used to reject querying the binaries of a program created from binaries until it was built, so 0a6012e built that program before the query. pocl_jll 7.2.2+8 carries pocl/pocl#2379, which fixes this (#537), and the spec allows the query, so the test now checks that binaries round-trip without a build.

Passes locally on PoCL (pocl and poclc), Intel's CPU runtime and NVIDIA.

PoCL used to reject querying the binaries of a program before it was
built, so 0a6012e built the program first. pocl_jll 7.2.2+8 fixes that
(#537), and the spec allows the query, so test it again.
@codecov

codecov Bot commented Oct 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.73%. Comparing base (f43c7bf) to head (6119244).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #577   +/-   ##
=======================================
  Coverage   87.73%   87.73%           
=======================================
  Files          18       18           
  Lines        1737     1737           
=======================================
  Hits         1524     1524           
  Misses        213      213           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@maleadt
maleadt merged commit 9c27048 into main Oct 21, 2026
17 checks passed
@maleadt
maleadt deleted the tb/unbuilt-program-binaries branch October 21, 2026 07: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.

1 participant