[PING][PATCH, PR67709 ] Don't call call_cgraph_insertion_hooks in simd_clone_create

Tom de Vries Tom_deVries@mentor.com
Tue Feb 16 20:52:00 GMT 2016


On 16/02/16 17:54, Jakub Jelinek wrote:
> On Tue, Feb 16, 2016 at 05:52:55PM +0100, Tom de Vries wrote:
>> Committed both patches to 4.9 and 5 branches.
>>
>> In order to run testsuite/libgomp.fortran/declare-simd-4.f90 with the 4.9
>> branch build, I needed in addition:
>> - r212268
>>    https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=212268
>>    https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00218.html
>> - r219606
>>    https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=219606
>>    https://gcc.gnu.org/ml/gcc-patches/2015-01/msg00615.html
>>
>> Should these also be back-ported to 4.9 branch (and in case of r219606, also
>> follow-up fix r219722,
>> https://gcc.gnu.org/viewcvs?rev=219722&root=gcc&view=rev , fix for
>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64605 )?
>
> IMHO just don't backport that test, test it by hand only.
>

I've already committed the testcase, and without these two patches the 
test shows up as unsupported, so I'll leave it at that.

As for the lto bit (r219606) I can see your point, it's the only libgomp 
test that uses lto.

But there are 16 test cases which are currently unsupported in the 4.9 
branch, and would be supported with the fortran patch (r212268):
...
libgomp.fortran/allocatable11.f90
libgomp.fortran/allocatable2.f90
libgomp.fortran/allocatable8.f90
libgomp.fortran/appendix-a/a.22.7.f90
libgomp.fortran/appendix-a/a.22.8.f90
libgomp.fortran/crayptr2.f90
libgomp.fortran/declare-simd-1.f90
libgomp.fortran/declare-simd-2.f90
libgomp.fortran/omp_parse3.f90
libgomp.fortran/pointer2.f90
libgomp.fortran/pr25162.f
libgomp.fortran/pr32550.f90
libgomp.fortran/threadprivate1.f90
libgomp.fortran/threadprivate2.f90
libgomp.fortran/threadprivate3.f90
libgomp.fortran/threadprivate4.f90
...
Perhaps that makes it worthwhile to back-port the patch.

Thanks,
- Tom



More information about the Gcc-patches mailing list