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
Switch x86_64 asm to use "i" instead of "n" for immediate values. #599
pull gmaxwell wants to merge 1 commits into bitcoin-core:master from gmaxwell:2019-03-asm-interm changing 1 files +3 −3-
gmaxwell commented at 11:03 PM on March 10, 2019: contributor
-
d58bc93f2c
Switch x86_64 asm to use "i" instead of "n" for immediate values.
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
-
gmaxwell commented at 11:05 PM on March 10, 2019: contributor
I know the issue has been closed, but since this was reported multiple times independently (in bitcoin repo too) we're likely to see more of it.
-
sipa commented at 11:14 PM on March 10, 2019: contributor
utACK d58bc93f2cdcb0e3ffccdc091c3f25ac095c88b4
- fanquake cross-referenced this on Mar 11, 2019 from issue Error compiling on debian sid with clang-8 by MarcoFalke
-
real-or-random commented at 11:18 AM on March 11, 2019: contributor
utACK d58bc93f2cdcb0e3ffccdc091c3f25ac095c88b4
- gmaxwell merged this on Mar 11, 2019
- gmaxwell closed this on Mar 11, 2019
- gmaxwell referenced this in commit ee99f12f3d on Mar 11, 2019
Contributors