Changes for these files(except sourcebuild one, will fix that) are included in patch I sent. I only removed from the patch deletion of the folders I mentioned.
Julia > -----Original Message----- > From: Rainer Orth [mailto:r...@cebitec.uni-bielefeld.de] > Sent: Wednesday, November 22, 2017 10:11 AM > To: Koval, Julia <julia.ko...@intel.com> > Cc: Jeff Law <l...@redhat.com>; Jakub Jelinek <ja...@redhat.com>; GCC > Patches <gcc-patches@gcc.gnu.org> > Subject: Re: [patch] remove cilk-plus > > Hi Julia, > > >> So it's not important, but the patch doesn't have the removal of the > >> cilk+ testsuite or runtime. BUt again, it's not a big deal, I can guess > >> what that part of the patch looks like. > > > > I used Jakub's suggestion in > > https://gcc.gnu.org/ml/gcc-patches/2017-11/msg01348.html and didn't add > > libcilkrts and cilk-plus directories to patch(without this patch doesn't > > fit in gcc-patches), only to change log. I will include them, when I'll > > commit the patch, but I guess there is nothing to review here: > > rm -rf gcc/testsuite/c-c++-common/cilk-plus > > rm -rf gcc/testsuite/g++.dg/cilk-plus > > rm -rf gcc/testsuite/gcc.dg/cilk-plus > > rm -rf libcilkrts > > I can send it as an additional patch(or patches) if this is required. > > there's more in the testsuite: > > gcc/testsuite/lib/cilk-plus-dg.exp > gcc/testsuite/lib/target-supports.exp (check_effective_target_cilkplus, > check_effective_target_cilkplus_runtime) > gcc/doc/sourcebuild.texi (cilkplus_runtime effective-target keyword) > > Rainer > > -- > ----------------------------------------------------------------------------- > Rainer Orth, Center for Biotechnology, Bielefeld University
