Jan Wassenberg

Thanks @Mousius for updating! I've commented with a small change required for CI to pass.

Apologies, missed something the CI found. Our gtest wrapper does not support ASSERT_EQ in abort_test. Would you mind changing to HWY_ASSERT_EQ?

Ah, right. The simplest fix would be to just HWY_ASSERT(expected == actual), because we're not very likely to want to print the actual value in case of failure.

Thanks for making the change. Rerunning CI :)

Looks like non-MSVC compilers are good so far, MSVC requires an extra line at the end of the test, added comment :)

No worries :)

Thanks for reporting. Looks like IsInf is not returning true as expected. Would you mind inserting these lines before line 380 of highway_test.cc and sharing the output? ``` Print(d, "input...

I was hoping to run this in Godbolt but looks like they do not support that (qemu) yet: https://gcc.godbolt.org/z/1qMM3bz8r

Yes, I'm currently seeing NEON tests (run_tests.sh with -11 changed to -13) pass. This is GCC 13.2.0. Is it an option to update to that compiler?

Dear @DiamonDinoia, thanks for making the changes. Sorry that our builds are currently red, our internal build has different settings and unfortunately this leads to whack-a-mole, there is a larger...