Skip to content

Conversation

@maribu
Copy link
Member

@maribu maribu commented Nov 29, 2025

Contribution description

  • add a compilation fix to micro-ecc
  • add a work around for a (presumably) compiler bug in GCC 15.2.0 to vli_mmod_fast_secp256r1()

Testing procedure

make BOARD=arduino-mega2560 -C tests/pkg/micro-ecc flash test

will fail to build without the first patch (0002-...patch) added, and will fail in the test without the second patch (0003-...patch) added when compiled with GCC 15.2.0.

Issues/PRs references

@github-actions github-actions bot added the Area: pkg Area: External package ports label Nov 29, 2025
@maribu maribu added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Nov 29, 2025
@riot-ci
Copy link

riot-ci commented Nov 29, 2025

Murdock results

✔️ PASSED

5b562e6 pkg/micro-ecc: fix on GCC 15.2.0 for AVR

Success Failures Total Runtime
10950 0 10950 13m:19s

Artifacts

@maribu maribu marked this pull request as draft November 30, 2025 07:50
@maribu maribu force-pushed the pkg/micro-ecc/gcc-15-fixes branch from ab4eac9 to 770fa86 Compare November 30, 2025 12:07
@maribu maribu marked this pull request as ready for review November 30, 2025 12:09
- add a work around for a (presumably) compiler bug in GCC 15.2.0
  register allocation code by manually a function argument into the
  Y register.
- add missing `"memory"` clobber to inline asm implementations. With
  GCC 15.2.0 the `vli_mmod_fast_secp256r1()` function would not compute
  correctly without the `"memory"` clobber.
@maribu maribu force-pushed the pkg/micro-ecc/gcc-15-fixes branch from 770fa86 to 5b562e6 Compare November 30, 2025 19:23
@maribu maribu marked this pull request as draft November 30, 2025 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants