On 11/30/20 9:26 AM, Jakub Jelinek wrote:
> On Mon, Nov 30, 2020 at 09:23:15AM -0700, Jeff Law wrote:
>>
>> On 11/12/20 11:21 PM, Hongyu Wang wrote:
>>> Hi
>>>
>>> Thanks for reminding me about this patch. I didn't remove any existing
>>> intrinsics, just remove redundant builtin functions that
On Mon, Nov 30, 2020 at 09:23:15AM -0700, Jeff Law wrote:
>
>
> On 11/12/20 11:21 PM, Hongyu Wang wrote:
> > Hi
> >
> > Thanks for reminding me about this patch. I didn't remove any existing
> > intrinsics, just remove redundant builtin functions that end-users
> > would not likely to use.
> >
>
On 11/12/20 11:21 PM, Hongyu Wang wrote:
> Hi
>
> Thanks for reminding me about this patch. I didn't remove any existing
> intrinsics, just remove redundant builtin functions that end-users
> would not likely to use.
>
> Also I'm OK to keep current implementation, in case there might be
>
eff
>
>
> Hongyu Wang
>
>
> 0001-Remove-redundant-round-builtins-for-avx512f-scalar-i.patch
>
> From 9cc4928aad5770c53ff580f5c996092cdaf2f9ba Mon Sep 17 00:00:00 2001
> From: hongyuw1
> Date: Wed, 18 Dec 2019 14:52:54 +
> Subject: [PATCH] Remove redundant rou
t removing existing intrinsics would
be a mistake since end-users could be using them in their code. I'd
think we'd want to keep the existing APIs, even if we change the
implementation under the hood.
Thoughts?
jeff
> Hongyu Wang
>
> 0001-Remove-redundant-round-builtins-for-avx512f-sca
For sure.
Jeff Law 于2020年1月15日周三 上午4:48写道：
>
> On Tue, 2019-12-24 at 13:31 +0800, Hongyu Wang wrote:
> > Hi:
> > For avx512f scalar instructions, current builtin function like
> > __builtin_ia32_*{sd,ss}_round can be replaced by
> > __builtin_ia32_*{sd,ss}_mask_round with mask parameter set to
On Tue, 2019-12-24 at 13:31 +0800, Hongyu Wang wrote:
> Hi:
> For avx512f scalar instructions, current builtin function like
> __builtin_ia32_*{sd,ss}_round can be replaced by
> __builtin_ia32_*{sd,ss}_mask_round with mask parameter set to -1. This
> patch did the replacement and remove the
: [PATCH] Remove redundant round builtins for avx512f scalar
instructions
Changelog
gcc/
* config/i386/avx512fintrin.h
(_mm_add_round_sd, _mm_add_round_ss): Use
__builtin_ia32_adds?_mask_round builtins instead of
__builtin_ia32_adds?_round.
(_mm_sub_round_sd, _mm_sub_round_ss
