Skip to content

Update openblas-experimental to develop @ 803f36bc - #6611

Merged
MMesch merged 1 commit into
emscripten-forge:mainfrom
jjerphan:openblas-experimental-803f36bc
Sep 8, 2026
Merged

MMesch merged 1 commit into
emscripten-forge:mainfrom
jjerphan:openblas-experimental-803f36bc

Conversation

@jjerphan

@jjerphan jjerphan commented Sep 8, 2026 •

Copy link
Copy Markdown
Contributor

Template B: Checklist for updating a package

  • ⚠️ Bump build number if the version remains unchanged
  • Or reset build number to 0 if updating the package to a newer version

PR Formatting

  • PR title follows format: Add [package-name] or Update [package-name] to [version]
  • PR description includes:
    • Version being added/updated
    • Any special build considerations or patches applied

Package Details

Build Notes

This refreshes the develop snapshot from 632ef874 to 803f36bc (64 commits newer) and rebases the emforge patch series onto it. The rebased branch is v0.3.35.dev0-emforge at https://github.com/jjerphan/OpenBLAS.

The package version string is unchanged, so the build number is bumped to 1 rather than reset.

Patches now upstream (dropped)

Three of the fourteen patches have landed upstream and were removed:

  • 0006-Int-to-void-return-types-of-lapack-functions-used-by — superseded by OpenBLAS #6017
  • 0013-wasm-guard-KERNEL-assignments-with-ifndef — OpenBLAS #5996
  • 0014-wasm-use-relaxed-simd-madd-for-v_muladd — OpenBLAS #6001

Patches reduced

  • 0002-Skip-linktest no longer touches the WASM code path of exports/Makefile, since upstream added an ifneq ($(ARCH),wasm) guard there (#6007). It now only disables linktest for the remaining compiler branches and the test target.
  • 0005-Return-void-not-int shrinks from ~1170 changed lines to ~40: upstream converted the reimplemented LAPACK routines to void, so only xerbla and the OpenBLAS extension declarations that upstream left returning int remain.
  • 0004-Add-dummy-length-arguments-for-char-arguments was rebased onto the new signatures, so the Fortran hidden-length arguments are now added to void OPENBLAS_API(...) declarations instead of the old int BLASFUNC(...) ones.

The remaining 11 patches were renumbered, and the patch-range comments in recipe.yaml and build.sh were updated accordingly.

Local verification

Built locally with pixi run -e rattler-build-env build-emscripten-wasm32-pkg recipes/recipes_emscripten/openblas-experimental:

  • All 11 patches apply without fuzz.
  • utest and the CBLAS ctest levels 1-3 (real and complex, column- and row-major) pass under Node.
  • The packaged test_openblas.py passes in Chromium, Firefox and WebKit.

Refresh the develop snapshot from 632ef874 to 803f36bc and rebase the
emforge patch series onto it. Three patches are now upstream and were
dropped: the int-to-void lapacke return types (0006), the KERNEL ifndef
guards (0013, OpenBLAS emscripten-forge#5993) and the WASM relaxed-simd madd (0014,
OpenBLAS emscripten-forge#6001). The linktest skip and the void-return patches shrink
accordingly since upstream covers the WASM linktest guard and most of
the reimplemented-LAPACK signatures.

Bump the build number as the package version string is unchanged.

@MMesch MMesch left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Waiting for build to run though.

@MMesch
MMesch merged commit 049366c into emscripten-forge:main Sep 8, 2026
4 checks passed
@jjerphan
jjerphan deleted the openblas-experimental-803f36bc branch September 8, 2026 09:29
IsabelParedes pushed a commit that referenced this pull request Sep 8, 2026
)

Co-authored-by: emscripten-forge-bot <emscripten-forge-bot@users.noreply.github.com>
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