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 <tho...@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