[Bug tree-optimization/61403] New: An opportunity for x86 gcc vectorizer (~40% gain)

[Bug tree-optimization/61403] New: An opportunity for x86 gcc vectorizer (~40% gain)

[evstupac at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61403

            Bug ID: 61403
           Summary: An opportunity for x86 gcc vectorizer (~40% gain)
           Product: gcc
           Version: 4.10.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: tree-optimization
          Assignee: unassigned at gcc dot gnu.org
          Reporter: evstupac at gmail dot com

Test normalizing 3D coordinates failed to vectorize

#include <math.h> 

struct XYZ 
{ 
  float x; 
  float y; 
  float z; 
}; 

void 
norm (struct XYZ *in, struct XYZ *out, int size) 
{ 
  int i; 
  for (i = 0; i < size; ++i) 
    { 
      float n = sqrt(in[i].x * in[i].x + in[i].y * in[i].y + in[i].z *
in[i].z); 
      out[i].x = in[i].x / n;
      out[i].y = in[i].y / n;
      out[i].z = in[i].z / n;
    } 
} 

gcc norm.c -Ofast -S -mssse3 -fdump-tree-vect-details

norm.c:14:3: note: type of def: 3. 
norm.c:14:3: note: no array mode for V4SF[3] 
norm.c:14:3: note: the size of the group of accesses is not a power of 2
norm.c:14:3: note: not vectorized: relevant stmt not supported: _19->x = _20;

Vectorization should give ~40% gain on x86

[Bug tree-optimization/61403] An opportunity for x86 gcc vectorizer (~40% gain)

[kyukhin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61403

--- Comment #1 from Kirill Yukhin <kyukhin at gcc dot gnu.org> ---
Author: kyukhin
Date: Wed Jun 18 07:46:18 2014
New Revision: 211769

URL: https://gcc.gnu.org/viewcvs?rev=211769&root=gcc&view=rev
Log:
gcc/
    * config/i386/i386.c (ix86_reassociation_width): Add alternative for
    vector case.
    * config/i386/i386.h (TARGET_VECTOR_PARALLEL_EXECUTION): New.
    * config/i386/x86-tune.def (X86_TUNE_VECTOR_PARALLEL_EXECUTION): New.
    * tree-vect-data-refs.c (vect_shift_permute_load_chain): New.
    Introduces alternative way of loads group permutaions.
    (vect_transform_grouped_load): Try alternative way of permutations.

gcc/testsuite/
    PR tree-optimization/52252
    * gcc.target/i386/pr52252-atom.c: Test on loads group of size 3.
    * gcc.target/i386/pr52252-core.c: Ditto.

    PR tree-optimization/61403
    * gcc.target/i386/pr61403.c: Test on loads and stores group of size 3.


Added:
    trunk/gcc/testsuite/gcc.target/i386/pr52252-atom.c
    trunk/gcc/testsuite/gcc.target/i386/pr52252-core.c
    trunk/gcc/testsuite/gcc.target/i386/pr61403.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/i386/i386.c
    trunk/gcc/config/i386/i386.h
    trunk/gcc/config/i386/x86-tune.def
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-vect-data-refs.c

[Bug tree-optimization/61403] An opportunity for x86 gcc vectorizer (~40% gain)

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61403

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jakub at gcc dot gnu.org

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
So is this fixed now?

[Bug tree-optimization/61403] An opportunity for x86 gcc vectorizer (~40% gain)

[evstupac at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61403

Stupachenko Evgeny <evstupac at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Stupachenko Evgeny <evstupac at gmail dot com> ---
(In reply to Jakub Jelinek from comment #3)
> So is this fixed now?

Yes. It is fixed.