Skip to content

silence werror#2

Open
PhilipDeegan wants to merge 1 commit into
olawlor:masterfrom
PhilipDeegan:werror
Open

silence werror#2
PhilipDeegan wants to merge 1 commit into
olawlor:masterfrom
PhilipDeegan:werror

Conversation

@PhilipDeegan

Copy link
Copy Markdown

I was testing out your allocator for some AVX things

for the case of float/8 (__m256) it has proved necessary, but not for float/4, double/2 or double/4

@olawlor

olawlor commented May 11, 2022

Copy link
Copy Markdown
Owner

Thank you for the fix!

For float4, the default allocator on most current platforms seems to already be aligned by 16 bytes, so the alignocator indeed isn't necessary.

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

Successfully merging this pull request may close these issues.

2 participants