This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] x86: _mm512_set1_p[sd]
- From: Ulrich Drepper <drepper at gmail dot com>
- To: Kirill Yukhin <kirill dot yukhin at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 24 Mar 2014 07:13:59 -0400
- Subject: Re: [PATCH] x86: _mm512_set1_p[sd]
- Authentication-results: sourceware.org; auth=none
- References: <8738idzj52 dot fsf at x240 dot local dot i-did-not-set--mail-host-address--so-tickle-me> <20140324055034 dot GA17283 at msticlxl57 dot ims dot intel dot com>
On Mon, Mar 24, 2014 at 1:50 AM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
> Your patch is correct IMHO, but maybe it worst to add all missing
> `mm512_set1*' stuff?
>
> According to trunk and [1] we're still missing (beside mentioned by you)
> _mm512_set1_epi16 and _mm512_set1_epi8 broadcasts.
Yes, more are missing, but I think those will need new builtins. The
_ps and _pd don't require additional instructions.
_mm512_set1_epi16 might have to map to vpbroadcastw. _mm512_set1_epi8
might have to map to vpbroadcastb. I haven't seen a way to generate
those instructions if needed and so this work was out of scope for now
due to time constraints. I agree, they should be added as quickly as
possible to avoid releasing headers with incomplete APIs.
What is the verdict on checking these changes in? Too late for the
next release?