[Bug middle-end/61252] Invalid code produced for omp simd reduction(min:var) where var is reference

[egallager at gcc dot gnu.org]

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

Eric Gallager <egallager at gcc dot gnu.org> changed:

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

--- Comment #7 from Eric Gallager <egallager at gcc dot gnu.org> ---
(In reply to Jakub Jelinek from comment #5)
> Author: jakub
> Date: Wed May 21 08:04:03 2014
> New Revision: 210679
> 
> URL: http://gcc.gnu.org/viewcvs?rev=210679&root=gcc&view=rev
> Log:
> 	PR middle-end/61252
> 	* omp-low.c (handle_simd_reference): New function.
> 	(lower_rec_input_clauses): Use it.  Defer adding reference
> 	initialization even for reduction without placeholder if in simd,
> 	handle it properly later on.
> 
> 	* testsuite/libgomp.c++/simd-9.C: New test.
> 
> Added:
>     trunk/libgomp/testsuite/libgomp.c++/simd-9.C
> Modified:
>     trunk/gcc/ChangeLog
>     trunk/gcc/omp-low.c
>     trunk/libgomp/ChangeLog

(In reply to Jakub Jelinek from comment #6)
> Author: jakub
> Date: Wed May 21 08:06:04 2014
> New Revision: 210681
> 
> URL: http://gcc.gnu.org/viewcvs?rev=210681&root=gcc&view=rev
> Log:
> 	PR middle-end/61252
> 	* omp-low.c (handle_simd_reference): New function.
> 	(lower_rec_input_clauses): Use it.  Defer adding reference
> 	initialization even for reduction without placeholder if in simd,
> 	handle it properly later on.
> 
> 	* testsuite/libgomp.c++/simd-9.C: New test.
> 
> Added:
>     branches/gcc-4_9-branch/libgomp/testsuite/libgomp.c++/simd-9.C
> Modified:
>     branches/gcc-4_9-branch/gcc/ChangeLog
>     branches/gcc-4_9-branch/gcc/omp-low.c
>     branches/gcc-4_9-branch/libgomp/ChangeLog

Did these fix it?