highway
highway copied to clipboard
fix compile error on arm32 (missing float64x1_t type)
this small commit fixes compiling error on some arm32 devices. cuz lack of float64x1_t definition. ( float64x1_t is only available on aarch64 document).
[build] In file included from /workspaces/demo/builds/release/_deps/highway-src/hwy/highway.h:588:0,
[build] from /workspaces/demo/builds/release/_deps/highway-src/hwy/timer-inl.h:26,
[build] from /workspaces/demo/builds/release/_deps/highway-src/hwy/nanobenchmark.cc:28:
[build] /workspaces/demo/builds/release/_deps/highway-src/hwy/ops/arm_neon-inl.h:2494:12: error: ‘float64x1_t’ does not name a type
[build] HWY_INLINE float64x1_t vrecpe_f64(float64x1_t raw) {
[build] ^~~~~~~~~~~
[build] ninja: build stopped: subcommand failed.
so the additional HWY_HAVE_FLOAT64 is added to determine whether the vrecpe_f64 func should be defined.
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.
Why hasn't this pull request been already merged?
Checking, thanks for following up..
Compile logs have already been deleted but it looks like a build timeout is at least contributing to failing internal CI and preventing the merge. Retrying..
Yes, looks like it was a build timeout.