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

Reply via email to