This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFC] Add check_effective_target_vect_min_max
- From: Rainer Orth <ro at CeBiTec dot Uni-Bielefeld dot DE>
- To: Tom de Vries <Tom_deVries at mentor dot com>
- Cc: "gcc-patches\ at gnu dot org" <gcc-patches at gnu dot org>, Richard Biener <rguenther at suse dot de>
- Date: Wed, 12 Aug 2015 11:18:33 +0200
- Subject: Re: [RFC] Add check_effective_target_vect_min_max
- Authentication-results: sourceware.org; auth=none
- References: <55CAE044 dot 9040006 at mentor dot com> <55CAE8FA dot 1020504 at mentor dot com> <yddmvxwc2k0 dot fsf at lokon dot CeBiTec dot Uni-Bielefeld dot DE> <55CB0E85 dot 3020403 at mentor dot com>
Tom de Vries <Tom_deVries@mentor.com> writes:
> On 12/08/15 10:51, Rainer Orth wrote:
>> Tom de Vries <Tom_deVries@mentor.com> writes:
>>
>>> This follow-up patch introduces a new effective target vect_min_max,
>>> similar to how effective target vect_bswap is implemented.
>>>
>>> Any comments?
>>>
>>> Thanks,
>>> - Tom
>>>
>>> Add check_effective_target_vect_min_max
>>>
>>> 2015-08-12 Tom de Vries <tom@codesourcery.com>
>>>
>>> * lib/target-supports.exp (check_effective_target_vect_min_max): New
>>> proc.
>>> * gcc.dg/vect/trapv-vect-reduc-4.c: Use vect_min_max effective target.
>>
>> Looks good to me, but the new effective-target keyword needs documenting
>> in sourcebuild.texi.
>>
>
> Hmm, in sourcebuild.texi I found:
> ...
> @item vect_no_int_max
> Target does not support a vector max instruction on @code{int}.
> ...
>
> That looks related. [ I also found a patch introducing vect_no_uint_max
> here: https://gcc.gnu.org/ml/gcc-patches/2010-01/msg00152.html. ]
>
> I'm not sure where to take it from here. Should I introduce
> vect_no_int_min, and use that in combination with vect_no_int_max?
I'd say this is something for the vectorizer maintainers to decide. Richi?
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University