cespare / xxhash

A Go implementation of the 64-bit xxHash algorithm (XXH64)
MIT License
1.79k stars 123 forks source link

Fix R15 asm error when build in dynamic link mode #55

Closed MXWXZ closed 3 years ago

MXWXZ commented 3 years ago

Fix #54, When imported and build with -buildmode=plugin or anything use dynamic link, go will throw

when dynamic linking, R15 is clobbered by a global variable access and is used here

ref: review src issue

greatroar commented 3 years ago

Looks like SI and DI are free, why not use one of those?

cespare commented 3 years ago

Thanks, but I'd like to fix this a different way. See #57.