Split out OMP constructs' SIMD clone supporting code
Thomas Schwinge
thomas@codesourcery.com
Fri Apr 15 06:25:00 GMT 2016
Hi!
On Thu, 14 Apr 2016 22:27:40 +0200, I wrote:
> On Thu, 14 Apr 2016 18:01:13 +0200, I wrote:
> > "simdclone" pass, with the
> > respective supporting code. I will certainly submit line-diff patches if
> > we agree that this is sound -- these two may actually be good candidates
> > to do first, individually, and do that now, because they're completely
> > self-contained. Makes sense?
>
> ;-) Made enough sense to me, so that I prepared the attached patch. I'm
> also attaching a "-C" variant that I created using Git's -C5% option, and
> which shows how the new file gcc/omp-simd-clone.c can be created from the
> original gcc/omp-low.c. (Is that useful for review, or are you manually
> doing something like that anyway?)
>
> > Should possibly rename omp-simd.c to omp-simd-clone.c to make it clear
> > that's the only thing it does, the "simdclone" pass?
>
> I did that.
>
> I manually determined a reduced #include list for the new file
> gcc/omp-simd-clone.c. Hope that's alright.
>
> OK to commit once bootstrap testing succeeded?
Bootstrap/testing look good. OK to commit?
> commit 8f33dc59ad24a995694d42ee9013e0853426e190
> Author: Thomas Schwinge <thomas@codesourcery.com>
> Date: Thu Apr 14 21:56:31 2016 +0200
>
> Split out OMP constructs' SIMD clone supporting code
>
> gcc/
> * omp-low.c (simd_clone_struct_alloc, simd_clone_struct_copy)
> (simd_clone_vector_of_formal_parm_types)
> (simd_clone_clauses_extract, simd_clone_compute_base_data_type)
> (simd_clone_mangle, simd_clone_create)
> (simd_clone_adjust_return_type, create_tmp_simd_array)
> (simd_clone_adjust_argument_types, simd_clone_init_simd_arrays)
> (struct modify_stmt_info, ipa_simd_modify_stmt_ops)
> (ipa_simd_modify_function_body, simd_clone_linear_addend)
> (simd_clone_adjust, expand_simd_clones, ipa_omp_simd_clone)
> (pass_data_omp_simd_clone, class pass_omp_simd_clone)
> (pass_omp_simd_clone::gate, make_pass_omp_simd_clone): Move
> into...
> * omp-simd-clone.c: ... this new file.
> (simd_clone_vector_of_formal_parm_types): Make it static.
> * Makefile.in (OBJS): Add omp-simd-clone.o.
>
> gcc/Makefile.in | 1 +
> gcc/omp-low.c | 1606 ------------------------------------------------
> gcc/omp-simd-clone.c | 1654 ++++++++++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 1655 insertions(+), 1606 deletions(-)
Grüße
Thomas
More information about the Gcc-patches
mailing list