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