Fix Linux native flags for non-x86 CI builds by yawkat · Pull Request #42 · yawkat/lz4-java

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@yawkat

Copy link

Owner

@yawkat yawkat commented

Mar 2, 2026

Summary

  • split Linux CI native flags into architecture-neutral base flags and x86-only extras
  • apply x86-only flags only for amd64/i386, avoiding unsupported options on aarch64/ppc64le/s390x
  • pass -m32 in i386 linker flags to keep 32-bit object/link architecture consistent

Verification

  • ./mvnw -q verify passed locally
  • i386 cross build was exercised locally but failed due missing host 32-bit libc headers (bits/libc-header-start.h), so full i386 verification remains in GitHub Actions

@yawkat yawkat force-pushed the perf/native-build-tuning branch from 701ad7b to 395b8d9 Compare

March 2, 2026 17:00

@yawkat yawkat merged commit de5e4db into main

Mar 2, 2026

11 checks passed

@yawkat yawkat deleted the perf/native-build-tuning branch

March 2, 2026 17:21

@github-actions

Copy link

github-actions bot commented

Mar 2, 2026

mergify bot added a commit to ArcadeData/arcadedb that referenced this pull request

Mar 9, 2026
Bumps [at.yawk.lz4:lz4-java](https://github.com/yawkat/lz4-java) from 1.10.3 to 1.10.4.
Release notes

*Sourced from [at.yawk.lz4:lz4-java's releases](https://github.com/yawkat/lz4-java/releases).*

> lz4-java v1.10.4
> ----------------
>
> What's Changed
> --------------
>
> * Tune Linux native build flags and xxhash compilation by [`@​yawkat`](https://github.com/yawkat) in [yawkat/lz4-java#41](https://redirect.github.com/yawkat/lz4-java/pull/41)
> * Fix Linux native flags for non-x86 CI builds by [`@​yawkat`](https://github.com/yawkat) in [yawkat/lz4-java#42](https://redirect.github.com/yawkat/lz4-java/pull/42)
>
> These changes attempt to fix the native performance regression in 1.9+. They should have no functional or security impact.
>
> **Full Changelog**: <yawkat/lz4-java@v1.10.3...v1.10.4>


Commits

* [`de5e4db`](yawkat/lz4-java@de5e4db) Fix Linux native flags for non-x86 CI builds ([#42](https://redirect.github.com/yawkat/lz4-java/issues/42))
* [`f659131`](yawkat/lz4-java@f659131) Tune Linux native build flags and xxhash compilation ([#41](https://redirect.github.com/yawkat/lz4-java/issues/41))
* See full diff in [compare view](yawkat/lz4-java@v1.10.3...v1.10.4)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=at.yawk.lz4:lz4-java&package-manager=maven&previous-version=1.10.3&new-version=1.10.4)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
Dependabot commands and options
  
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot show  ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@yawkat