Re: Ping: RFA: Consider int and same-size short as equivalent vector components

2013-09-07 Thread Jason Merrill
On 09/06/2013 08:58 PM, Joern Rennecke wrote: vector_targets_convertible_p is used for pointer types. The callers do a hop, skip and dance to check that the qualifiers are satisfactory, while OTOH vector_targets_convertible_p ignores the number of elements in the vectors. That's fine with vecto

Re: Ping: RFA: Consider int and same-size short as equivalent vector components

2013-09-06 Thread Joern Rennecke
Quoting Jason Merrill : On 09/05/2013 10:50 AM, Joern Rennecke wrote: (vector_types_compatible_elements_p): New function. Why do we need this as well as vector_types_convertible_p? For that matter, why do we need both vector_types_convertible_p and vector_targets_convertible_p? vec

Re: Ping: RFA: Consider int and same-size short as equivalent vector components

2013-09-06 Thread Jason Merrill
On 09/05/2013 10:50 AM, Joern Rennecke wrote: (vector_types_compatible_elements_p): New function. Why do we need this as well as vector_types_convertible_p? For that matter, why do we need both vector_types_convertible_p and vector_targets_convertible_p? Jason

RFA: accept opaque vectors as compatible to same-size,same-element-size vectors in binary operations (Was: Re: Ping: RFA: Consider int and same-size short as equivalent vector components)

2013-09-05 Thread Joern Rennecke
Quoting Nathan Sidwell : On 09/05/13 15:50, Joern Rennecke wrote: Apart from Paolo Carlini's suggestion to improve the wording of the ChangeLog, this patch hasn't been reviewed for more than a week: http://gcc.gnu.org/ml/gcc-patches/2013-08/msg01634.html I suppose the original subject has

Ping: RFA: Consider int and same-size short as equivalent vector components

2013-09-05 Thread Joern Rennecke
Apart from Paolo Carlini's suggestion to improve the wording of the ChangeLog, this patch hasn't been reviewed for more than a week: http://gcc.gnu.org/ml/gcc-patches/2013-08/msg01634.html

Re: Ping: RFA: Consider int and same-size short as equivalent vector components

2013-09-05 Thread Nathan Sidwell
On 09/05/13 15:50, Joern Rennecke wrote: Apart from Paolo Carlini's suggestion to improve the wording of the ChangeLog, this patch hasn't been reviewed for more than a week: http://gcc.gnu.org/ml/gcc-patches/2013-08/msg01634.html I don't think the issue is whether they should be treated as equi

Re: RFA: Consider int and same-size short as equivalent vector components

2013-08-27 Thread Paolo Carlini
Hi, On 08/28/2013 12:29 AM, Joern Rennecke wrote: * c-common.h: (same_scalar_type_ignoring_signedness): Delete prototype. (vector_types_compatible_elements_p): Prototype. Sorry for nitpicking, but since we are now using C++ I think declaration is more correct than prototype.

Re: RFA: Consider int and same-size short as equivalent vector components

2013-08-27 Thread Joern Rennecke
Quoting Marc Glisse : On Mon, 26 Aug 2013, Joern Rennecke wrote: Quoting Marc Glisse : The issue seems larger than just short/int. On x86, (l I don't understand what you mean here. Could you post the actual code sample? typedef long vl __attribute__((vector_size(16))); void f(vl l){

Re: RFA: Consider int and same-size short as equivalent vector components

2013-08-26 Thread Marc Glisse
On Mon, 26 Aug 2013, Joern Rennecke wrote: Quoting Marc Glisse : The issue seems larger than just short/int. On x86, (l I don't understand what you mean here. Could you post the actual code sample? typedef long vl __attribute__((vector_size(16))); void f(vl l){ (l(btw, my sentence was

Re: RFA: Consider int and same-size short as equivalent vector components

2013-08-26 Thread Joern Rennecke
Quoting Marc Glisse : The issue seems larger than just short/int. On x86, (l I don't understand what you mean here. Could you post the actual code sample?

Re: RFA: Consider int and same-size short as equivalent vector components

2013-08-26 Thread Marc Glisse
On Mon, 26 Aug 2013, Joern Rennecke wrote: avr currently shows the following failure: FAIL: c-c++-common/vector-scalar.c -Wc++-compat (test for excess errors) Excess errors: /home/amylaar/atmel/4.8/unisrc-mainline/gcc/testsuite/c-c++-common/vector-scalar .c:9:34: error: invalid operands to bin

RFA: Consider int and same-size short as equivalent vector components

2013-08-26 Thread Joern Rennecke
avr currently shows the following failure: FAIL: c-c++-common/vector-scalar.c -Wc++-compat (test for excess errors) Excess errors: /home/amylaar/atmel/4.8/unisrc-mainline/gcc/testsuite/c-c++-common/vector-scalar .c:9:34: error: invalid operands to binary | (have '__vector(8) int' and 'veci')