Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build failure on x86 #4

Closed
a16bitsysop opened this issue Dec 22, 2020 · 7 comments
Closed

build failure on x86 #4

a16bitsysop opened this issue Dec 22, 2020 · 7 comments

Comments

@a16bitsysop
Copy link

Build fails for x86 when building on Alpine Linux, log here:
https://gitlab.alpinelinux.org/a16bitsysop/aports/-/jobs/278030/raw

407/builds/a16bitsysop/aports/community/hyperscan/src/hyperscan-5.3.1/src/util/arch/x86/simd_utils.h: In function 'movq':
408/builds/a16bitsysop/aports/community/hyperscan/src/hyperscan-5.3.1/src/util/arch/x86/simd_utils.h:128:12: error: implicit declaration of function '_mm_cvtsi128_si64'; did you mean '_mm_cvtsi128_si32'? [-Werror=implicit-function-declaration]
409  128 |     return _mm_cvtsi128_si64(in);
410      |            ^~~~~~~~~~~~~~~~~
411      |            _mm_cvtsi128_si32
@markos
Copy link

markos commented Dec 30, 2020

hi, @a16bitsysop, could you please check branch https://github.com/VectorCamp/hyperscan/tree/bugfix/fix-ia32-build? It should fix the x64/IA32 build failures -at least it does on my local Debian i386 environment.

@a16bitsysop
Copy link
Author

Yes that fixes it for building as well
https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/16306/pipelines

@a16bitsysop
Copy link
Author

Trying a non release build now

@a16bitsysop
Copy link
Author

Yes non release builds works as well thanks, are you going to tag a new release with these fixes?

@markos
Copy link

markos commented Dec 31, 2020

@a16bitsysop
Copy link
Author

Thanks

@a16bitsysop
Copy link
Author

The new tag doesn’t include the masked_move.cpp patch, but it’s not needed for release building.

It will be updated pretty soon on alpine Linux to 5.3.2, have enabled hyperscan for suricata on aarch64 and working on enabling it for rspamd on aarch64

markos pushed a commit that referenced this issue Aug 29, 2022
Throw std::bad_alloc instead of returning nullptr from
ue2::AlignedAllocator. Allocators for STL containers are expected never
to return with an invalid pointer, and instead must throw on failure.
Violating this expectation can lead to invalid pointer dereferences.

Co-authored-by: johanngan <johanngan.us@gmail.com>

fixes github issue #317 (PR #320)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants