Page MenuHomePhabricator

Switch x86_64 asm to use "i" instead of "n" for immediate values.
ClosedPublic

Authored by deadalnix on Jan 17 2020, 12:17.

Details

Summary

The constraint "n" is a more obscure setting which is needed for
special cases that don't apply to our use.

Use of "n" exposed some bugs in pre-release versions of clang.
This fixes #594

This is a backport of secp256k1's PR599

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
secppr599
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 9012
Build 15986: Default Diff Build & Tests
Build 15985: arc lint + arc unit