public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [patch] remove cilk-plus
@ 2017-11-16 15:48 Koval, Julia
  2017-11-16 15:49 ` Marek Polacek
                   ` (4 more replies)
  0 siblings, 5 replies; 26+ messages in thread
From: Koval, Julia @ 2017-11-16 15:48 UTC (permalink / raw)
  To: GCC Patches; +Cc: law

// I failed to send patch itself, it is too big even in gzipped form.  What is the right way to send such big patches?

Hi, this patch removes cilkplus. Ok for trunk?

2017-11-16  Julia Koval  <julia.koval@intel.com>
	        Sebastian Peryt  <sebastian.peryt@intel.com>
gcc/
	* Makefile.def (target_modules): Remove libcilkrts.
	* Makefile.in: Ditto.
	* configure: Ditto.
	* configure.ac: Ditto.
	* contrib/gcc_update: Ditto.
	* Makefile.in (cilkplus.def, cilk-builtins.def, c-family/cilk.o, 
	c-family/c-cilkplus.o, c-family/array-notation-common.o,
	cilk-common.o, cilk.h, cilk-common.c): Remove.
	* builtin-types.def (BT_FN_INT_PTR_PTR_PTR_FTYPE_BT_INT_BT_PTR_BT_PTR
	_BT_PTR): Remove.
	* builtins.c (is_builtin_name): Remove cilkplus condition.
	(BUILT_IN_CILK_DETACH, BUILT_IN_CILK_POP_FRAME): Remove.
	* builtins.def (DEF_CILK_BUILTIN_STUB, DEF_CILKPLUS_BUILTIN,
	cilk-builtins.def, cilkplus.def): Remove.
	* c-family/array-notation-common.c: Delete.
	* c-family/c-cilkplus.c: Ditto.
	* c-family/c-common.c (_Cilk_spawn, _Cilk_sync, _Cilk_for): Remove.
	* c-family/c-common.def (ARRAY_NOTATION_REF): Remove.
	* c-family/c-common.h (RID_CILK_SPAWN, build_array_notation_expr,
	build_array_notation_ref, C_ORT_CILK, c_check_cilk_loop,
	c_validate_cilk_plus_loop, cilkplus_an_parts, cilk_ignorable_spawn_rhs_op,
	cilk_recognize_spawn): Remove.
	* c-family/c-gimplify.c (CILK_SPAWN_STMT): Remove.
	* c-family/c-omp.c: Remove CILK_SIMD check.
	* c-family/c-pragma.c: Ditto.
	* c-family/c-pragma.h: Remove CILK related pragmas.
	* c-family/c-pretty-print.c (c_pretty_printer::postfix_expression): Remove
	ARRAY_NOTATION_REF condition.
	(c_pretty_printer::expression): Ditto.
	* c-family/c.opt (fcilkplus): Remove.
	* c-family/cilk.c: Delete.
	* c/Make-lang.in (c/c-array-notation.o): Remove.
	* c/c-array-notation.c: Delete.
	* c/c-decl.c: Remove cilkplus condition.
	* c/c-parser.c (c_parser_cilk_simd, c_parser_cilk_for,
	c_parser_cilk_verify_simd, c_parser_array_notation,
	c_parser_cilk_clause_vectorlength, c_parser_cilk_grainsize,
	c_parser_cilk_simd_fn_vector_attrs, c_finish_cilk_simd_fn_tokens): Delete.
	(c_parser_declaration_or_fndef): Remove cilkplus condition.
	(c_parser_direct_declarator_inner): Ditto.
	(CILK_SIMD_FN_CLAUSE_MASK): Delete.
	(c_parser_attributes): Remove cilk-plus condition.
	(c_parser_compound_statement): Ditto.
	(c_parser_statement_after_labels): Ditto.
	(c_parser_if_statement): Ditto.
	(c_parser_switch_statement): Ditto.
	(c_parser_while_statement): Ditto.
	(c_parser_do_statement): Ditto.
	(c_parser_for_statement): Ditto.
	(c_parser_unary_expression): Ditto.
	(c_parser_postfix_expression): Ditto.
	(c_parser_postfix_expression_after_primary): Ditto.
	(c_parser_pragma): Ditto.
	(c_parser_omp_clause_name): Ditto.
	(c_parser_omp_all_clauses): Ditto.
	(c_parser_omp_for_loop): Ditto.
	(c_finish_omp_declare_simd): Ditto.
	* c/c-typeck.c (build_array_ref, build_function_call_vec, convert_arguments,
	lvalue_p, build_compound_expr, c_finish_return, c_finish_if_stmt,
	c_finish_loop, build_binary_op): Remove cilkplus condition.
	* cif-code.def (CILK_SPAWN): Remove.
	* cilk-builtins.def: Delete.
	* cilk-common.c: Ditto.
	* cilk.h: Ditto.
	* cilkplus.def: Ditto.
	* config/darwin.h (fcilkplus): Delete.
	* cp/Make-lang.in (cp/cp-array-notation.o, cp/cp-cilkplus.o): Delete.
	* cp/call.c (convert_for_arg_passing, build_cxx_call): Remove cilkplus.
	* cp/constexpr.c (potential_constant_expression_1): Ditto.
	* cp/cp-array-notation.c: Delete.
	* cp/cp-cilkplus.c: Ditto.
	* cp/cp-cilkplus.h: Ditto.
	* cp/cp-gimplify.c (cp_gimplify_expr, cp_fold_r, cp_genericize): Remove
	cilkplus condition.
	* cp/cp-objcp-common.c (ARRAY_NOTATION_REF): Delete.
	* cp/cp-tree.h (cilkplus_an_triplet_types_ok_p): Delete.
	* cp/decl.c (grokfndecl, finish_function): Remove cilkplus condition.
	* cp/error.c (dump_decl, dump_expr): Remove ARRAY_NOTATION_REF condition.
	* cp/lambda.c (cp-cilkplus.h): Remove.
	* cp/parser.c (cp_parser_cilk_simd, cp_parser_cilk_for,
	cp_parser_cilk_simd_vectorlength): Delete.
	(cp_debug_parser, cp_parser_ctor_initializer_opt_and_function_body,
	cp_parser_postfix_expression, cp_parser_postfix_open_square_expression,
	cp_parser_statement, cp_parser_jump_statement, cp_parser_direct_declarator,
	cp_parser_late_return_type_opt, cp_parser_gnu_attribute_list,
	cp_parser_omp_clause_name, cp_parser_omp_clause_aligned,
	cp_parser_omp_clause_linear, cp_parser_omp_all_clauses, cp_parser_omp_flush,
	cp_parser_omp_for_cond, cp_parser_omp_for_incr, cp_parser_omp_for_loop_init,
	cp_parser_omp_for_loop, cp_parser_omp_declare_simd): Remove cilkplus condtions.
	(CILK_SIMD_FN_CLAUSE_MASK, cp_parser_late_parsing_cilk_simd_fn_info,
	cp_parser_cilk_grainsize): Remove.
	(cp_parser_pragma, c_parse_file): Remove cilkplus condition.
	(cp_parser_cilk_simd_vectorlength, cp_parser_cilk_simd_linear,
	cp_parser_cilk_simd_clause_name, cp_parser_cilk_simd_all_clauses,
	cp_parser_cilk_simd, cp_parser_cilk_for): Remove.
	* cp/parser.h (IN_CILK_SIMD_FOR, IN_CILK_SPAWN): Remove.
	* cp/pt.c (tsubst_attribute, tsubst_expr, tsubst_copy_and_build): Remove
	cilkplus condition.
	* cp/semantics.c (finish_goto_stmt, begin_while_stmt, finish_do_body,
	finish_init_stmt, finish_switch_cond, simplify_aggr_init_expr,
	finish_omp_clauses, finish_omp_clauses, finish_omp_for): Remove cilkplus
	condition.
	* cp/tree.c (lvalue_kind): Remove ARRAY_NOTATION_REF conditon.
	* cp/typeck.c (cp_build_array_ref, cp_build_compound_expr,
	check_return_expr): Remove cilkplus conditions.
	* cppbuiltin.c: Ditto.
	* doc/extend.texi: Remove cilkplus doc.
	* doc/generic.texi: Ditto.
	* doc/invoke.texi: Ditto.
	* doc/passes.texi: Ditto.
	* gcc.c (fcilkplus): Remove.
	* gengtype.c (cilk.h): Remove.
	* gimple-pretty-print.c (dump_gimple_omp_for): Remove cilkplus conditions.
	* gimple.h (GF_OMP_FOR_KIND_CILKFOR, GF_OMP_FOR_KIND_CILKSIMD): Remove.
	* gimplify.c (gimplify_return_expr, maybe_fold_stmt, gimplify_call_expr,
	is_gimple_stmt, gimplify_modify_expr, gimplify_scan_omp_clauses,
	gimplify_adjust_omp_clauses, gimplify_omp_for, gimplify_expr): Remove
	cilkplus conditions.
	* ipa-fnsummary.c (ipa_dump_fn_summary, compute_fn_summary,
	inline_read_section): Ditto.
	* ipa-inline-analysis.c (cilk.h): Remove.
	* ira.c (ira_setup_eliminable_regset): Remove cilkplus condition.
	* lto-wrapper.c (merge_and_complain, append_compiler_options,
	append_linker_options): Remove condition for fcilkplus.
	* lto/lto-lang.c (cilk.h): Remove.
	(lto_init): Remove condition for fcilkplus.
	* omp-expand.c (expand_cilk_for_call): Delete.
	(expand_omp_taskreg, expand_omp_for_static_chunk,
	expand_omp_for): Remove cilkplus
	conditions.
	(expand_cilk_for): Delete.
	* omp-general.c (omp_extract_for_data): Remove cilkplus condition.
	* omp-low.c (scan_sharing_clauses, create_omp_child_function,
	execute_lower_omp, diagnose_sb_0): Ditto.
	* omp-simd-clone.c (simd_clone_clauses_extract): Ditto.
	* tree-core.h
	* tree-nested.c
	* tree-pretty-print.c
	* tree.c
	* tree.def
	* tree.h

gcc/testsuite/
	* c-c++-common/attr-simd-3.c: Delete.
	* c-c++-common/cilk-plus/AN/an-if.c: Delete.
	* c-c++-common/cilk-plus/AN/array_test1.c: Delete.
	* c-c++-common/cilk-plus/AN/array_test2.c: Delete.
	* c-c++-common/cilk-plus/AN/array_test_ND.c: Delete.
	* c-c++-common/cilk-plus/AN/builtin_fn_custom.c: Delete.
	* c-c++-common/cilk-plus/AN/builtin_fn_mutating.c: Delete.
	* c-c++-common/cilk-plus/AN/builtin_func_double.c: Delete.
	* c-c++-common/cilk-plus/AN/builtin_func_double2.c: Delete.
	* c-c++-common/cilk-plus/AN/comma_exp.c: Delete.
	* c-c++-common/cilk-plus/AN/conditional.c: Delete.
	* c-c++-common/cilk-plus/AN/decl-ptr-colon.c: Delete.
	* c-c++-common/cilk-plus/AN/dimensionless-arrays.c: Delete.
	* c-c++-common/cilk-plus/AN/exec-once.c: Delete.
	* c-c++-common/cilk-plus/AN/exec-once2.c: Delete.
	* c-c++-common/cilk-plus/AN/fn_ptr-2.c: Delete.
	* c-c++-common/cilk-plus/AN/fn_ptr.c: Delete.
	* c-c++-common/cilk-plus/AN/fp_triplet_values.c: Delete.
	* c-c++-common/cilk-plus/AN/gather-scatter-errors.c: Delete.
	* c-c++-common/cilk-plus/AN/gather_scatter.c: Delete.
	* c-c++-common/cilk-plus/AN/if_test.c: Delete.
	* c-c++-common/cilk-plus/AN/if_test_errors.c: Delete.
	* c-c++-common/cilk-plus/AN/misc.c: Delete.
	* c-c++-common/cilk-plus/AN/n-ptr-test.c: Delete.
	* c-c++-common/cilk-plus/AN/parser_errors.c: Delete.
	* c-c++-common/cilk-plus/AN/parser_errors2.c: Delete.
	* c-c++-common/cilk-plus/AN/parser_errors3.c: Delete.
	* c-c++-common/cilk-plus/AN/parser_errors4.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57457-2.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57457.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57490.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57541-2.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57541.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57577.c: Delete.
	* c-c++-common/cilk-plus/AN/pr58942.c: Delete.
	* c-c++-common/cilk-plus/AN/pr61191.c: Delete.
	* c-c++-common/cilk-plus/AN/pr61455-2.c: Delete.
	* c-c++-common/cilk-plus/AN/pr61455.c: Delete.
	* c-c++-common/cilk-plus/AN/pr61962.c: Delete.
	* c-c++-common/cilk-plus/AN/pr61963.c: Delete.
	* c-c++-common/cilk-plus/AN/pr62008.c: Delete.
	* c-c++-common/cilk-plus/AN/pr63884.c: Delete.
	* c-c++-common/cilk-plus/AN/rank_mismatch.c: Delete.
	* c-c++-common/cilk-plus/AN/rank_mismatch2.c: Delete.
	* c-c++-common/cilk-plus/AN/rank_mismatch3.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_implicit.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_implicit2.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_implicit_ex.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_reduce_ind_same_value.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_reduce_max_min_ind.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_reduce_return.c: Delete.
	* c-c++-common/cilk-plus/AN/side-effects-1.c: Delete.
	* c-c++-common/cilk-plus/AN/test_builtin_return.c: Delete.
	* c-c++-common/cilk-plus/AN/test_sec_limits.c: Delete.
	* c-c++-common/cilk-plus/AN/tst_lngth.c: Delete.
	* c-c++-common/cilk-plus/AN/vla.c: Delete.
	* c-c++-common/cilk-plus/CK/Wparentheses-1.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk-for-2.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk-for-3.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk-fors.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk_for_errors.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk_for_grain.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk_for_grain_errors.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk_for_ptr_iter.c: Delete.
	* c-c++-common/cilk-plus/CK/compound_cilk_spawn.c: Delete.
	* c-c++-common/cilk-plus/CK/concec_cilk_spawn.c: Delete.
	* c-c++-common/cilk-plus/CK/errors.c: Delete.
	* c-c++-common/cilk-plus/CK/fib.c: Delete.
	* c-c++-common/cilk-plus/CK/fib_init_expr_xy.c: Delete.
	* c-c++-common/cilk-plus/CK/fib_no_return.c: Delete.
	* c-c++-common/cilk-plus/CK/fib_no_sync.c: Delete.
	* c-c++-common/cilk-plus/CK/invalid_spawns.c: Delete.
	* c-c++-common/cilk-plus/CK/invalid_sync.c: Delete.c
	* c-c++-common/cilk-plus/CK/nested_cilk_for.c: Delete.
	* c-c++-common/cilk-plus/CK/no_args_error.c: Delete.
	* c-c++-common/cilk-plus/CK/pr59631.c: Delete.
	* c-c++-common/cilk-plus/CK/pr60197-2.c: Delete.
	* c-c++-common/cilk-plus/CK/pr60197.c: Delete.
	* c-c++-common/cilk-plus/CK/pr60469.c: Delete.
	* c-c++-common/cilk-plus/CK/pr60586.c: Delete.
	* c-c++-common/cilk-plus/CK/pr63307.c: Delete.
	* c-c++-common/cilk-plus/CK/pr69826-1.c: Delete.
	* c-c++-common/cilk-plus/CK/pr69826-2.c: Delete.
	* c-c++-common/cilk-plus/CK/pr79428-4.c: Delete.
	* c-c++-common/cilk-plus/CK/pr79428-7.c: Delete.
	* c-c++-common/cilk-plus/CK/spawn_in_return.c: Delete.
	* c-c++-common/cilk-plus/CK/spawnee_inline.c: Delete.
	* c-c++-common/cilk-plus/CK/spawner_inline.c: Delete.
	* c-c++-common/cilk-plus/CK/spawning_arg.c: Delete.
	* c-c++-common/cilk-plus/CK/steal_check.c: Delete.
	* c-c++-common/cilk-plus/CK/sync_wo_spawn.c: Delete.
	* c-c++-common/cilk-plus/CK/test__cilk.c: Delete.
	* c-c++-common/cilk-plus/CK/varargs_test.c: Delete.
	* c-c++-common/cilk-plus/PS/Wparentheses-1.c: Delete.
	* c-c++-common/cilk-plus/PS/body.c: Delete.
	* c-c++-common/cilk-plus/PS/clauses1.c: Delete.
	* c-c++-common/cilk-plus/PS/clauses2.c: Delete.
	* c-c++-common/cilk-plus/PS/clauses3.c: Delete.
	* c-c++-common/cilk-plus/PS/clauses4.c: Delete.
	* c-c++-common/cilk-plus/PS/for1.c: Delete.
	* c-c++-common/cilk-plus/PS/for2.c: Delete.
	* c-c++-common/cilk-plus/PS/for3.c: Delete.
	* c-c++-common/cilk-plus/PS/pr69363.c: Delete.
	* c-c++-common/cilk-plus/PS/reduction-1.c: Delete.
	* c-c++-common/cilk-plus/PS/reduction-2.c: Delete.
	* c-c++-common/cilk-plus/PS/reduction-3.c: Delete.
	* c-c++-common/cilk-plus/PS/run-1.c: Delete.
	* c-c++-common/cilk-plus/PS/safelen.c: Delete.
	* c-c++-common/cilk-plus/PS/vectorlength-2.c: Delete.
	* c-c++-common/cilk-plus/PS/vectorlength-3.c: Delete.
	* c-c++-common/cilk-plus/PS/vectorlength.c: Delete.
	* c-c++-common/cilk-plus/SE/ef_error.c: Delete.
	* c-c++-common/cilk-plus/SE/ef_error2.c: Delete.
	* c-c++-common/cilk-plus/SE/ef_error3.c: Delete.
	* c-c++-common/cilk-plus/SE/ef_test.c: Delete.
	* c-c++-common/cilk-plus/SE/ef_test2.c: Delete.
	* c-c++-common/cilk-plus/SE/vlength_errors.c: Delete.
	* g++.dg/cilk-plus/AN/array_function.c: Delete.c
	* g++.dg/cilk-plus/AN/array_test1_tplt.c: Delete.c
	* g++.dg/cilk-plus/AN/array_test2_tplt.c: Delete.c
	* g++.dg/cilk-plus/AN/array_test_ND_tplt.c: Delete.c
	* g++.dg/cilk-plus/AN/braced_list.c: Delete.c
	* g++.dg/cilk-plus/AN/builtin_fn_custom_tplt.c: Delete.c
	* g++.dg/cilk-plus/AN/builtin_fn_mutating_tplt.c: Delete.c
	* g++.dg/cilk-plus/AN/fp_triplet_values_tplt.c: Delete.
	* g++.dg/cilk-plus/AN/postincr_test.c: Delete.c
	* g++.dg/cilk-plus/AN/preincr_test.c: Delete.c
	* g++.dg/cilk-plus/CK/catch_exc.c: Delete.c
	* g++.dg/cilk-plus/CK/cf3.c: Delete.c
	* g++.dg/cilk-plus/CK/cilk-for-tplt.c: Delete.c
	* g++.dg/cilk-plus/CK/const_spawn.c: Delete.c
	* g++.dg/cilk-plus/CK/fib-opr-overload.c: Delete.c
	* g++.dg/cilk-plus/CK/fib-tplt.c: Delete.c
	* g++.dg/cilk-plus/CK/for1.c: Delete.c
	* g++.dg/cilk-plus/CK/lambda_spawns.c: Delete.c
	* g++.dg/cilk-plus/CK/lambda_spawns_tplt.c: Delete.c
	* g++.dg/cilk-plus/CK/pr60586.c: Delete.c
	* g++.dg/cilk-plus/CK/pr66326.c: Delete.c
	* g++.dg/cilk-plus/CK/pr68001.c: Delete.c
	* g++.dg/cilk-plus/CK/pr68997.c: Delete.c
	* g++.dg/cilk-plus/CK/pr69024.c: Delete.c
	* g++.dg/cilk-plus/CK/pr69048.c: Delete.c
	* g++.dg/cilk-plus/CK/pr69267.c: Delete.c
	* g++.dg/cilk-plus/CK/pr80038.c: Delete.c
	* g++.dg/cilk-plus/CK/stl_iter.c: Delete.c
	* g++.dg/cilk-plus/CK/stl_rev_iter.c: Delete.c
	* g++.dg/cilk-plus/CK/stl_test.c: Delete.c
	* g++.dg/cilk-plus/cilk-plus.exp
	* g++.dg/cilk-plus/ef_test.C: Delete.
	* g++.dg/cilk-plus/for.C: Delete.
	* g++.dg/cilk-plus/for2.C: Delete.
	* g++.dg/cilk-plus/for3.C: Delete.
	* g++.dg/cilk-plus/for4.C: Delete.
	* g++.dg/cilk-plus/pr60967.C: Delete.
	* g++.dg/cilk-plus/pr69028.C: Delete.
	* g++.dg/cilk-plus/pr70565.C: Delete.
	* g++.dg/pr57662.C: Delete.
	* gcc.dg/cilk-plus/cilk-plus.exp
	* gcc.dg/cilk-plus/for1.c: Delete.
	* gcc.dg/cilk-plus/for2.c: Delete.
	* gcc.dg/cilk-plus/jump-openmp.c: Delete.
	* gcc.dg/cilk-plus/jump.c: Delete.
	* gcc.dg/cilk-plus/pr69798-1.c: Delete.
	* gcc.dg/cilk-plus/pr69798-2.c: Delete.
	* gcc.dg/cilk-plus/pr78306.c: Delete.
	* gcc.dg/cilk-plus/pr79116.c: Delete.
	* gcc.dg/graphite/id-28.c: Delete.
	* lib/cilk-plus-dg.exp: Delete.
	* lib/target-supports.exp (cilkplus_runtime): Delete.

libcilkrts: Delete

Thanks,
Julia

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-16 15:48 [patch] remove cilk-plus Koval, Julia
@ 2017-11-16 15:49 ` Marek Polacek
  2017-11-16 16:21 ` Jakub Jelinek
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 26+ messages in thread
From: Marek Polacek @ 2017-11-16 15:49 UTC (permalink / raw)
  To: Koval, Julia; +Cc: GCC Patches, law

On Thu, Nov 16, 2017 at 03:33:40PM +0000, Koval, Julia wrote:
> // I failed to send patch itself, it is too big even in gzipped form.  What is the right way to send such big patches?
 
You can split the patch and then post each part in a separate e-mail.
Easier to review, too.

> Hi, this patch removes cilkplus. Ok for trunk?

Happy to see this, but the CL will have to be adjusted, e.g. no "c-family/"
prefix and similar.

	Marek

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-16 15:48 [patch] remove cilk-plus Koval, Julia
  2017-11-16 15:49 ` Marek Polacek
@ 2017-11-16 16:21 ` Jakub Jelinek
  2017-11-16 17:06   ` Koval, Julia
  2017-11-16 16:34 ` Eric Gallager
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 26+ messages in thread
From: Jakub Jelinek @ 2017-11-16 16:21 UTC (permalink / raw)
  To: Koval, Julia; +Cc: GCC Patches, law

On Thu, Nov 16, 2017 at 03:33:40PM +0000, Koval, Julia wrote:
> // I failed to send patch itself, it is too big even in gzipped form.  What is the right way to send such big patches?

Don't include the libcilkrts subtree in the patch nor /cilk-plus/
testcases that are going to be removed?

> Hi, this patch removes cilkplus. Ok for trunk?
> 
> 2017-11-16  Julia Koval  <julia.koval@intel.com>
> 	        Sebastian Peryt  <sebastian.peryt@intel.com>
> gcc/
> 	* Makefile.def (target_modules): Remove libcilkrts.
> 	* Makefile.in: Ditto.
> 	* configure: Ditto.
> 	* configure.ac: Ditto.

The ChangeLog needs work, e.g. we have many different ChangeLog files and
changes should be relative to that.  The above entries are for toplevel.

> 	* contrib/gcc_update: Ditto.

This one is for contrib/ChangeLog, so should be without contrib/
in the entry.

> 	* Makefile.in (cilkplus.def, cilk-builtins.def, c-family/cilk.o, 
> 	c-family/c-cilkplus.o, c-family/array-notation-common.o,
> 	cilk-common.o, cilk.h, cilk-common.c): Remove.
> 	* builtin-types.def (BT_FN_INT_PTR_PTR_PTR_FTYPE_BT_INT_BT_PTR_BT_PTR
> 	_BT_PTR): Remove.

There should be no linebreaks within one identifier.  So
	* builtin-types.def
	(BT_FN_INT_PTR_PTR_PTR_FTYPE_BT_INT_BT_PTR_BT_PTR_BT_PTR): Remove.

> 	* c-family/array-notation-common.c: Delete.
> 	* c-family/c-cilkplus.c: Ditto.
> 	* c-family/c-common.c (_Cilk_spawn, _Cilk_sync, _Cilk_for): Remove.
> 	* c-family/c-common.def (ARRAY_NOTATION_REF): Remove.
> 	* c-family/c-common.h (RID_CILK_SPAWN, build_array_notation_expr,
> 	build_array_notation_ref, C_ORT_CILK, c_check_cilk_loop,
> 	c_validate_cilk_plus_loop, cilkplus_an_parts, cilk_ignorable_spawn_rhs_op,
> 	cilk_recognize_spawn): Remove.
> 	* c-family/c-gimplify.c (CILK_SPAWN_STMT): Remove.
> 	* c-family/c-omp.c: Remove CILK_SIMD check.
> 	* c-family/c-pragma.c: Ditto.
> 	* c-family/c-pragma.h: Remove CILK related pragmas.
> 	* c-family/c-pretty-print.c (c_pretty_printer::postfix_expression): Remove
> 	ARRAY_NOTATION_REF condition.
> 	(c_pretty_printer::expression): Ditto.
> 	* c-family/c.opt (fcilkplus): Remove.
> 	* c-family/cilk.c: Delete.

c-family has its own ChangeLog, no c-family/ prefix (similarly for c/, cp/,
etc.).

> 	* c/Make-lang.in (c/c-array-notation.o): Remove.
> 	* c/c-array-notation.c: Delete.
> 	* c/c-decl.c: Remove cilkplus condition.
> 	* c/c-parser.c (c_parser_cilk_simd, c_parser_cilk_for,
> 	c_parser_cilk_verify_simd, c_parser_array_notation,
> 	c_parser_cilk_clause_vectorlength, c_parser_cilk_grainsize,
 	c_parser_cilk_simd_fn_vector_attrs, c_finish_cilk_simd_fn_tokens): Delete.

Too long line.

> 	(c_parser_declaration_or_fndef): Remove cilkplus condition.
> 	(c_parser_direct_declarator_inner): Ditto.
> 	(CILK_SIMD_FN_CLAUSE_MASK): Delete.
> 	(c_parser_attributes): Remove cilk-plus condition.
> 	(c_parser_compound_statement): Ditto.
> 	(c_parser_statement_after_labels): Ditto.
> 	(c_parser_if_statement): Ditto.
> 	(c_parser_switch_statement): Ditto.
> 	(c_parser_while_statement): Ditto.
> 	(c_parser_do_statement): Ditto.
> 	(c_parser_for_statement): Ditto.
> 	(c_parser_unary_expression): Ditto.
> 	(c_parser_postfix_expression): Ditto.
> 	(c_parser_postfix_expression_after_primary): Ditto.
> 	(c_parser_pragma): Ditto.
> 	(c_parser_omp_clause_name): Ditto.
> 	(c_parser_omp_all_clauses): Ditto.
> 	(c_parser_omp_for_loop): Ditto.
> 	(c_finish_omp_declare_simd): Ditto.

Perhaps you could shorten by writing:
	(c_parser_attributes, c_parser_compound_statement,
	c_parser_statement_after_labels, c_parser_if_statement,
	c_parser_switch_statement, c_parser_while_statement,
	c_parser_do_statement, c_parser_for_statement,
	c_parser_unary_expression, c_parser_postfix_expression,
	c_parser_postfix_expression_after_primary, c_parser_pragma,
	c_parser_omp_clause_name, c_parser_omp_all_clauses,
	c_parser_omp_for_loop, c_finish_omp_declare_simd):
	Remove cilkplus support.
etc.
 	* c/c-typeck.c (build_array_ref, build_function_call_vec, convert_arguments,

Too long line (various others).

> 	* tree-core.h
> 	* tree-nested.c
> 	* tree-pretty-print.c
> 	* tree.c
> 	* tree.def
> 	* tree.h

Description on what changed is missing.

> 	* g++.dg/cilk-plus/AN/array_function.c: Delete.c

Delete.c ?  Should be Delete.

	Jakub

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-16 15:48 [patch] remove cilk-plus Koval, Julia
  2017-11-16 15:49 ` Marek Polacek
  2017-11-16 16:21 ` Jakub Jelinek
@ 2017-11-16 16:34 ` Eric Gallager
  2017-11-16 17:59   ` Jeff Law
  2017-12-01 10:28 ` Paolo Carlini
  2018-03-02 16:37 ` Thomas Schwinge
  4 siblings, 1 reply; 26+ messages in thread
From: Eric Gallager @ 2017-11-16 16:34 UTC (permalink / raw)
  To: Koval, Julia; +Cc: GCC Patches, law

On 11/16/17, Koval, Julia <julia.koval@intel.com> wrote:
> // I failed to send patch itself, it is too big even in gzipped form.  What
> is the right way to send such big patches?
>
> Hi, this patch removes cilkplus. Ok for trunk?

I'm not a reviewer, but just as an onlooker, I'd want to see notes
about the removal in the caveats section of
https://gcc.gnu.org/gcc-8/changes.html

>
> 2017-11-16  Julia Koval  <julia.koval@intel.com>
> 	        Sebastian Peryt  <sebastian.peryt@intel.com>
> gcc/
> 	* Makefile.def (target_modules): Remove libcilkrts.
> 	* Makefile.in: Ditto.
> 	* configure: Ditto.
> 	* configure.ac: Ditto.
> 	* contrib/gcc_update: Ditto.
> 	* Makefile.in (cilkplus.def, cilk-builtins.def, c-family/cilk.o,
> 	c-family/c-cilkplus.o, c-family/array-notation-common.o,
> 	cilk-common.o, cilk.h, cilk-common.c): Remove.
> 	* builtin-types.def (BT_FN_INT_PTR_PTR_PTR_FTYPE_BT_INT_BT_PTR_BT_PTR
> 	_BT_PTR): Remove.
> 	* builtins.c (is_builtin_name): Remove cilkplus condition.
> 	(BUILT_IN_CILK_DETACH, BUILT_IN_CILK_POP_FRAME): Remove.
> 	* builtins.def (DEF_CILK_BUILTIN_STUB, DEF_CILKPLUS_BUILTIN,
> 	cilk-builtins.def, cilkplus.def): Remove.
> 	* c-family/array-notation-common.c: Delete.
> 	* c-family/c-cilkplus.c: Ditto.
> 	* c-family/c-common.c (_Cilk_spawn, _Cilk_sync, _Cilk_for): Remove.
> 	* c-family/c-common.def (ARRAY_NOTATION_REF): Remove.
> 	* c-family/c-common.h (RID_CILK_SPAWN, build_array_notation_expr,
> 	build_array_notation_ref, C_ORT_CILK, c_check_cilk_loop,
> 	c_validate_cilk_plus_loop, cilkplus_an_parts, cilk_ignorable_spawn_rhs_op,
> 	cilk_recognize_spawn): Remove.
> 	* c-family/c-gimplify.c (CILK_SPAWN_STMT): Remove.
> 	* c-family/c-omp.c: Remove CILK_SIMD check.
> 	* c-family/c-pragma.c: Ditto.
> 	* c-family/c-pragma.h: Remove CILK related pragmas.
> 	* c-family/c-pretty-print.c (c_pretty_printer::postfix_expression): Remove
> 	ARRAY_NOTATION_REF condition.
> 	(c_pretty_printer::expression): Ditto.
> 	* c-family/c.opt (fcilkplus): Remove.
> 	* c-family/cilk.c: Delete.
> 	* c/Make-lang.in (c/c-array-notation.o): Remove.
> 	* c/c-array-notation.c: Delete.
> 	* c/c-decl.c: Remove cilkplus condition.
> 	* c/c-parser.c (c_parser_cilk_simd, c_parser_cilk_for,
> 	c_parser_cilk_verify_simd, c_parser_array_notation,
> 	c_parser_cilk_clause_vectorlength, c_parser_cilk_grainsize,
> 	c_parser_cilk_simd_fn_vector_attrs, c_finish_cilk_simd_fn_tokens): Delete.
> 	(c_parser_declaration_or_fndef): Remove cilkplus condition.
> 	(c_parser_direct_declarator_inner): Ditto.
> 	(CILK_SIMD_FN_CLAUSE_MASK): Delete.
> 	(c_parser_attributes): Remove cilk-plus condition.
> 	(c_parser_compound_statement): Ditto.
> 	(c_parser_statement_after_labels): Ditto.
> 	(c_parser_if_statement): Ditto.
> 	(c_parser_switch_statement): Ditto.
> 	(c_parser_while_statement): Ditto.
> 	(c_parser_do_statement): Ditto.
> 	(c_parser_for_statement): Ditto.
> 	(c_parser_unary_expression): Ditto.
> 	(c_parser_postfix_expression): Ditto.
> 	(c_parser_postfix_expression_after_primary): Ditto.
> 	(c_parser_pragma): Ditto.
> 	(c_parser_omp_clause_name): Ditto.
> 	(c_parser_omp_all_clauses): Ditto.
> 	(c_parser_omp_for_loop): Ditto.
> 	(c_finish_omp_declare_simd): Ditto.
> 	* c/c-typeck.c (build_array_ref, build_function_call_vec,
> convert_arguments,
> 	lvalue_p, build_compound_expr, c_finish_return, c_finish_if_stmt,
> 	c_finish_loop, build_binary_op): Remove cilkplus condition.
> 	* cif-code.def (CILK_SPAWN): Remove.
> 	* cilk-builtins.def: Delete.
> 	* cilk-common.c: Ditto.
> 	* cilk.h: Ditto.
> 	* cilkplus.def: Ditto.
> 	* config/darwin.h (fcilkplus): Delete.
> 	* cp/Make-lang.in (cp/cp-array-notation.o, cp/cp-cilkplus.o): Delete.
> 	* cp/call.c (convert_for_arg_passing, build_cxx_call): Remove cilkplus.
> 	* cp/constexpr.c (potential_constant_expression_1): Ditto.
> 	* cp/cp-array-notation.c: Delete.
> 	* cp/cp-cilkplus.c: Ditto.
> 	* cp/cp-cilkplus.h: Ditto.
> 	* cp/cp-gimplify.c (cp_gimplify_expr, cp_fold_r, cp_genericize): Remove
> 	cilkplus condition.
> 	* cp/cp-objcp-common.c (ARRAY_NOTATION_REF): Delete.
> 	* cp/cp-tree.h (cilkplus_an_triplet_types_ok_p): Delete.
> 	* cp/decl.c (grokfndecl, finish_function): Remove cilkplus condition.
> 	* cp/error.c (dump_decl, dump_expr): Remove ARRAY_NOTATION_REF condition.
> 	* cp/lambda.c (cp-cilkplus.h): Remove.
> 	* cp/parser.c (cp_parser_cilk_simd, cp_parser_cilk_for,
> 	cp_parser_cilk_simd_vectorlength): Delete.
> 	(cp_debug_parser, cp_parser_ctor_initializer_opt_and_function_body,
> 	cp_parser_postfix_expression, cp_parser_postfix_open_square_expression,
> 	cp_parser_statement, cp_parser_jump_statement,
> cp_parser_direct_declarator,
> 	cp_parser_late_return_type_opt, cp_parser_gnu_attribute_list,
> 	cp_parser_omp_clause_name, cp_parser_omp_clause_aligned,
> 	cp_parser_omp_clause_linear, cp_parser_omp_all_clauses,
> cp_parser_omp_flush,
> 	cp_parser_omp_for_cond, cp_parser_omp_for_incr,
> cp_parser_omp_for_loop_init,
> 	cp_parser_omp_for_loop, cp_parser_omp_declare_simd): Remove cilkplus
> condtions.
> 	(CILK_SIMD_FN_CLAUSE_MASK, cp_parser_late_parsing_cilk_simd_fn_info,
> 	cp_parser_cilk_grainsize): Remove.
> 	(cp_parser_pragma, c_parse_file): Remove cilkplus condition.
> 	(cp_parser_cilk_simd_vectorlength, cp_parser_cilk_simd_linear,
> 	cp_parser_cilk_simd_clause_name, cp_parser_cilk_simd_all_clauses,
> 	cp_parser_cilk_simd, cp_parser_cilk_for): Remove.
> 	* cp/parser.h (IN_CILK_SIMD_FOR, IN_CILK_SPAWN): Remove.
> 	* cp/pt.c (tsubst_attribute, tsubst_expr, tsubst_copy_and_build): Remove
> 	cilkplus condition.
> 	* cp/semantics.c (finish_goto_stmt, begin_while_stmt, finish_do_body,
> 	finish_init_stmt, finish_switch_cond, simplify_aggr_init_expr,
> 	finish_omp_clauses, finish_omp_clauses, finish_omp_for): Remove cilkplus
> 	condition.
> 	* cp/tree.c (lvalue_kind): Remove ARRAY_NOTATION_REF conditon.
> 	* cp/typeck.c (cp_build_array_ref, cp_build_compound_expr,
> 	check_return_expr): Remove cilkplus conditions.
> 	* cppbuiltin.c: Ditto.
> 	* doc/extend.texi: Remove cilkplus doc.
> 	* doc/generic.texi: Ditto.
> 	* doc/invoke.texi: Ditto.
> 	* doc/passes.texi: Ditto.
> 	* gcc.c (fcilkplus): Remove.
> 	* gengtype.c (cilk.h): Remove.
> 	* gimple-pretty-print.c (dump_gimple_omp_for): Remove cilkplus conditions.
> 	* gimple.h (GF_OMP_FOR_KIND_CILKFOR, GF_OMP_FOR_KIND_CILKSIMD): Remove.
> 	* gimplify.c (gimplify_return_expr, maybe_fold_stmt, gimplify_call_expr,
> 	is_gimple_stmt, gimplify_modify_expr, gimplify_scan_omp_clauses,
> 	gimplify_adjust_omp_clauses, gimplify_omp_for, gimplify_expr): Remove
> 	cilkplus conditions.
> 	* ipa-fnsummary.c (ipa_dump_fn_summary, compute_fn_summary,
> 	inline_read_section): Ditto.
> 	* ipa-inline-analysis.c (cilk.h): Remove.
> 	* ira.c (ira_setup_eliminable_regset): Remove cilkplus condition.
> 	* lto-wrapper.c (merge_and_complain, append_compiler_options,
> 	append_linker_options): Remove condition for fcilkplus.
> 	* lto/lto-lang.c (cilk.h): Remove.
> 	(lto_init): Remove condition for fcilkplus.
> 	* omp-expand.c (expand_cilk_for_call): Delete.
> 	(expand_omp_taskreg, expand_omp_for_static_chunk,
> 	expand_omp_for): Remove cilkplus
> 	conditions.
> 	(expand_cilk_for): Delete.
> 	* omp-general.c (omp_extract_for_data): Remove cilkplus condition.
> 	* omp-low.c (scan_sharing_clauses, create_omp_child_function,
> 	execute_lower_omp, diagnose_sb_0): Ditto.
> 	* omp-simd-clone.c (simd_clone_clauses_extract): Ditto.
> 	* tree-core.h
> 	* tree-nested.c
> 	* tree-pretty-print.c
> 	* tree.c
> 	* tree.def
> 	* tree.h
>
> gcc/testsuite/
> 	* c-c++-common/attr-simd-3.c: Delete.
> 	* c-c++-common/cilk-plus/AN/an-if.c: Delete.
> 	* c-c++-common/cilk-plus/AN/array_test1.c: Delete.
> 	* c-c++-common/cilk-plus/AN/array_test2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/array_test_ND.c: Delete.
> 	* c-c++-common/cilk-plus/AN/builtin_fn_custom.c: Delete.
> 	* c-c++-common/cilk-plus/AN/builtin_fn_mutating.c: Delete.
> 	* c-c++-common/cilk-plus/AN/builtin_func_double.c: Delete.
> 	* c-c++-common/cilk-plus/AN/builtin_func_double2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/comma_exp.c: Delete.
> 	* c-c++-common/cilk-plus/AN/conditional.c: Delete.
> 	* c-c++-common/cilk-plus/AN/decl-ptr-colon.c: Delete.
> 	* c-c++-common/cilk-plus/AN/dimensionless-arrays.c: Delete.
> 	* c-c++-common/cilk-plus/AN/exec-once.c: Delete.
> 	* c-c++-common/cilk-plus/AN/exec-once2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/fn_ptr-2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/fn_ptr.c: Delete.
> 	* c-c++-common/cilk-plus/AN/fp_triplet_values.c: Delete.
> 	* c-c++-common/cilk-plus/AN/gather-scatter-errors.c: Delete.
> 	* c-c++-common/cilk-plus/AN/gather_scatter.c: Delete.
> 	* c-c++-common/cilk-plus/AN/if_test.c: Delete.
> 	* c-c++-common/cilk-plus/AN/if_test_errors.c: Delete.
> 	* c-c++-common/cilk-plus/AN/misc.c: Delete.
> 	* c-c++-common/cilk-plus/AN/n-ptr-test.c: Delete.
> 	* c-c++-common/cilk-plus/AN/parser_errors.c: Delete.
> 	* c-c++-common/cilk-plus/AN/parser_errors2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/parser_errors3.c: Delete.
> 	* c-c++-common/cilk-plus/AN/parser_errors4.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57457-2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57457.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57490.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57541-2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57541.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57577.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr58942.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr61191.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr61455-2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr61455.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr61962.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr61963.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr62008.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr63884.c: Delete.
> 	* c-c++-common/cilk-plus/AN/rank_mismatch.c: Delete.
> 	* c-c++-common/cilk-plus/AN/rank_mismatch2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/rank_mismatch3.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_implicit.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_implicit2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_implicit_ex.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_reduce_ind_same_value.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_reduce_max_min_ind.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_reduce_return.c: Delete.
> 	* c-c++-common/cilk-plus/AN/side-effects-1.c: Delete.
> 	* c-c++-common/cilk-plus/AN/test_builtin_return.c: Delete.
> 	* c-c++-common/cilk-plus/AN/test_sec_limits.c: Delete.
> 	* c-c++-common/cilk-plus/AN/tst_lngth.c: Delete.
> 	* c-c++-common/cilk-plus/AN/vla.c: Delete.
> 	* c-c++-common/cilk-plus/CK/Wparentheses-1.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk-for-2.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk-for-3.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk-fors.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk_for_errors.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk_for_grain.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk_for_grain_errors.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk_for_ptr_iter.c: Delete.
> 	* c-c++-common/cilk-plus/CK/compound_cilk_spawn.c: Delete.
> 	* c-c++-common/cilk-plus/CK/concec_cilk_spawn.c: Delete.
> 	* c-c++-common/cilk-plus/CK/errors.c: Delete.
> 	* c-c++-common/cilk-plus/CK/fib.c: Delete.
> 	* c-c++-common/cilk-plus/CK/fib_init_expr_xy.c: Delete.
> 	* c-c++-common/cilk-plus/CK/fib_no_return.c: Delete.
> 	* c-c++-common/cilk-plus/CK/fib_no_sync.c: Delete.
> 	* c-c++-common/cilk-plus/CK/invalid_spawns.c: Delete.
> 	* c-c++-common/cilk-plus/CK/invalid_sync.c: Delete.c
> 	* c-c++-common/cilk-plus/CK/nested_cilk_for.c: Delete.
> 	* c-c++-common/cilk-plus/CK/no_args_error.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr59631.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr60197-2.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr60197.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr60469.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr60586.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr63307.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr69826-1.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr69826-2.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr79428-4.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr79428-7.c: Delete.
> 	* c-c++-common/cilk-plus/CK/spawn_in_return.c: Delete.
> 	* c-c++-common/cilk-plus/CK/spawnee_inline.c: Delete.
> 	* c-c++-common/cilk-plus/CK/spawner_inline.c: Delete.
> 	* c-c++-common/cilk-plus/CK/spawning_arg.c: Delete.
> 	* c-c++-common/cilk-plus/CK/steal_check.c: Delete.
> 	* c-c++-common/cilk-plus/CK/sync_wo_spawn.c: Delete.
> 	* c-c++-common/cilk-plus/CK/test__cilk.c: Delete.
> 	* c-c++-common/cilk-plus/CK/varargs_test.c: Delete.
> 	* c-c++-common/cilk-plus/PS/Wparentheses-1.c: Delete.
> 	* c-c++-common/cilk-plus/PS/body.c: Delete.
> 	* c-c++-common/cilk-plus/PS/clauses1.c: Delete.
> 	* c-c++-common/cilk-plus/PS/clauses2.c: Delete.
> 	* c-c++-common/cilk-plus/PS/clauses3.c: Delete.
> 	* c-c++-common/cilk-plus/PS/clauses4.c: Delete.
> 	* c-c++-common/cilk-plus/PS/for1.c: Delete.
> 	* c-c++-common/cilk-plus/PS/for2.c: Delete.
> 	* c-c++-common/cilk-plus/PS/for3.c: Delete.
> 	* c-c++-common/cilk-plus/PS/pr69363.c: Delete.
> 	* c-c++-common/cilk-plus/PS/reduction-1.c: Delete.
> 	* c-c++-common/cilk-plus/PS/reduction-2.c: Delete.
> 	* c-c++-common/cilk-plus/PS/reduction-3.c: Delete.
> 	* c-c++-common/cilk-plus/PS/run-1.c: Delete.
> 	* c-c++-common/cilk-plus/PS/safelen.c: Delete.
> 	* c-c++-common/cilk-plus/PS/vectorlength-2.c: Delete.
> 	* c-c++-common/cilk-plus/PS/vectorlength-3.c: Delete.
> 	* c-c++-common/cilk-plus/PS/vectorlength.c: Delete.
> 	* c-c++-common/cilk-plus/SE/ef_error.c: Delete.
> 	* c-c++-common/cilk-plus/SE/ef_error2.c: Delete.
> 	* c-c++-common/cilk-plus/SE/ef_error3.c: Delete.
> 	* c-c++-common/cilk-plus/SE/ef_test.c: Delete.
> 	* c-c++-common/cilk-plus/SE/ef_test2.c: Delete.
> 	* c-c++-common/cilk-plus/SE/vlength_errors.c: Delete.
> 	* g++.dg/cilk-plus/AN/array_function.c: Delete.c
> 	* g++.dg/cilk-plus/AN/array_test1_tplt.c: Delete.c
> 	* g++.dg/cilk-plus/AN/array_test2_tplt.c: Delete.c
> 	* g++.dg/cilk-plus/AN/array_test_ND_tplt.c: Delete.c
> 	* g++.dg/cilk-plus/AN/braced_list.c: Delete.c
> 	* g++.dg/cilk-plus/AN/builtin_fn_custom_tplt.c: Delete.c
> 	* g++.dg/cilk-plus/AN/builtin_fn_mutating_tplt.c: Delete.c
> 	* g++.dg/cilk-plus/AN/fp_triplet_values_tplt.c: Delete.
> 	* g++.dg/cilk-plus/AN/postincr_test.c: Delete.c
> 	* g++.dg/cilk-plus/AN/preincr_test.c: Delete.c
> 	* g++.dg/cilk-plus/CK/catch_exc.c: Delete.c
> 	* g++.dg/cilk-plus/CK/cf3.c: Delete.c
> 	* g++.dg/cilk-plus/CK/cilk-for-tplt.c: Delete.c
> 	* g++.dg/cilk-plus/CK/const_spawn.c: Delete.c
> 	* g++.dg/cilk-plus/CK/fib-opr-overload.c: Delete.c
> 	* g++.dg/cilk-plus/CK/fib-tplt.c: Delete.c
> 	* g++.dg/cilk-plus/CK/for1.c: Delete.c
> 	* g++.dg/cilk-plus/CK/lambda_spawns.c: Delete.c
> 	* g++.dg/cilk-plus/CK/lambda_spawns_tplt.c: Delete.c
> 	* g++.dg/cilk-plus/CK/pr60586.c: Delete.c
> 	* g++.dg/cilk-plus/CK/pr66326.c: Delete.c
> 	* g++.dg/cilk-plus/CK/pr68001.c: Delete.c
> 	* g++.dg/cilk-plus/CK/pr68997.c: Delete.c
> 	* g++.dg/cilk-plus/CK/pr69024.c: Delete.c
> 	* g++.dg/cilk-plus/CK/pr69048.c: Delete.c
> 	* g++.dg/cilk-plus/CK/pr69267.c: Delete.c
> 	* g++.dg/cilk-plus/CK/pr80038.c: Delete.c
> 	* g++.dg/cilk-plus/CK/stl_iter.c: Delete.c
> 	* g++.dg/cilk-plus/CK/stl_rev_iter.c: Delete.c
> 	* g++.dg/cilk-plus/CK/stl_test.c: Delete.c
> 	* g++.dg/cilk-plus/cilk-plus.exp
> 	* g++.dg/cilk-plus/ef_test.C: Delete.
> 	* g++.dg/cilk-plus/for.C: Delete.
> 	* g++.dg/cilk-plus/for2.C: Delete.
> 	* g++.dg/cilk-plus/for3.C: Delete.
> 	* g++.dg/cilk-plus/for4.C: Delete.
> 	* g++.dg/cilk-plus/pr60967.C: Delete.
> 	* g++.dg/cilk-plus/pr69028.C: Delete.
> 	* g++.dg/cilk-plus/pr70565.C: Delete.
> 	* g++.dg/pr57662.C: Delete.
> 	* gcc.dg/cilk-plus/cilk-plus.exp
> 	* gcc.dg/cilk-plus/for1.c: Delete.
> 	* gcc.dg/cilk-plus/for2.c: Delete.
> 	* gcc.dg/cilk-plus/jump-openmp.c: Delete.
> 	* gcc.dg/cilk-plus/jump.c: Delete.
> 	* gcc.dg/cilk-plus/pr69798-1.c: Delete.
> 	* gcc.dg/cilk-plus/pr69798-2.c: Delete.
> 	* gcc.dg/cilk-plus/pr78306.c: Delete.
> 	* gcc.dg/cilk-plus/pr79116.c: Delete.
> 	* gcc.dg/graphite/id-28.c: Delete.
> 	* lib/cilk-plus-dg.exp: Delete.
> 	* lib/target-supports.exp (cilkplus_runtime): Delete.
>
> libcilkrts: Delete
>
> Thanks,
> Julia
>
>

^ permalink raw reply	[flat|nested] 26+ messages in thread

* RE: [patch] remove cilk-plus
  2017-11-16 16:21 ` Jakub Jelinek
@ 2017-11-16 17:06   ` Koval, Julia
  2017-11-16 23:51     ` Joseph Myers
  2017-11-21  7:46     ` Jeff Law
  0 siblings, 2 replies; 26+ messages in thread
From: Koval, Julia @ 2017-11-16 17:06 UTC (permalink / raw)
  To: Jakub Jelinek; +Cc: GCC Patches, law

[-- Attachment #1: Type: text/plain, Size: 20397 bytes --]

Thanks for your comments, fixed it.

2017-11-16  Julia Koval  <julia.koval@intel.com>
	        Sebastian Peryt  <sebastian.peryt@intel.com>

	* Makefile.def (target_modules): Remove libcilkrts.
	* Makefile.in: Ditto.
	* configure: Ditto.
	* configure.ac: Ditto.

contrib/
	* contrib/gcc_update: Ditto.

gcc/
	* Makefile.in (cilkplus.def, cilk-builtins.def, c-family/cilk.o, 
	c-family/c-cilkplus.o, c-family/array-notation-common.o,
	cilk-common.o, cilk.h, cilk-common.c): Remove.
	* builtin-types.def
	(BT_FN_INT_PTR_PTR_PTR_FTYPE_BT_INT_BT_PTR_BT_PTR_BT_PTR): Remove.
	* builtins.c (is_builtin_name): Remove cilkplus condition.
	(BUILT_IN_CILK_DETACH, BUILT_IN_CILK_POP_FRAME): Remove.
	* builtins.def (DEF_CILK_BUILTIN_STUB, DEF_CILKPLUS_BUILTIN,
	cilk-builtins.def, cilkplus.def): Remove.
	* cif-code.def (CILK_SPAWN): Remove.
	* cilk-builtins.def: Delete.
	* cilk-common.c: Ditto.
	* cilk.h: Ditto.
	* cilkplus.def: Ditto.
	* config/darwin.h (fcilkplus): Delete.
	* cppbuiltin.c: Ditto.
	* doc/extend.texi: Remove cilkplus doc.
	* doc/generic.texi: Ditto.
	* doc/invoke.texi: Ditto.
	* doc/passes.texi: Ditto.
	* gcc.c (fcilkplus): Remove.
	* gengtype.c (cilk.h): Remove.
	* gimple-pretty-print.c (dump_gimple_omp_for): Remove cilkplus support.
	* gimple.h (GF_OMP_FOR_KIND_CILKFOR, GF_OMP_FOR_KIND_CILKSIMD): Remove.
	* gimplify.c (gimplify_return_expr, maybe_fold_stmt, gimplify_call_expr,
	is_gimple_stmt, gimplify_modify_expr, gimplify_scan_omp_clauses,
	gimplify_adjust_omp_clauses, gimplify_omp_for, gimplify_expr): Remove
	cilkplus conditions.
	* ipa-fnsummary.c (ipa_dump_fn_summary, compute_fn_summary,
	inline_read_section): Ditto.
	* ipa-inline-analysis.c (cilk.h): Remove.
	* ira.c (ira_setup_eliminable_regset): Remove cilkplus support.
	* lto-wrapper.c (merge_and_complain, append_compiler_options,
	append_linker_options): Remove condition for fcilkplus.
	* lto/lto-lang.c (cilk.h): Remove.
	(lto_init): Remove condition for fcilkplus.
	* omp-expand.c (expand_cilk_for_call): Delete.
	(expand_omp_taskreg, expand_omp_for_static_chunk,
	expand_omp_for): Remove cilkplus
	conditions.
	(expand_cilk_for): Delete.
	* omp-general.c (omp_extract_for_data): Remove cilkplus support.
	* omp-low.c (scan_sharing_clauses, create_omp_child_function,
	execute_lower_omp, diagnose_sb_0): Ditto.
	* omp-simd-clone.c (simd_clone_clauses_extract): Ditto.
	* tree-core.h (OMP_CLAUSE__CILK_FOR_COUNT_): Delete.
	* tree-nested.c: Ditto.
	* tree-pretty-print.c (dump_omp_clause): Remove cilkplus support.
	(dump_generic_node): Ditto.
	* tree.c (OMP_CLAUSE__CILK_FOR_COUNT_): Delete.
	* tree.def (cilk_simd, cilk_for, cilk_spawn_stmt, cilk_sync_stmt): Delete.
	* tree.h (CILK_SPAWN_FN, EXPR_CILK_SPAWN): Delete.

gcc/c-family/
	* array-notation-common.c: Delete.
	* c-cilkplus.c: Ditto.
	* c-common.c (_Cilk_spawn, _Cilk_sync, _Cilk_for): Remove.
	* c-common.def (ARRAY_NOTATION_REF): Remove.
	* c-common.h (RID_CILK_SPAWN, build_array_notation_expr,
	build_array_notation_ref, C_ORT_CILK, c_check_cilk_loop,
	c_validate_cilk_plus_loop, cilkplus_an_parts, cilk_ignorable_spawn_rhs_op,
	cilk_recognize_spawn): Remove.
	* c-gimplify.c (CILK_SPAWN_STMT): Remove.
	* c-omp.c: Remove CILK_SIMD check.
	* c-pragma.c: Ditto.
	* c-pragma.h: Remove CILK related pragmas.
	* c-pretty-print.c (c_pretty_printer::postfix_expression): Remove
	ARRAY_NOTATION_REF condition.
	(c_pretty_printer::expression): Ditto.
	* c.opt (fcilkplus): Remove.
	* cilk.c: Delete.

gcc/c/
	* Make-lang.in (c/c-array-notation.o): Remove.
	* c-array-notation.c: Delete.
	* c-decl.c: Remove cilkplus condition.
	* c-parser.c (c_parser_cilk_simd, c_parser_cilk_for,
	c_parser_cilk_verify_simd, c_parser_array_notation,
	c_parser_cilk_clause_vectorlength, c_parser_cilk_grainsize,
	c_parser_cilk_simd_fn_vector_attrs,
	c_finish_cilk_simd_fn_tokens): Delete.
	(c_parser_declaration_or_fndef): Remove cilkplus condition.
	(c_parser_direct_declarator_inner): Ditto.
	(CILK_SIMD_FN_CLAUSE_MASK): Delete.
	(c_parser_attributes, c_parser_compound_statement,
	c_parser_statement_after_labels, c_parser_if_statement,
	c_parser_switch_statement, c_parser_while_statement,
	c_parser_do_statement, c_parser_for_statement,
	c_parser_unary_expression, c_parser_postfix_expression,
	c_parser_postfix_expression_after_primary,
	c_parser_pragma, c_parser_omp_clause_name, c_parser_omp_all_clauses,
	c_parser_omp_for_loop, c_finish_omp_declare_simd): Remove cilkplus support.
	* c-typeck.c (build_array_ref, build_function_call_vec, convert_arguments,
	lvalue_p, build_compound_expr, c_finish_return, c_finish_if_stmt,
	c_finish_loop, build_binary_op): Remove cilkplus support.
	
gcc/cp/
	* Make-lang.in (cp/cp-array-notation.o, cp/cp-cilkplus.o): Delete.
	* call.c (convert_for_arg_passing, build_cxx_call): Remove cilkplus.
	* constexpr.c (potential_constant_expression_1): Ditto.
	* cp-array-notation.c: Delete.
	* cp-cilkplus.c: Ditto.
	* cp-cilkplus.h: Ditto.
	* cp-gimplify.c (cp_gimplify_expr, cp_fold_r, cp_genericize): Remove
	cilkplus condition.
	* cp-objcp-common.c (ARRAY_NOTATION_REF): Delete.
	* cp-tree.h (cilkplus_an_triplet_types_ok_p): Delete.
	* decl.c (grokfndecl, finish_function): Remove cilkplus condition.
	* error.c (dump_decl, dump_expr): Remove ARRAY_NOTATION_REF condition.
	* lambda.c (cp-cilkplus.h): Remove.
	* parser.c (cp_parser_cilk_simd, cp_parser_cilk_for,
	cp_parser_cilk_simd_vectorlength): Delete.
	(cp_debug_parser, cp_parser_ctor_initializer_opt_and_function_body,
	cp_parser_postfix_expression, cp_parser_postfix_open_square_expression,
	cp_parser_statement, cp_parser_jump_statement, cp_parser_direct_declarator,
	cp_parser_late_return_type_opt, cp_parser_gnu_attribute_list,
	cp_parser_omp_clause_name, cp_parser_omp_clause_aligned,
	cp_parser_omp_clause_linear, cp_parser_omp_all_clauses, cp_parser_omp_flush,
	cp_parser_omp_for_cond, cp_parser_omp_for_incr, cp_parser_omp_for_loop_init,
	cp_parser_omp_for_loop,
	cp_parser_omp_declare_simd): Remove cilkplus support.
	(CILK_SIMD_FN_CLAUSE_MASK, cp_parser_late_parsing_cilk_simd_fn_info,
	cp_parser_cilk_grainsize): Remove.
	(cp_parser_pragma, c_parse_file): Remove cilkplus support.
	(cp_parser_cilk_simd_vectorlength, cp_parser_cilk_simd_linear,
	cp_parser_cilk_simd_clause_name, cp_parser_cilk_simd_all_clauses,
	cp_parser_cilk_simd, cp_parser_cilk_for): Remove.
	* parser.h (IN_CILK_SIMD_FOR, IN_CILK_SPAWN): Remove.
	* pt.c (tsubst_attribute, tsubst_expr, tsubst_copy_and_build): Remove
	cilkplus support.
	* semantics.c (finish_goto_stmt, begin_while_stmt, finish_do_body,
	finish_init_stmt, finish_switch_cond, simplify_aggr_init_expr,
	finish_omp_clauses, finish_omp_clauses, finish_omp_for): Remove cilkplus
	support.
	* tree.c (lvalue_kind): Remove ARRAY_NOTATION_REF conditon.
	* typeck.c (cp_build_array_ref, cp_build_compound_expr,
	check_return_expr): Remove cilkplus support.

gcc/testsuite/
	* c-c++-common/attr-simd-3.c: Delete.
	* c-c++-common/cilk-plus/AN/an-if.c: Delete.
	* c-c++-common/cilk-plus/AN/array_test1.c: Delete.
	* c-c++-common/cilk-plus/AN/array_test2.c: Delete.
	* c-c++-common/cilk-plus/AN/array_test_ND.c: Delete.
	* c-c++-common/cilk-plus/AN/builtin_fn_custom.c: Delete.
	* c-c++-common/cilk-plus/AN/builtin_fn_mutating.c: Delete.
	* c-c++-common/cilk-plus/AN/builtin_func_double.c: Delete.
	* c-c++-common/cilk-plus/AN/builtin_func_double2.c: Delete.
	* c-c++-common/cilk-plus/AN/comma_exp.c: Delete.
	* c-c++-common/cilk-plus/AN/conditional.c: Delete.
	* c-c++-common/cilk-plus/AN/decl-ptr-colon.c: Delete.
	* c-c++-common/cilk-plus/AN/dimensionless-arrays.c: Delete.
	* c-c++-common/cilk-plus/AN/exec-once.c: Delete.
	* c-c++-common/cilk-plus/AN/exec-once2.c: Delete.
	* c-c++-common/cilk-plus/AN/fn_ptr-2.c: Delete.
	* c-c++-common/cilk-plus/AN/fn_ptr.c: Delete.
	* c-c++-common/cilk-plus/AN/fp_triplet_values.c: Delete.
	* c-c++-common/cilk-plus/AN/gather-scatter-errors.c: Delete.
	* c-c++-common/cilk-plus/AN/gather_scatter.c: Delete.
	* c-c++-common/cilk-plus/AN/if_test.c: Delete.
	* c-c++-common/cilk-plus/AN/if_test_errors.c: Delete.
	* c-c++-common/cilk-plus/AN/misc.c: Delete.
	* c-c++-common/cilk-plus/AN/n-ptr-test.c: Delete.
	* c-c++-common/cilk-plus/AN/parser_errors.c: Delete.
	* c-c++-common/cilk-plus/AN/parser_errors2.c: Delete.
	* c-c++-common/cilk-plus/AN/parser_errors3.c: Delete.
	* c-c++-common/cilk-plus/AN/parser_errors4.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57457-2.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57457.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57490.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57541-2.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57541.c: Delete.
	* c-c++-common/cilk-plus/AN/pr57577.c: Delete.
	* c-c++-common/cilk-plus/AN/pr58942.c: Delete.
	* c-c++-common/cilk-plus/AN/pr61191.c: Delete.
	* c-c++-common/cilk-plus/AN/pr61455-2.c: Delete.
	* c-c++-common/cilk-plus/AN/pr61455.c: Delete.
	* c-c++-common/cilk-plus/AN/pr61962.c: Delete.
	* c-c++-common/cilk-plus/AN/pr61963.c: Delete.
	* c-c++-common/cilk-plus/AN/pr62008.c: Delete.
	* c-c++-common/cilk-plus/AN/pr63884.c: Delete.
	* c-c++-common/cilk-plus/AN/rank_mismatch.c: Delete.
	* c-c++-common/cilk-plus/AN/rank_mismatch2.c: Delete.
	* c-c++-common/cilk-plus/AN/rank_mismatch3.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_implicit.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_implicit2.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_implicit_ex.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_reduce_ind_same_value.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_reduce_max_min_ind.c: Delete.
	* c-c++-common/cilk-plus/AN/sec_reduce_return.c: Delete.
	* c-c++-common/cilk-plus/AN/side-effects-1.c: Delete.
	* c-c++-common/cilk-plus/AN/test_builtin_return.c: Delete.
	* c-c++-common/cilk-plus/AN/test_sec_limits.c: Delete.
	* c-c++-common/cilk-plus/AN/tst_lngth.c: Delete.
	* c-c++-common/cilk-plus/AN/vla.c: Delete.
	* c-c++-common/cilk-plus/CK/Wparentheses-1.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk-for-2.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk-for-3.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk-fors.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk_for_errors.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk_for_grain.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk_for_grain_errors.c: Delete.
	* c-c++-common/cilk-plus/CK/cilk_for_ptr_iter.c: Delete.
	* c-c++-common/cilk-plus/CK/compound_cilk_spawn.c: Delete.
	* c-c++-common/cilk-plus/CK/concec_cilk_spawn.c: Delete.
	* c-c++-common/cilk-plus/CK/errors.c: Delete.
	* c-c++-common/cilk-plus/CK/fib.c: Delete.
	* c-c++-common/cilk-plus/CK/fib_init_expr_xy.c: Delete.
	* c-c++-common/cilk-plus/CK/fib_no_return.c: Delete.
	* c-c++-common/cilk-plus/CK/fib_no_sync.c: Delete.
	* c-c++-common/cilk-plus/CK/invalid_spawns.c: Delete.
	* c-c++-common/cilk-plus/CK/invalid_sync.c: Delete.c
	* c-c++-common/cilk-plus/CK/nested_cilk_for.c: Delete.
	* c-c++-common/cilk-plus/CK/no_args_error.c: Delete.
	* c-c++-common/cilk-plus/CK/pr59631.c: Delete.
	* c-c++-common/cilk-plus/CK/pr60197-2.c: Delete.
	* c-c++-common/cilk-plus/CK/pr60197.c: Delete.
	* c-c++-common/cilk-plus/CK/pr60469.c: Delete.
	* c-c++-common/cilk-plus/CK/pr60586.c: Delete.
	* c-c++-common/cilk-plus/CK/pr63307.c: Delete.
	* c-c++-common/cilk-plus/CK/pr69826-1.c: Delete.
	* c-c++-common/cilk-plus/CK/pr69826-2.c: Delete.
	* c-c++-common/cilk-plus/CK/pr79428-4.c: Delete.
	* c-c++-common/cilk-plus/CK/pr79428-7.c: Delete.
	* c-c++-common/cilk-plus/CK/spawn_in_return.c: Delete.
	* c-c++-common/cilk-plus/CK/spawnee_inline.c: Delete.
	* c-c++-common/cilk-plus/CK/spawner_inline.c: Delete.
	* c-c++-common/cilk-plus/CK/spawning_arg.c: Delete.
	* c-c++-common/cilk-plus/CK/steal_check.c: Delete.
	* c-c++-common/cilk-plus/CK/sync_wo_spawn.c: Delete.
	* c-c++-common/cilk-plus/CK/test__cilk.c: Delete.
	* c-c++-common/cilk-plus/CK/varargs_test.c: Delete.
	* c-c++-common/cilk-plus/PS/Wparentheses-1.c: Delete.
	* c-c++-common/cilk-plus/PS/body.c: Delete.
	* c-c++-common/cilk-plus/PS/clauses1.c: Delete.
	* c-c++-common/cilk-plus/PS/clauses2.c: Delete.
	* c-c++-common/cilk-plus/PS/clauses3.c: Delete.
	* c-c++-common/cilk-plus/PS/clauses4.c: Delete.
	* c-c++-common/cilk-plus/PS/for1.c: Delete.
	* c-c++-common/cilk-plus/PS/for2.c: Delete.
	* c-c++-common/cilk-plus/PS/for3.c: Delete.
	* c-c++-common/cilk-plus/PS/pr69363.c: Delete.
	* c-c++-common/cilk-plus/PS/reduction-1.c: Delete.
	* c-c++-common/cilk-plus/PS/reduction-2.c: Delete.
	* c-c++-common/cilk-plus/PS/reduction-3.c: Delete.
	* c-c++-common/cilk-plus/PS/run-1.c: Delete.
	* c-c++-common/cilk-plus/PS/safelen.c: Delete.
	* c-c++-common/cilk-plus/PS/vectorlength-2.c: Delete.
	* c-c++-common/cilk-plus/PS/vectorlength-3.c: Delete.
	* c-c++-common/cilk-plus/PS/vectorlength.c: Delete.
	* c-c++-common/cilk-plus/SE/ef_error.c: Delete.
	* c-c++-common/cilk-plus/SE/ef_error2.c: Delete.
	* c-c++-common/cilk-plus/SE/ef_error3.c: Delete.
	* c-c++-common/cilk-plus/SE/ef_test.c: Delete.
	* c-c++-common/cilk-plus/SE/ef_test2.c: Delete.
	* c-c++-common/cilk-plus/SE/vlength_errors.c: Delete.
	* g++.dg/cilk-plus/AN/array_function.c: Delete.
	* g++.dg/cilk-plus/AN/array_test1_tplt.c: Delete.
	* g++.dg/cilk-plus/AN/array_test2_tplt.c: Delete.
	* g++.dg/cilk-plus/AN/array_test_ND_tplt.c: Delete.
	* g++.dg/cilk-plus/AN/braced_list.c: Delete.
	* g++.dg/cilk-plus/AN/builtin_fn_custom_tplt.c: Delete.
	* g++.dg/cilk-plus/AN/builtin_fn_mutating_tplt.c: Delete.
	* g++.dg/cilk-plus/AN/fp_triplet_values_tplt.c: Delete.
	* g++.dg/cilk-plus/AN/postincr_test.c: Delete.
	* g++.dg/cilk-plus/AN/preincr_test.c: Delete.
	* g++.dg/cilk-plus/CK/catch_exc.c: Delete.
	* g++.dg/cilk-plus/CK/cf3.c: Delete.
	* g++.dg/cilk-plus/CK/cilk-for-tplt.c: Delete.
	* g++.dg/cilk-plus/CK/const_spawn.c: Delete.
	* g++.dg/cilk-plus/CK/fib-opr-overload.c: Delete.
	* g++.dg/cilk-plus/CK/fib-tplt.c: Delete.
	* g++.dg/cilk-plus/CK/for1.c: Delete.
	* g++.dg/cilk-plus/CK/lambda_spawns.c: Delete.
	* g++.dg/cilk-plus/CK/lambda_spawns_tplt.c: Delete.
	* g++.dg/cilk-plus/CK/pr60586.c: Delete.
	* g++.dg/cilk-plus/CK/pr66326.c: Delete.
	* g++.dg/cilk-plus/CK/pr68001.c: Delete.
	* g++.dg/cilk-plus/CK/pr68997.c: Delete.
	* g++.dg/cilk-plus/CK/pr69024.c: Delete.
	* g++.dg/cilk-plus/CK/pr69048.c: Delete.
	* g++.dg/cilk-plus/CK/pr69267.c: Delete.
	* g++.dg/cilk-plus/CK/pr80038.c: Delete.
	* g++.dg/cilk-plus/CK/stl_iter.c: Delete.
	* g++.dg/cilk-plus/CK/stl_rev_iter.c: Delete.
	* g++.dg/cilk-plus/CK/stl_test.c: Delete.
	* g++.dg/cilk-plus/cilk-plus.exp
	* g++.dg/cilk-plus/ef_test.C: Delete.
	* g++.dg/cilk-plus/for.C: Delete.
	* g++.dg/cilk-plus/for2.C: Delete.
	* g++.dg/cilk-plus/for3.C: Delete.
	* g++.dg/cilk-plus/for4.C: Delete.
	* g++.dg/cilk-plus/pr60967.C: Delete.
	* g++.dg/cilk-plus/pr69028.C: Delete.
	* g++.dg/cilk-plus/pr70565.C: Delete.
	* g++.dg/pr57662.C: Delete.
	* gcc.dg/cilk-plus/cilk-plus.exp
	* gcc.dg/cilk-plus/for1.c: Delete.
	* gcc.dg/cilk-plus/for2.c: Delete.
	* gcc.dg/cilk-plus/jump-openmp.c: Delete.
	* gcc.dg/cilk-plus/jump.c: Delete.
	* gcc.dg/cilk-plus/pr69798-1.c: Delete.
	* gcc.dg/cilk-plus/pr69798-2.c: Delete.
	* gcc.dg/cilk-plus/pr78306.c: Delete.
	* gcc.dg/cilk-plus/pr79116.c: Delete.
	* gcc.dg/graphite/id-28.c: Delete.
	* lib/cilk-plus-dg.exp: Delete.
	* lib/target-supports.exp (cilkplus_runtime): Delete.

libcilkrts: Delete

> -----Original Message-----
> From: Jakub Jelinek [mailto:jakub@redhat.com]
> Sent: Thursday, November 16, 2017 4:49 PM
> To: Koval, Julia <julia.koval@intel.com>
> Cc: GCC Patches <gcc-patches@gcc.gnu.org>; law@redhat.com
> Subject: Re: [patch] remove cilk-plus
> 
> On Thu, Nov 16, 2017 at 03:33:40PM +0000, Koval, Julia wrote:
> > // I failed to send patch itself, it is too big even in gzipped form.  What is the
> right way to send such big patches?
> 
> Don't include the libcilkrts subtree in the patch nor /cilk-plus/
> testcases that are going to be removed?
> 
> > Hi, this patch removes cilkplus. Ok for trunk?
> >
> > 2017-11-16  Julia Koval  <julia.koval@intel.com>
> > 	        Sebastian Peryt  <sebastian.peryt@intel.com>
> > gcc/
> > 	* Makefile.def (target_modules): Remove libcilkrts.
> > 	* Makefile.in: Ditto.
> > 	* configure: Ditto.
> > 	* configure.ac: Ditto.
> 
> The ChangeLog needs work, e.g. we have many different ChangeLog files and
> changes should be relative to that.  The above entries are for toplevel.
> 
> > 	* contrib/gcc_update: Ditto.
> 
> This one is for contrib/ChangeLog, so should be without contrib/
> in the entry.
> 
> > 	* Makefile.in (cilkplus.def, cilk-builtins.def, c-family/cilk.o,
> > 	c-family/c-cilkplus.o, c-family/array-notation-common.o,
> > 	cilk-common.o, cilk.h, cilk-common.c): Remove.
> > 	* builtin-types.def
> (BT_FN_INT_PTR_PTR_PTR_FTYPE_BT_INT_BT_PTR_BT_PTR
> > 	_BT_PTR): Remove.
> 
> There should be no linebreaks within one identifier.  So
> 	* builtin-types.def
> 	(BT_FN_INT_PTR_PTR_PTR_FTYPE_BT_INT_BT_PTR_BT_PTR_BT_PTR):
> Remove.
> 
> > 	* c-family/array-notation-common.c: Delete.
> > 	* c-family/c-cilkplus.c: Ditto.
> > 	* c-family/c-common.c (_Cilk_spawn, _Cilk_sync, _Cilk_for): Remove.
> > 	* c-family/c-common.def (ARRAY_NOTATION_REF): Remove.
> > 	* c-family/c-common.h (RID_CILK_SPAWN, build_array_notation_expr,
> > 	build_array_notation_ref, C_ORT_CILK, c_check_cilk_loop,
> > 	c_validate_cilk_plus_loop, cilkplus_an_parts,
> cilk_ignorable_spawn_rhs_op,
> > 	cilk_recognize_spawn): Remove.
> > 	* c-family/c-gimplify.c (CILK_SPAWN_STMT): Remove.
> > 	* c-family/c-omp.c: Remove CILK_SIMD check.
> > 	* c-family/c-pragma.c: Ditto.
> > 	* c-family/c-pragma.h: Remove CILK related pragmas.
> > 	* c-family/c-pretty-print.c (c_pretty_printer::postfix_expression):
> Remove
> > 	ARRAY_NOTATION_REF condition.
> > 	(c_pretty_printer::expression): Ditto.
> > 	* c-family/c.opt (fcilkplus): Remove.
> > 	* c-family/cilk.c: Delete.
> 
> c-family has its own ChangeLog, no c-family/ prefix (similarly for c/, cp/,
> etc.).
> 
> > 	* c/Make-lang.in (c/c-array-notation.o): Remove.
> > 	* c/c-array-notation.c: Delete.
> > 	* c/c-decl.c: Remove cilkplus condition.
> > 	* c/c-parser.c (c_parser_cilk_simd, c_parser_cilk_for,
> > 	c_parser_cilk_verify_simd, c_parser_array_notation,
> > 	c_parser_cilk_clause_vectorlength, c_parser_cilk_grainsize,
>  	c_parser_cilk_simd_fn_vector_attrs, c_finish_cilk_simd_fn_tokens):
> Delete.
> 
> Too long line.
> 
> > 	(c_parser_declaration_or_fndef): Remove cilkplus condition.
> > 	(c_parser_direct_declarator_inner): Ditto.
> > 	(CILK_SIMD_FN_CLAUSE_MASK): Delete.
> > 	(c_parser_attributes): Remove cilk-plus condition.
> > 	(c_parser_compound_statement): Ditto.
> > 	(c_parser_statement_after_labels): Ditto.
> > 	(c_parser_if_statement): Ditto.
> > 	(c_parser_switch_statement): Ditto.
> > 	(c_parser_while_statement): Ditto.
> > 	(c_parser_do_statement): Ditto.
> > 	(c_parser_for_statement): Ditto.
> > 	(c_parser_unary_expression): Ditto.
> > 	(c_parser_postfix_expression): Ditto.
> > 	(c_parser_postfix_expression_after_primary): Ditto.
> > 	(c_parser_pragma): Ditto.
> > 	(c_parser_omp_clause_name): Ditto.
> > 	(c_parser_omp_all_clauses): Ditto.
> > 	(c_parser_omp_for_loop): Ditto.
> > 	(c_finish_omp_declare_simd): Ditto.
> 
> Perhaps you could shorten by writing:
> 	(c_parser_attributes, c_parser_compound_statement,
> 	c_parser_statement_after_labels, c_parser_if_statement,
> 	c_parser_switch_statement, c_parser_while_statement,
> 	c_parser_do_statement, c_parser_for_statement,
> 	c_parser_unary_expression, c_parser_postfix_expression,
> 	c_parser_postfix_expression_after_primary, c_parser_pragma,
> 	c_parser_omp_clause_name, c_parser_omp_all_clauses,
> 	c_parser_omp_for_loop, c_finish_omp_declare_simd):
> 	Remove cilkplus support.
> etc.
>  	* c/c-typeck.c (build_array_ref, build_function_call_vec,
> convert_arguments,
> 
> Too long line (various others).
> 
> > 	* tree-core.h
> > 	* tree-nested.c
> > 	* tree-pretty-print.c
> > 	* tree.c
> > 	* tree.def
> > 	* tree.h
> 
> Description on what changed is missing.
> 
> > 	* g++.dg/cilk-plus/AN/array_function.c: Delete.c
> 
> Delete.c ?  Should be Delete.
> 
> 	Jakub

[-- Attachment #2: cilk-plus.tar.xz --]
[-- Type: application/octet-stream, Size: 81084 bytes --]

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-16 16:34 ` Eric Gallager
@ 2017-11-16 17:59   ` Jeff Law
  0 siblings, 0 replies; 26+ messages in thread
From: Jeff Law @ 2017-11-16 17:59 UTC (permalink / raw)
  To: Eric Gallager, Koval, Julia; +Cc: GCC Patches

On 11/16/2017 09:22 AM, Eric Gallager wrote:
> On 11/16/17, Koval, Julia <julia.koval@intel.com> wrote:
>> // I failed to send patch itself, it is too big even in gzipped form.  What
>> is the right way to send such big patches?
>>
>> Hi, this patch removes cilkplus. Ok for trunk?
> 
> I'm not a reviewer, but just as an onlooker, I'd want to see notes
> about the removal in the caveats section of
> https://gcc.gnu.org/gcc-8/changes.html
Cilk+ was deprecated in gcc-7 and announced as-such.

But I do think a one-liner to the gcc-8 page would be appropriate to
note its removal.

Jeff

^ permalink raw reply	[flat|nested] 26+ messages in thread

* RE: [patch] remove cilk-plus
  2017-11-16 17:06   ` Koval, Julia
@ 2017-11-16 23:51     ` Joseph Myers
  2017-11-21  7:46     ` Jeff Law
  1 sibling, 0 replies; 26+ messages in thread
From: Joseph Myers @ 2017-11-16 23:51 UTC (permalink / raw)
  To: Koval, Julia; +Cc: Jakub Jelinek, GCC Patches, law

There are places in the c-parser.c changes where there is a comment 
referring to array notation and you remove the subsequent Cilk-specific 
code, but not the comment.  In at least one such place, the contents of 
the "else" block that's made unconditional also need to be reindented.

-- 
Joseph S. Myers
joseph@codesourcery.com

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-16 17:06   ` Koval, Julia
  2017-11-16 23:51     ` Joseph Myers
@ 2017-11-21  7:46     ` Jeff Law
  2017-11-22  8:47       ` Koval, Julia
  1 sibling, 1 reply; 26+ messages in thread
From: Jeff Law @ 2017-11-21  7:46 UTC (permalink / raw)
  To: Koval, Julia, Jakub Jelinek; +Cc: GCC Patches

On 11/16/2017 10:02 AM, Koval, Julia wrote:
> Thanks for your comments, fixed it.
> 
> 2017-11-16  Julia Koval  <julia.koval@intel.com>
> 	        Sebastian Peryt  <sebastian.peryt@intel.com>
> 
> 	* Makefile.def (target_modules): Remove libcilkrts.
> 	* Makefile.in: Ditto.
> 	* configure: Ditto.
> 	* configure.ac: Ditto.
> 
> contrib/
> 	* contrib/gcc_update: Ditto.
> 
> gcc/
> 	* Makefile.in (cilkplus.def, cilk-builtins.def, c-family/cilk.o, 
> 	c-family/c-cilkplus.o, c-family/array-notation-common.o,
> 	cilk-common.o, cilk.h, cilk-common.c): Remove.
> 	* builtin-types.def
> 	(BT_FN_INT_PTR_PTR_PTR_FTYPE_BT_INT_BT_PTR_BT_PTR_BT_PTR): Remove.
> 	* builtins.c (is_builtin_name): Remove cilkplus condition.
> 	(BUILT_IN_CILK_DETACH, BUILT_IN_CILK_POP_FRAME): Remove.
> 	* builtins.def (DEF_CILK_BUILTIN_STUB, DEF_CILKPLUS_BUILTIN,
> 	cilk-builtins.def, cilkplus.def): Remove.
> 	* cif-code.def (CILK_SPAWN): Remove.
> 	* cilk-builtins.def: Delete.
> 	* cilk-common.c: Ditto.
> 	* cilk.h: Ditto.
> 	* cilkplus.def: Ditto.
> 	* config/darwin.h (fcilkplus): Delete.
> 	* cppbuiltin.c: Ditto.
> 	* doc/extend.texi: Remove cilkplus doc.
> 	* doc/generic.texi: Ditto.
> 	* doc/invoke.texi: Ditto.
> 	* doc/passes.texi: Ditto.
> 	* gcc.c (fcilkplus): Remove.
> 	* gengtype.c (cilk.h): Remove.
> 	* gimple-pretty-print.c (dump_gimple_omp_for): Remove cilkplus support.
> 	* gimple.h (GF_OMP_FOR_KIND_CILKFOR, GF_OMP_FOR_KIND_CILKSIMD): Remove.
> 	* gimplify.c (gimplify_return_expr, maybe_fold_stmt, gimplify_call_expr,
> 	is_gimple_stmt, gimplify_modify_expr, gimplify_scan_omp_clauses,
> 	gimplify_adjust_omp_clauses, gimplify_omp_for, gimplify_expr): Remove
> 	cilkplus conditions.
> 	* ipa-fnsummary.c (ipa_dump_fn_summary, compute_fn_summary,
> 	inline_read_section): Ditto.
> 	* ipa-inline-analysis.c (cilk.h): Remove.
> 	* ira.c (ira_setup_eliminable_regset): Remove cilkplus support.
> 	* lto-wrapper.c (merge_and_complain, append_compiler_options,
> 	append_linker_options): Remove condition for fcilkplus.
> 	* lto/lto-lang.c (cilk.h): Remove.
> 	(lto_init): Remove condition for fcilkplus.
> 	* omp-expand.c (expand_cilk_for_call): Delete.
> 	(expand_omp_taskreg, expand_omp_for_static_chunk,
> 	expand_omp_for): Remove cilkplus
> 	conditions.
> 	(expand_cilk_for): Delete.
> 	* omp-general.c (omp_extract_for_data): Remove cilkplus support.
> 	* omp-low.c (scan_sharing_clauses, create_omp_child_function,
> 	execute_lower_omp, diagnose_sb_0): Ditto.
> 	* omp-simd-clone.c (simd_clone_clauses_extract): Ditto.
> 	* tree-core.h (OMP_CLAUSE__CILK_FOR_COUNT_): Delete.
> 	* tree-nested.c: Ditto.
> 	* tree-pretty-print.c (dump_omp_clause): Remove cilkplus support.
> 	(dump_generic_node): Ditto.
> 	* tree.c (OMP_CLAUSE__CILK_FOR_COUNT_): Delete.
> 	* tree.def (cilk_simd, cilk_for, cilk_spawn_stmt, cilk_sync_stmt): Delete.
> 	* tree.h (CILK_SPAWN_FN, EXPR_CILK_SPAWN): Delete.
> 
> gcc/c-family/
> 	* array-notation-common.c: Delete.
> 	* c-cilkplus.c: Ditto.
> 	* c-common.c (_Cilk_spawn, _Cilk_sync, _Cilk_for): Remove.
> 	* c-common.def (ARRAY_NOTATION_REF): Remove.
> 	* c-common.h (RID_CILK_SPAWN, build_array_notation_expr,
> 	build_array_notation_ref, C_ORT_CILK, c_check_cilk_loop,
> 	c_validate_cilk_plus_loop, cilkplus_an_parts, cilk_ignorable_spawn_rhs_op,
> 	cilk_recognize_spawn): Remove.
> 	* c-gimplify.c (CILK_SPAWN_STMT): Remove.
> 	* c-omp.c: Remove CILK_SIMD check.
> 	* c-pragma.c: Ditto.
> 	* c-pragma.h: Remove CILK related pragmas.
> 	* c-pretty-print.c (c_pretty_printer::postfix_expression): Remove
> 	ARRAY_NOTATION_REF condition.
> 	(c_pretty_printer::expression): Ditto.
> 	* c.opt (fcilkplus): Remove.
> 	* cilk.c: Delete.
> 
> gcc/c/
> 	* Make-lang.in (c/c-array-notation.o): Remove.
> 	* c-array-notation.c: Delete.
> 	* c-decl.c: Remove cilkplus condition.
> 	* c-parser.c (c_parser_cilk_simd, c_parser_cilk_for,
> 	c_parser_cilk_verify_simd, c_parser_array_notation,
> 	c_parser_cilk_clause_vectorlength, c_parser_cilk_grainsize,
> 	c_parser_cilk_simd_fn_vector_attrs,
> 	c_finish_cilk_simd_fn_tokens): Delete.
> 	(c_parser_declaration_or_fndef): Remove cilkplus condition.
> 	(c_parser_direct_declarator_inner): Ditto.
> 	(CILK_SIMD_FN_CLAUSE_MASK): Delete.
> 	(c_parser_attributes, c_parser_compound_statement,
> 	c_parser_statement_after_labels, c_parser_if_statement,
> 	c_parser_switch_statement, c_parser_while_statement,
> 	c_parser_do_statement, c_parser_for_statement,
> 	c_parser_unary_expression, c_parser_postfix_expression,
> 	c_parser_postfix_expression_after_primary,
> 	c_parser_pragma, c_parser_omp_clause_name, c_parser_omp_all_clauses,
> 	c_parser_omp_for_loop, c_finish_omp_declare_simd): Remove cilkplus support.
> 	* c-typeck.c (build_array_ref, build_function_call_vec, convert_arguments,
> 	lvalue_p, build_compound_expr, c_finish_return, c_finish_if_stmt,
> 	c_finish_loop, build_binary_op): Remove cilkplus support.
> 	
> gcc/cp/
> 	* Make-lang.in (cp/cp-array-notation.o, cp/cp-cilkplus.o): Delete.
> 	* call.c (convert_for_arg_passing, build_cxx_call): Remove cilkplus.
> 	* constexpr.c (potential_constant_expression_1): Ditto.
> 	* cp-array-notation.c: Delete.
> 	* cp-cilkplus.c: Ditto.
> 	* cp-cilkplus.h: Ditto.
> 	* cp-gimplify.c (cp_gimplify_expr, cp_fold_r, cp_genericize): Remove
> 	cilkplus condition.
> 	* cp-objcp-common.c (ARRAY_NOTATION_REF): Delete.
> 	* cp-tree.h (cilkplus_an_triplet_types_ok_p): Delete.
> 	* decl.c (grokfndecl, finish_function): Remove cilkplus condition.
> 	* error.c (dump_decl, dump_expr): Remove ARRAY_NOTATION_REF condition.
> 	* lambda.c (cp-cilkplus.h): Remove.
> 	* parser.c (cp_parser_cilk_simd, cp_parser_cilk_for,
> 	cp_parser_cilk_simd_vectorlength): Delete.
> 	(cp_debug_parser, cp_parser_ctor_initializer_opt_and_function_body,
> 	cp_parser_postfix_expression, cp_parser_postfix_open_square_expression,
> 	cp_parser_statement, cp_parser_jump_statement, cp_parser_direct_declarator,
> 	cp_parser_late_return_type_opt, cp_parser_gnu_attribute_list,
> 	cp_parser_omp_clause_name, cp_parser_omp_clause_aligned,
> 	cp_parser_omp_clause_linear, cp_parser_omp_all_clauses, cp_parser_omp_flush,
> 	cp_parser_omp_for_cond, cp_parser_omp_for_incr, cp_parser_omp_for_loop_init,
> 	cp_parser_omp_for_loop,
> 	cp_parser_omp_declare_simd): Remove cilkplus support.
> 	(CILK_SIMD_FN_CLAUSE_MASK, cp_parser_late_parsing_cilk_simd_fn_info,
> 	cp_parser_cilk_grainsize): Remove.
> 	(cp_parser_pragma, c_parse_file): Remove cilkplus support.
> 	(cp_parser_cilk_simd_vectorlength, cp_parser_cilk_simd_linear,
> 	cp_parser_cilk_simd_clause_name, cp_parser_cilk_simd_all_clauses,
> 	cp_parser_cilk_simd, cp_parser_cilk_for): Remove.
> 	* parser.h (IN_CILK_SIMD_FOR, IN_CILK_SPAWN): Remove.
> 	* pt.c (tsubst_attribute, tsubst_expr, tsubst_copy_and_build): Remove
> 	cilkplus support.
> 	* semantics.c (finish_goto_stmt, begin_while_stmt, finish_do_body,
> 	finish_init_stmt, finish_switch_cond, simplify_aggr_init_expr,
> 	finish_omp_clauses, finish_omp_clauses, finish_omp_for): Remove cilkplus
> 	support.
> 	* tree.c (lvalue_kind): Remove ARRAY_NOTATION_REF conditon.
> 	* typeck.c (cp_build_array_ref, cp_build_compound_expr,
> 	check_return_expr): Remove cilkplus support.
> 
> gcc/testsuite/
> 	* c-c++-common/attr-simd-3.c: Delete.
> 	* c-c++-common/cilk-plus/AN/an-if.c: Delete.
> 	* c-c++-common/cilk-plus/AN/array_test1.c: Delete.
> 	* c-c++-common/cilk-plus/AN/array_test2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/array_test_ND.c: Delete.
> 	* c-c++-common/cilk-plus/AN/builtin_fn_custom.c: Delete.
> 	* c-c++-common/cilk-plus/AN/builtin_fn_mutating.c: Delete.
> 	* c-c++-common/cilk-plus/AN/builtin_func_double.c: Delete.
> 	* c-c++-common/cilk-plus/AN/builtin_func_double2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/comma_exp.c: Delete.
> 	* c-c++-common/cilk-plus/AN/conditional.c: Delete.
> 	* c-c++-common/cilk-plus/AN/decl-ptr-colon.c: Delete.
> 	* c-c++-common/cilk-plus/AN/dimensionless-arrays.c: Delete.
> 	* c-c++-common/cilk-plus/AN/exec-once.c: Delete.
> 	* c-c++-common/cilk-plus/AN/exec-once2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/fn_ptr-2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/fn_ptr.c: Delete.
> 	* c-c++-common/cilk-plus/AN/fp_triplet_values.c: Delete.
> 	* c-c++-common/cilk-plus/AN/gather-scatter-errors.c: Delete.
> 	* c-c++-common/cilk-plus/AN/gather_scatter.c: Delete.
> 	* c-c++-common/cilk-plus/AN/if_test.c: Delete.
> 	* c-c++-common/cilk-plus/AN/if_test_errors.c: Delete.
> 	* c-c++-common/cilk-plus/AN/misc.c: Delete.
> 	* c-c++-common/cilk-plus/AN/n-ptr-test.c: Delete.
> 	* c-c++-common/cilk-plus/AN/parser_errors.c: Delete.
> 	* c-c++-common/cilk-plus/AN/parser_errors2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/parser_errors3.c: Delete.
> 	* c-c++-common/cilk-plus/AN/parser_errors4.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57457-2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57457.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57490.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57541-2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57541.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr57577.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr58942.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr61191.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr61455-2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr61455.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr61962.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr61963.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr62008.c: Delete.
> 	* c-c++-common/cilk-plus/AN/pr63884.c: Delete.
> 	* c-c++-common/cilk-plus/AN/rank_mismatch.c: Delete.
> 	* c-c++-common/cilk-plus/AN/rank_mismatch2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/rank_mismatch3.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_implicit.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_implicit2.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_implicit_ex.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_reduce_ind_same_value.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_reduce_max_min_ind.c: Delete.
> 	* c-c++-common/cilk-plus/AN/sec_reduce_return.c: Delete.
> 	* c-c++-common/cilk-plus/AN/side-effects-1.c: Delete.
> 	* c-c++-common/cilk-plus/AN/test_builtin_return.c: Delete.
> 	* c-c++-common/cilk-plus/AN/test_sec_limits.c: Delete.
> 	* c-c++-common/cilk-plus/AN/tst_lngth.c: Delete.
> 	* c-c++-common/cilk-plus/AN/vla.c: Delete.
> 	* c-c++-common/cilk-plus/CK/Wparentheses-1.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk-for-2.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk-for-3.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk-fors.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk_for_errors.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk_for_grain.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk_for_grain_errors.c: Delete.
> 	* c-c++-common/cilk-plus/CK/cilk_for_ptr_iter.c: Delete.
> 	* c-c++-common/cilk-plus/CK/compound_cilk_spawn.c: Delete.
> 	* c-c++-common/cilk-plus/CK/concec_cilk_spawn.c: Delete.
> 	* c-c++-common/cilk-plus/CK/errors.c: Delete.
> 	* c-c++-common/cilk-plus/CK/fib.c: Delete.
> 	* c-c++-common/cilk-plus/CK/fib_init_expr_xy.c: Delete.
> 	* c-c++-common/cilk-plus/CK/fib_no_return.c: Delete.
> 	* c-c++-common/cilk-plus/CK/fib_no_sync.c: Delete.
> 	* c-c++-common/cilk-plus/CK/invalid_spawns.c: Delete.
> 	* c-c++-common/cilk-plus/CK/invalid_sync.c: Delete.c
> 	* c-c++-common/cilk-plus/CK/nested_cilk_for.c: Delete.
> 	* c-c++-common/cilk-plus/CK/no_args_error.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr59631.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr60197-2.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr60197.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr60469.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr60586.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr63307.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr69826-1.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr69826-2.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr79428-4.c: Delete.
> 	* c-c++-common/cilk-plus/CK/pr79428-7.c: Delete.
> 	* c-c++-common/cilk-plus/CK/spawn_in_return.c: Delete.
> 	* c-c++-common/cilk-plus/CK/spawnee_inline.c: Delete.
> 	* c-c++-common/cilk-plus/CK/spawner_inline.c: Delete.
> 	* c-c++-common/cilk-plus/CK/spawning_arg.c: Delete.
> 	* c-c++-common/cilk-plus/CK/steal_check.c: Delete.
> 	* c-c++-common/cilk-plus/CK/sync_wo_spawn.c: Delete.
> 	* c-c++-common/cilk-plus/CK/test__cilk.c: Delete.
> 	* c-c++-common/cilk-plus/CK/varargs_test.c: Delete.
> 	* c-c++-common/cilk-plus/PS/Wparentheses-1.c: Delete.
> 	* c-c++-common/cilk-plus/PS/body.c: Delete.
> 	* c-c++-common/cilk-plus/PS/clauses1.c: Delete.
> 	* c-c++-common/cilk-plus/PS/clauses2.c: Delete.
> 	* c-c++-common/cilk-plus/PS/clauses3.c: Delete.
> 	* c-c++-common/cilk-plus/PS/clauses4.c: Delete.
> 	* c-c++-common/cilk-plus/PS/for1.c: Delete.
> 	* c-c++-common/cilk-plus/PS/for2.c: Delete.
> 	* c-c++-common/cilk-plus/PS/for3.c: Delete.
> 	* c-c++-common/cilk-plus/PS/pr69363.c: Delete.
> 	* c-c++-common/cilk-plus/PS/reduction-1.c: Delete.
> 	* c-c++-common/cilk-plus/PS/reduction-2.c: Delete.
> 	* c-c++-common/cilk-plus/PS/reduction-3.c: Delete.
> 	* c-c++-common/cilk-plus/PS/run-1.c: Delete.
> 	* c-c++-common/cilk-plus/PS/safelen.c: Delete.
> 	* c-c++-common/cilk-plus/PS/vectorlength-2.c: Delete.
> 	* c-c++-common/cilk-plus/PS/vectorlength-3.c: Delete.
> 	* c-c++-common/cilk-plus/PS/vectorlength.c: Delete.
> 	* c-c++-common/cilk-plus/SE/ef_error.c: Delete.
> 	* c-c++-common/cilk-plus/SE/ef_error2.c: Delete.
> 	* c-c++-common/cilk-plus/SE/ef_error3.c: Delete.
> 	* c-c++-common/cilk-plus/SE/ef_test.c: Delete.
> 	* c-c++-common/cilk-plus/SE/ef_test2.c: Delete.
> 	* c-c++-common/cilk-plus/SE/vlength_errors.c: Delete.
> 	* g++.dg/cilk-plus/AN/array_function.c: Delete.
> 	* g++.dg/cilk-plus/AN/array_test1_tplt.c: Delete.
> 	* g++.dg/cilk-plus/AN/array_test2_tplt.c: Delete.
> 	* g++.dg/cilk-plus/AN/array_test_ND_tplt.c: Delete.
> 	* g++.dg/cilk-plus/AN/braced_list.c: Delete.
> 	* g++.dg/cilk-plus/AN/builtin_fn_custom_tplt.c: Delete.
> 	* g++.dg/cilk-plus/AN/builtin_fn_mutating_tplt.c: Delete.
> 	* g++.dg/cilk-plus/AN/fp_triplet_values_tplt.c: Delete.
> 	* g++.dg/cilk-plus/AN/postincr_test.c: Delete.
> 	* g++.dg/cilk-plus/AN/preincr_test.c: Delete.
> 	* g++.dg/cilk-plus/CK/catch_exc.c: Delete.
> 	* g++.dg/cilk-plus/CK/cf3.c: Delete.
> 	* g++.dg/cilk-plus/CK/cilk-for-tplt.c: Delete.
> 	* g++.dg/cilk-plus/CK/const_spawn.c: Delete.
> 	* g++.dg/cilk-plus/CK/fib-opr-overload.c: Delete.
> 	* g++.dg/cilk-plus/CK/fib-tplt.c: Delete.
> 	* g++.dg/cilk-plus/CK/for1.c: Delete.
> 	* g++.dg/cilk-plus/CK/lambda_spawns.c: Delete.
> 	* g++.dg/cilk-plus/CK/lambda_spawns_tplt.c: Delete.
> 	* g++.dg/cilk-plus/CK/pr60586.c: Delete.
> 	* g++.dg/cilk-plus/CK/pr66326.c: Delete.
> 	* g++.dg/cilk-plus/CK/pr68001.c: Delete.
> 	* g++.dg/cilk-plus/CK/pr68997.c: Delete.
> 	* g++.dg/cilk-plus/CK/pr69024.c: Delete.
> 	* g++.dg/cilk-plus/CK/pr69048.c: Delete.
> 	* g++.dg/cilk-plus/CK/pr69267.c: Delete.
> 	* g++.dg/cilk-plus/CK/pr80038.c: Delete.
> 	* g++.dg/cilk-plus/CK/stl_iter.c: Delete.
> 	* g++.dg/cilk-plus/CK/stl_rev_iter.c: Delete.
> 	* g++.dg/cilk-plus/CK/stl_test.c: Delete.
> 	* g++.dg/cilk-plus/cilk-plus.exp
> 	* g++.dg/cilk-plus/ef_test.C: Delete.
> 	* g++.dg/cilk-plus/for.C: Delete.
> 	* g++.dg/cilk-plus/for2.C: Delete.
> 	* g++.dg/cilk-plus/for3.C: Delete.
> 	* g++.dg/cilk-plus/for4.C: Delete.
> 	* g++.dg/cilk-plus/pr60967.C: Delete.
> 	* g++.dg/cilk-plus/pr69028.C: Delete.
> 	* g++.dg/cilk-plus/pr70565.C: Delete.
> 	* g++.dg/pr57662.C: Delete.
> 	* gcc.dg/cilk-plus/cilk-plus.exp
> 	* gcc.dg/cilk-plus/for1.c: Delete.
> 	* gcc.dg/cilk-plus/for2.c: Delete.
> 	* gcc.dg/cilk-plus/jump-openmp.c: Delete.
> 	* gcc.dg/cilk-plus/jump.c: Delete.
> 	* gcc.dg/cilk-plus/pr69798-1.c: Delete.
> 	* gcc.dg/cilk-plus/pr69798-2.c: Delete.
> 	* gcc.dg/cilk-plus/pr78306.c: Delete.
> 	* gcc.dg/cilk-plus/pr79116.c: Delete.
> 	* gcc.dg/graphite/id-28.c: Delete.
> 	* lib/cilk-plus-dg.exp: Delete.
> 	* lib/target-supports.exp (cilkplus_runtime): Delete.
> 
> libcilkrts: Delete
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.

It's not unusual to find lingering bits after the removal.  So if after
the kit is ack'd & committed you find additional tidbits, don't hesitate
to let us know.


In c-family/c-gimplify.c:c_gimplify_expr you have this change:

     case MODIFY_EXPR:
     case INIT_EXPR:
     case CALL_EXPR:
-      if (fn_contains_cilk_spawn_p (cfun)
-         && cilk_detect_spawn_and_unwrap (expr_p)
-         /* If an error is found, the spawn wrapper is removed and the
-            original expression (MODIFY/INIT/CALL_EXPR) is processes as
-            it is supposed to be.  */
-         && !seen_error ())
-        return (enum gimplify_status) gimplify_cilk_spawn (expr_p);
-
     default:;
I wonder if we should just remove the case labels as well since they do
nothing after the cilk+ bits are removed.

In c-family/c.opt rather than deleting the flag entirely, standard
practice is leave the flag and issue a Deprecated error/warning.  There
should be examples in c-family/c.opt.

In c/c-parser:

@@ -2338,8 +2330,6 @@ c_parser_declaration_or_fndef (c_parser *parser,
bool fndef_ok,
       else
        {
          fnbody = c_parser_compound_statement (parser);
-         if (flag_cilkplus && contains_array_notation_expr (fnbody))
-           fnbody = expand_array_notation_exprs (fnbody);
        }
       tree fndecl = current_function_decl;
       if (nested)

I think the curly-braces become redundant after your remove the cilk+
specific code.

Also in c/c-parser.c, this time in
c_parser_postfix_expression_after_primary.  The assignment to idx before
this comment:

+      /* Here we have 3 options:
+        1. Array [EXPR] -- Normal Array call.
+        2. Array [EXPR : EXPR] -- Array notation without stride.
+        3. Array [EXPR : EXPR : EXPR] -- Array notation with stride.
+
+        For 1, we just handle it just like a normal array expression.
+        For 2 and 3 we handle it like we handle array notations.  The
+        idx value we have above becomes the initial/start index.
+      */

Appears to be indented too far.  Also the close comment should be at the
end of the previous line.


In these two hunks, you need to reindent the "else" clause that has
become unconditional:
@@ -9416,15 +9195,9 @@ c_parser_postfix_expression_after_primary
(c_parser *parser,
          finish = c_parser_peek_token (parser)->get_finish ();
          c_parser_consume_token (parser);
          /* If the expressions have array notations, we expand them.  */
-         if (flag_cilkplus
-             && TREE_CODE (expr.value) == ARRAY_NOTATION_REF)
-           expr = fix_array_notation_expr (expr_loc,
POSTINCREMENT_EXPR, expr);
-         else
-           {
              expr = default_function_array_read_conversion (expr_loc,
expr);
              expr.value = build_unary_op (op_loc, POSTINCREMENT_EXPR,
                                           expr.value, false);
-           }
          set_c_expr_source_range (&expr, start, finish);
          expr.original_code = ERROR_MARK;
          expr.original_type = NULL;
@@ -9435,15 +9208,9 @@ c_parser_postfix_expression_after_primary
(c_parser *parser,
          finish = c_parser_peek_token (parser)->get_finish ();
          c_parser_consume_token (parser);
          /* If the expressions have array notations, we expand them.  */
-         if (flag_cilkplus
-             && TREE_CODE (expr.value) == ARRAY_NOTATION_REF)
-           expr = fix_array_notation_expr (expr_loc,
POSTDECREMENT_EXPR, expr);
-         else
-           {
              expr = default_function_array_read_conversion (expr_loc,
expr);
              expr.value = build_unary_op (op_loc, POSTDECREMENT_EXPR,
                                           expr.value, false);
-           }
          set_c_expr_source_range (&expr, start, finish);
          expr.original_code = ERROR_MARK;
          expr.original_type = NULL;


In c_parser_omp_for_loop:


@@ -15882,8 +15604,6 @@ c_parser_omp_for_loop (location_t loc, c_parser
*parser, enum tree_code code,
            case LE_EXPR:
              break;
            case NE_EXPR:
-             if (code == CILK_SIMD || code == CILK_FOR)
-               break;
              /* FALLTHRU.  */
            default:
              /* Can't be cond = error_mark_node, because we want to
preserve

I think you can just delete the NE_EXPR case label and the associated
FALLTHRU comment.


The && part of this condition needs reindenting as a result of pulling
the code out of the conditional.



-      if (!is_cilkplus_cilk_simd_fn)
-       gcc_assert (token->type == CPP_NAME
+      gcc_assert (token->type == CPP_NAME
                    && strcmp (IDENTIFIER_POINTER (token->value),
"simd") == 0);


In cp/parser.c:

@@ -25090,12 +24798,6 @@ cp_parser_gnu_attribute_list (cp_parser* parser)
              vec<tree, va_gc> *vec;
              int attr_flag = (attribute_takes_identifier_p (identifier)
                               ? id_attr : normal_attr);
-             if (is_cilkplus_vector_p (identifier))
-               {
-                 cp_parser_cilk_simd_fn_vector_attrs (parser, id_token);
-                 continue;
-               }
-             else
                vec = cp_parser_parenthesized_expression_list
                  (parser, attr_flag, /*cast_p=*/false,
                   /*allow_expansion_p=*/false,
You need to reindent the code from the else clause that you're keeping.


I think the outer curleys are redundant now:


@@ -34825,12 +34495,8 @@ cp_parser_omp_for_loop_init (cp_parser *parser,
              if (cp_lexer_next_token_is (parser->lexer,
                                          CPP_OPEN_PAREN))
                {
-                 if (code != CILK_SIMD && code != CILK_FOR)
-                   error ("parenthesized initialization is not allowed in "
+                  error ("parenthesized initialization is not allowed in "
                           "OpenMP %<for%> loop");
-                 else
-                   error ("parenthesized initialization is "
-                          "not allowed in for-loop");
                }
              else
                /* Trigger an error.  */



Indentation on the else clause here needs to be fixed:

@@ -35267,9 +34921,6 @@ cp_parser_omp_for_loop (cp_parser *parser, enum
tree_code code, tree clauses,

   /* Note that we saved the original contents of this flag when we entered
      the structured block, and so we don't need to re-save it here.  */
-  if (code == CILK_SIMD || code == CILK_FOR)
-    parser->in_statement = IN_CILK_SIMD_FOR;
-  else
     parser->in_statement = IN_OMP_FOR;

   /* Note that the grammar doesn't call for a structured block here,




I know it seems like a lot of issues, but I'm encouraged by the fact
that they're all just formatting things.  The one or two correctness
fragments I was worried about turned out to be non-issues when I looked
at the larger context.

So if you could fix the things noted above and repost.  I'm pretty sure
it'll be approved at that point.

jeff

^ permalink raw reply	[flat|nested] 26+ messages in thread

* RE: [patch] remove cilk-plus
  2017-11-21  7:46     ` Jeff Law
@ 2017-11-22  8:47       ` Koval, Julia
  2017-11-22  9:15         ` Rainer Orth
                           ` (2 more replies)
  0 siblings, 3 replies; 26+ messages in thread
From: Koval, Julia @ 2017-11-22  8:47 UTC (permalink / raw)
  To: Jeff Law, Jakub Jelinek; +Cc: GCC Patches

[-- Attachment #1: Type: text/plain, Size: 25747 bytes --]

Hi,

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

Thanks for your comments, fixed them in attached patch.vm


> -----Original Message-----
> From: Jeff Law [mailto:law@redhat.com]
> Sent: Tuesday, November 21, 2017 8:41 AM
> To: Koval, Julia <julia.koval@intel.com>; Jakub Jelinek <jakub@redhat.com>
> Cc: GCC Patches <gcc-patches@gcc.gnu.org>
> Subject: Re: [patch] remove cilk-plus
> 
> On 11/16/2017 10:02 AM, Koval, Julia wrote:
> > Thanks for your comments, fixed it.
> >
> > 2017-11-16  Julia Koval  <julia.koval@intel.com>
> > 	        Sebastian Peryt  <sebastian.peryt@intel.com>
> >
> > 	* Makefile.def (target_modules): Remove libcilkrts.
> > 	* Makefile.in: Ditto.
> > 	* configure: Ditto.
> > 	* configure.ac: Ditto.
> >
> > contrib/
> > 	* contrib/gcc_update: Ditto.
> >
> > gcc/
> > 	* Makefile.in (cilkplus.def, cilk-builtins.def, c-family/cilk.o,
> > 	c-family/c-cilkplus.o, c-family/array-notation-common.o,
> > 	cilk-common.o, cilk.h, cilk-common.c): Remove.
> > 	* builtin-types.def
> > 	(BT_FN_INT_PTR_PTR_PTR_FTYPE_BT_INT_BT_PTR_BT_PTR_BT_PTR):
> Remove.
> > 	* builtins.c (is_builtin_name): Remove cilkplus condition.
> > 	(BUILT_IN_CILK_DETACH, BUILT_IN_CILK_POP_FRAME): Remove.
> > 	* builtins.def (DEF_CILK_BUILTIN_STUB, DEF_CILKPLUS_BUILTIN,
> > 	cilk-builtins.def, cilkplus.def): Remove.
> > 	* cif-code.def (CILK_SPAWN): Remove.
> > 	* cilk-builtins.def: Delete.
> > 	* cilk-common.c: Ditto.
> > 	* cilk.h: Ditto.
> > 	* cilkplus.def: Ditto.
> > 	* config/darwin.h (fcilkplus): Delete.
> > 	* cppbuiltin.c: Ditto.
> > 	* doc/extend.texi: Remove cilkplus doc.
> > 	* doc/generic.texi: Ditto.
> > 	* doc/invoke.texi: Ditto.
> > 	* doc/passes.texi: Ditto.
> > 	* gcc.c (fcilkplus): Remove.
> > 	* gengtype.c (cilk.h): Remove.
> > 	* gimple-pretty-print.c (dump_gimple_omp_for): Remove cilkplus
> support.
> > 	* gimple.h (GF_OMP_FOR_KIND_CILKFOR,
> GF_OMP_FOR_KIND_CILKSIMD): Remove.
> > 	* gimplify.c (gimplify_return_expr, maybe_fold_stmt,
> gimplify_call_expr,
> > 	is_gimple_stmt, gimplify_modify_expr, gimplify_scan_omp_clauses,
> > 	gimplify_adjust_omp_clauses, gimplify_omp_for, gimplify_expr):
> Remove
> > 	cilkplus conditions.
> > 	* ipa-fnsummary.c (ipa_dump_fn_summary, compute_fn_summary,
> > 	inline_read_section): Ditto.
> > 	* ipa-inline-analysis.c (cilk.h): Remove.
> > 	* ira.c (ira_setup_eliminable_regset): Remove cilkplus support.
> > 	* lto-wrapper.c (merge_and_complain, append_compiler_options,
> > 	append_linker_options): Remove condition for fcilkplus.
> > 	* lto/lto-lang.c (cilk.h): Remove.
> > 	(lto_init): Remove condition for fcilkplus.
> > 	* omp-expand.c (expand_cilk_for_call): Delete.
> > 	(expand_omp_taskreg, expand_omp_for_static_chunk,
> > 	expand_omp_for): Remove cilkplus
> > 	conditions.
> > 	(expand_cilk_for): Delete.
> > 	* omp-general.c (omp_extract_for_data): Remove cilkplus support.
> > 	* omp-low.c (scan_sharing_clauses, create_omp_child_function,
> > 	execute_lower_omp, diagnose_sb_0): Ditto.
> > 	* omp-simd-clone.c (simd_clone_clauses_extract): Ditto.
> > 	* tree-core.h (OMP_CLAUSE__CILK_FOR_COUNT_): Delete.
> > 	* tree-nested.c: Ditto.
> > 	* tree-pretty-print.c (dump_omp_clause): Remove cilkplus support.
> > 	(dump_generic_node): Ditto.
> > 	* tree.c (OMP_CLAUSE__CILK_FOR_COUNT_): Delete.
> > 	* tree.def (cilk_simd, cilk_for, cilk_spawn_stmt, cilk_sync_stmt): Delete.
> > 	* tree.h (CILK_SPAWN_FN, EXPR_CILK_SPAWN): Delete.
> >
> > gcc/c-family/
> > 	* array-notation-common.c: Delete.
> > 	* c-cilkplus.c: Ditto.
> > 	* c-common.c (_Cilk_spawn, _Cilk_sync, _Cilk_for): Remove.
> > 	* c-common.def (ARRAY_NOTATION_REF): Remove.
> > 	* c-common.h (RID_CILK_SPAWN, build_array_notation_expr,
> > 	build_array_notation_ref, C_ORT_CILK, c_check_cilk_loop,
> > 	c_validate_cilk_plus_loop, cilkplus_an_parts,
> cilk_ignorable_spawn_rhs_op,
> > 	cilk_recognize_spawn): Remove.
> > 	* c-gimplify.c (CILK_SPAWN_STMT): Remove.
> > 	* c-omp.c: Remove CILK_SIMD check.
> > 	* c-pragma.c: Ditto.
> > 	* c-pragma.h: Remove CILK related pragmas.
> > 	* c-pretty-print.c (c_pretty_printer::postfix_expression): Remove
> > 	ARRAY_NOTATION_REF condition.
> > 	(c_pretty_printer::expression): Ditto.
> > 	* c.opt (fcilkplus): Remove.
> > 	* cilk.c: Delete.
> >
> > gcc/c/
> > 	* Make-lang.in (c/c-array-notation.o): Remove.
> > 	* c-array-notation.c: Delete.
> > 	* c-decl.c: Remove cilkplus condition.
> > 	* c-parser.c (c_parser_cilk_simd, c_parser_cilk_for,
> > 	c_parser_cilk_verify_simd, c_parser_array_notation,
> > 	c_parser_cilk_clause_vectorlength, c_parser_cilk_grainsize,
> > 	c_parser_cilk_simd_fn_vector_attrs,
> > 	c_finish_cilk_simd_fn_tokens): Delete.
> > 	(c_parser_declaration_or_fndef): Remove cilkplus condition.
> > 	(c_parser_direct_declarator_inner): Ditto.
> > 	(CILK_SIMD_FN_CLAUSE_MASK): Delete.
> > 	(c_parser_attributes, c_parser_compound_statement,
> > 	c_parser_statement_after_labels, c_parser_if_statement,
> > 	c_parser_switch_statement, c_parser_while_statement,
> > 	c_parser_do_statement, c_parser_for_statement,
> > 	c_parser_unary_expression, c_parser_postfix_expression,
> > 	c_parser_postfix_expression_after_primary,
> > 	c_parser_pragma, c_parser_omp_clause_name,
> c_parser_omp_all_clauses,
> > 	c_parser_omp_for_loop, c_finish_omp_declare_simd): Remove cilkplus
> support.
> > 	* c-typeck.c (build_array_ref, build_function_call_vec,
> convert_arguments,
> > 	lvalue_p, build_compound_expr, c_finish_return, c_finish_if_stmt,
> > 	c_finish_loop, build_binary_op): Remove cilkplus support.
> >
> > gcc/cp/
> > 	* Make-lang.in (cp/cp-array-notation.o, cp/cp-cilkplus.o): Delete.
> > 	* call.c (convert_for_arg_passing, build_cxx_call): Remove cilkplus.
> > 	* constexpr.c (potential_constant_expression_1): Ditto.
> > 	* cp-array-notation.c: Delete.
> > 	* cp-cilkplus.c: Ditto.
> > 	* cp-cilkplus.h: Ditto.
> > 	* cp-gimplify.c (cp_gimplify_expr, cp_fold_r, cp_genericize): Remove
> > 	cilkplus condition.
> > 	* cp-objcp-common.c (ARRAY_NOTATION_REF): Delete.
> > 	* cp-tree.h (cilkplus_an_triplet_types_ok_p): Delete.
> > 	* decl.c (grokfndecl, finish_function): Remove cilkplus condition.
> > 	* error.c (dump_decl, dump_expr): Remove ARRAY_NOTATION_REF
> condition.
> > 	* lambda.c (cp-cilkplus.h): Remove.
> > 	* parser.c (cp_parser_cilk_simd, cp_parser_cilk_for,
> > 	cp_parser_cilk_simd_vectorlength): Delete.
> > 	(cp_debug_parser, cp_parser_ctor_initializer_opt_and_function_body,
> > 	cp_parser_postfix_expression,
> cp_parser_postfix_open_square_expression,
> > 	cp_parser_statement, cp_parser_jump_statement,
> cp_parser_direct_declarator,
> > 	cp_parser_late_return_type_opt, cp_parser_gnu_attribute_list,
> > 	cp_parser_omp_clause_name, cp_parser_omp_clause_aligned,
> > 	cp_parser_omp_clause_linear, cp_parser_omp_all_clauses,
> cp_parser_omp_flush,
> > 	cp_parser_omp_for_cond, cp_parser_omp_for_incr,
> cp_parser_omp_for_loop_init,
> > 	cp_parser_omp_for_loop,
> > 	cp_parser_omp_declare_simd): Remove cilkplus support.
> > 	(CILK_SIMD_FN_CLAUSE_MASK,
> cp_parser_late_parsing_cilk_simd_fn_info,
> > 	cp_parser_cilk_grainsize): Remove.
> > 	(cp_parser_pragma, c_parse_file): Remove cilkplus support.
> > 	(cp_parser_cilk_simd_vectorlength, cp_parser_cilk_simd_linear,
> > 	cp_parser_cilk_simd_clause_name, cp_parser_cilk_simd_all_clauses,
> > 	cp_parser_cilk_simd, cp_parser_cilk_for): Remove.
> > 	* parser.h (IN_CILK_SIMD_FOR, IN_CILK_SPAWN): Remove.
> > 	* pt.c (tsubst_attribute, tsubst_expr, tsubst_copy_and_build): Remove
> > 	cilkplus support.
> > 	* semantics.c (finish_goto_stmt, begin_while_stmt, finish_do_body,
> > 	finish_init_stmt, finish_switch_cond, simplify_aggr_init_expr,
> > 	finish_omp_clauses, finish_omp_clauses, finish_omp_for): Remove
> cilkplus
> > 	support.
> > 	* tree.c (lvalue_kind): Remove ARRAY_NOTATION_REF conditon.
> > 	* typeck.c (cp_build_array_ref, cp_build_compound_expr,
> > 	check_return_expr): Remove cilkplus support.
> >
> > gcc/testsuite/
> > 	* c-c++-common/attr-simd-3.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/an-if.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/array_test1.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/array_test2.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/array_test_ND.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/builtin_fn_custom.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/builtin_fn_mutating.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/builtin_func_double.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/builtin_func_double2.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/comma_exp.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/conditional.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/decl-ptr-colon.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/dimensionless-arrays.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/exec-once.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/exec-once2.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/fn_ptr-2.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/fn_ptr.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/fp_triplet_values.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/gather-scatter-errors.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/gather_scatter.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/if_test.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/if_test_errors.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/misc.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/n-ptr-test.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/parser_errors.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/parser_errors2.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/parser_errors3.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/parser_errors4.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr57457-2.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr57457.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr57490.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr57541-2.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr57541.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr57577.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr58942.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr61191.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr61455-2.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr61455.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr61962.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr61963.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr62008.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/pr63884.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/rank_mismatch.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/rank_mismatch2.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/rank_mismatch3.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/sec_implicit.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/sec_implicit2.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/sec_implicit_ex.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/sec_reduce_ind_same_value.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/sec_reduce_max_min_ind.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/sec_reduce_return.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/side-effects-1.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/test_builtin_return.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/test_sec_limits.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/tst_lngth.c: Delete.
> > 	* c-c++-common/cilk-plus/AN/vla.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/Wparentheses-1.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/cilk-for-2.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/cilk-for-3.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/cilk-fors.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/cilk_for_errors.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/cilk_for_grain.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/cilk_for_grain_errors.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/cilk_for_ptr_iter.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/compound_cilk_spawn.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/concec_cilk_spawn.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/errors.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/fib.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/fib_init_expr_xy.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/fib_no_return.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/fib_no_sync.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/invalid_spawns.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/invalid_sync.c: Delete.c
> > 	* c-c++-common/cilk-plus/CK/nested_cilk_for.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/no_args_error.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/pr59631.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/pr60197-2.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/pr60197.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/pr60469.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/pr60586.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/pr63307.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/pr69826-1.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/pr69826-2.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/pr79428-4.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/pr79428-7.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/spawn_in_return.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/spawnee_inline.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/spawner_inline.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/spawning_arg.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/steal_check.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/sync_wo_spawn.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/test__cilk.c: Delete.
> > 	* c-c++-common/cilk-plus/CK/varargs_test.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/Wparentheses-1.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/body.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/clauses1.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/clauses2.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/clauses3.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/clauses4.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/for1.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/for2.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/for3.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/pr69363.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/reduction-1.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/reduction-2.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/reduction-3.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/run-1.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/safelen.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/vectorlength-2.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/vectorlength-3.c: Delete.
> > 	* c-c++-common/cilk-plus/PS/vectorlength.c: Delete.
> > 	* c-c++-common/cilk-plus/SE/ef_error.c: Delete.
> > 	* c-c++-common/cilk-plus/SE/ef_error2.c: Delete.
> > 	* c-c++-common/cilk-plus/SE/ef_error3.c: Delete.
> > 	* c-c++-common/cilk-plus/SE/ef_test.c: Delete.
> > 	* c-c++-common/cilk-plus/SE/ef_test2.c: Delete.
> > 	* c-c++-common/cilk-plus/SE/vlength_errors.c: Delete.
> > 	* g++.dg/cilk-plus/AN/array_function.c: Delete.
> > 	* g++.dg/cilk-plus/AN/array_test1_tplt.c: Delete.
> > 	* g++.dg/cilk-plus/AN/array_test2_tplt.c: Delete.
> > 	* g++.dg/cilk-plus/AN/array_test_ND_tplt.c: Delete.
> > 	* g++.dg/cilk-plus/AN/braced_list.c: Delete.
> > 	* g++.dg/cilk-plus/AN/builtin_fn_custom_tplt.c: Delete.
> > 	* g++.dg/cilk-plus/AN/builtin_fn_mutating_tplt.c: Delete.
> > 	* g++.dg/cilk-plus/AN/fp_triplet_values_tplt.c: Delete.
> > 	* g++.dg/cilk-plus/AN/postincr_test.c: Delete.
> > 	* g++.dg/cilk-plus/AN/preincr_test.c: Delete.
> > 	* g++.dg/cilk-plus/CK/catch_exc.c: Delete.
> > 	* g++.dg/cilk-plus/CK/cf3.c: Delete.
> > 	* g++.dg/cilk-plus/CK/cilk-for-tplt.c: Delete.
> > 	* g++.dg/cilk-plus/CK/const_spawn.c: Delete.
> > 	* g++.dg/cilk-plus/CK/fib-opr-overload.c: Delete.
> > 	* g++.dg/cilk-plus/CK/fib-tplt.c: Delete.
> > 	* g++.dg/cilk-plus/CK/for1.c: Delete.
> > 	* g++.dg/cilk-plus/CK/lambda_spawns.c: Delete.
> > 	* g++.dg/cilk-plus/CK/lambda_spawns_tplt.c: Delete.
> > 	* g++.dg/cilk-plus/CK/pr60586.c: Delete.
> > 	* g++.dg/cilk-plus/CK/pr66326.c: Delete.
> > 	* g++.dg/cilk-plus/CK/pr68001.c: Delete.
> > 	* g++.dg/cilk-plus/CK/pr68997.c: Delete.
> > 	* g++.dg/cilk-plus/CK/pr69024.c: Delete.
> > 	* g++.dg/cilk-plus/CK/pr69048.c: Delete.
> > 	* g++.dg/cilk-plus/CK/pr69267.c: Delete.
> > 	* g++.dg/cilk-plus/CK/pr80038.c: Delete.
> > 	* g++.dg/cilk-plus/CK/stl_iter.c: Delete.
> > 	* g++.dg/cilk-plus/CK/stl_rev_iter.c: Delete.
> > 	* g++.dg/cilk-plus/CK/stl_test.c: Delete.
> > 	* g++.dg/cilk-plus/cilk-plus.exp
> > 	* g++.dg/cilk-plus/ef_test.C: Delete.
> > 	* g++.dg/cilk-plus/for.C: Delete.
> > 	* g++.dg/cilk-plus/for2.C: Delete.
> > 	* g++.dg/cilk-plus/for3.C: Delete.
> > 	* g++.dg/cilk-plus/for4.C: Delete.
> > 	* g++.dg/cilk-plus/pr60967.C: Delete.
> > 	* g++.dg/cilk-plus/pr69028.C: Delete.
> > 	* g++.dg/cilk-plus/pr70565.C: Delete.
> > 	* g++.dg/pr57662.C: Delete.
> > 	* gcc.dg/cilk-plus/cilk-plus.exp
> > 	* gcc.dg/cilk-plus/for1.c: Delete.
> > 	* gcc.dg/cilk-plus/for2.c: Delete.
> > 	* gcc.dg/cilk-plus/jump-openmp.c: Delete.
> > 	* gcc.dg/cilk-plus/jump.c: Delete.
> > 	* gcc.dg/cilk-plus/pr69798-1.c: Delete.
> > 	* gcc.dg/cilk-plus/pr69798-2.c: Delete.
> > 	* gcc.dg/cilk-plus/pr78306.c: Delete.
> > 	* gcc.dg/cilk-plus/pr79116.c: Delete.
> > 	* gcc.dg/graphite/id-28.c: Delete.
> > 	* lib/cilk-plus-dg.exp: Delete.
> > 	* lib/target-supports.exp (cilkplus_runtime): Delete.
> >
> > libcilkrts: Delete
> 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.
> 
> It's not unusual to find lingering bits after the removal.  So if after
> the kit is ack'd & committed you find additional tidbits, don't hesitate
> to let us know.
> 
> 
> In c-family/c-gimplify.c:c_gimplify_expr you have this change:
> 
>      case MODIFY_EXPR:
>      case INIT_EXPR:
>      case CALL_EXPR:
> -      if (fn_contains_cilk_spawn_p (cfun)
> -         && cilk_detect_spawn_and_unwrap (expr_p)
> -         /* If an error is found, the spawn wrapper is removed and the
> -            original expression (MODIFY/INIT/CALL_EXPR) is processes as
> -            it is supposed to be.  */
> -         && !seen_error ())
> -        return (enum gimplify_status) gimplify_cilk_spawn (expr_p);
> -
>      default:;
> I wonder if we should just remove the case labels as well since they do
> nothing after the cilk+ bits are removed.
> 
> In c-family/c.opt rather than deleting the flag entirely, standard
> practice is leave the flag and issue a Deprecated error/warning.  There
> should be examples in c-family/c.opt.
> 
> In c/c-parser:
> 
> @@ -2338,8 +2330,6 @@ c_parser_declaration_or_fndef (c_parser *parser,
> bool fndef_ok,
>        else
>         {
>           fnbody = c_parser_compound_statement (parser);
> -         if (flag_cilkplus && contains_array_notation_expr (fnbody))
> -           fnbody = expand_array_notation_exprs (fnbody);
>         }
>        tree fndecl = current_function_decl;
>        if (nested)
> 
> I think the curly-braces become redundant after your remove the cilk+
> specific code.
> 
> Also in c/c-parser.c, this time in
> c_parser_postfix_expression_after_primary.  The assignment to idx before
> this comment:
> 
> +      /* Here we have 3 options:
> +        1. Array [EXPR] -- Normal Array call.
> +        2. Array [EXPR : EXPR] -- Array notation without stride.
> +        3. Array [EXPR : EXPR : EXPR] -- Array notation with stride.
> +
> +        For 1, we just handle it just like a normal array expression.
> +        For 2 and 3 we handle it like we handle array notations.  The
> +        idx value we have above becomes the initial/start index.
> +      */
> 
> Appears to be indented too far.  Also the close comment should be at the
> end of the previous line.
> 
> 
> In these two hunks, you need to reindent the "else" clause that has
> become unconditional:
> @@ -9416,15 +9195,9 @@ c_parser_postfix_expression_after_primary
> (c_parser *parser,
>           finish = c_parser_peek_token (parser)->get_finish ();
>           c_parser_consume_token (parser);
>           /* If the expressions have array notations, we expand them.  */
> -         if (flag_cilkplus
> -             && TREE_CODE (expr.value) == ARRAY_NOTATION_REF)
> -           expr = fix_array_notation_expr (expr_loc,
> POSTINCREMENT_EXPR, expr);
> -         else
> -           {
>               expr = default_function_array_read_conversion (expr_loc,
> expr);
>               expr.value = build_unary_op (op_loc, POSTINCREMENT_EXPR,
>                                            expr.value, false);
> -           }
>           set_c_expr_source_range (&expr, start, finish);
>           expr.original_code = ERROR_MARK;
>           expr.original_type = NULL;
> @@ -9435,15 +9208,9 @@ c_parser_postfix_expression_after_primary
> (c_parser *parser,
>           finish = c_parser_peek_token (parser)->get_finish ();
>           c_parser_consume_token (parser);
>           /* If the expressions have array notations, we expand them.  */
> -         if (flag_cilkplus
> -             && TREE_CODE (expr.value) == ARRAY_NOTATION_REF)
> -           expr = fix_array_notation_expr (expr_loc,
> POSTDECREMENT_EXPR, expr);
> -         else
> -           {
>               expr = default_function_array_read_conversion (expr_loc,
> expr);
>               expr.value = build_unary_op (op_loc, POSTDECREMENT_EXPR,
>                                            expr.value, false);
> -           }
>           set_c_expr_source_range (&expr, start, finish);
>           expr.original_code = ERROR_MARK;
>           expr.original_type = NULL;
> 
> 
> In c_parser_omp_for_loop:
> 
> 
> @@ -15882,8 +15604,6 @@ c_parser_omp_for_loop (location_t loc, c_parser
> *parser, enum tree_code code,
>             case LE_EXPR:
>               break;
>             case NE_EXPR:
> -             if (code == CILK_SIMD || code == CILK_FOR)
> -               break;
>               /* FALLTHRU.  */
>             default:
>               /* Can't be cond = error_mark_node, because we want to
> preserve
> 
> I think you can just delete the NE_EXPR case label and the associated
> FALLTHRU comment.
> 
> 
> The && part of this condition needs reindenting as a result of pulling
> the code out of the conditional.
> 
> 
> 
> -      if (!is_cilkplus_cilk_simd_fn)
> -       gcc_assert (token->type == CPP_NAME
> +      gcc_assert (token->type == CPP_NAME
>                     && strcmp (IDENTIFIER_POINTER (token->value),
> "simd") == 0);
> 
> 
> In cp/parser.c:
> 
> @@ -25090,12 +24798,6 @@ cp_parser_gnu_attribute_list (cp_parser* parser)
>               vec<tree, va_gc> *vec;
>               int attr_flag = (attribute_takes_identifier_p (identifier)
>                                ? id_attr : normal_attr);
> -             if (is_cilkplus_vector_p (identifier))
> -               {
> -                 cp_parser_cilk_simd_fn_vector_attrs (parser, id_token);
> -                 continue;
> -               }
> -             else
>                 vec = cp_parser_parenthesized_expression_list
>                   (parser, attr_flag, /*cast_p=*/false,
>                    /*allow_expansion_p=*/false,
> You need to reindent the code from the else clause that you're keeping.
> 
> 
> I think the outer curleys are redundant now:
> 
> 
> @@ -34825,12 +34495,8 @@ cp_parser_omp_for_loop_init (cp_parser *parser,
>               if (cp_lexer_next_token_is (parser->lexer,
>                                           CPP_OPEN_PAREN))
>                 {
> -                 if (code != CILK_SIMD && code != CILK_FOR)
> -                   error ("parenthesized initialization is not allowed in "
> +                  error ("parenthesized initialization is not allowed in "
>                            "OpenMP %<for%> loop");
> -                 else
> -                   error ("parenthesized initialization is "
> -                          "not allowed in for-loop");
>                 }
>               else
>                 /* Trigger an error.  */
> 
> 
> 
> Indentation on the else clause here needs to be fixed:
> 
> @@ -35267,9 +34921,6 @@ cp_parser_omp_for_loop (cp_parser *parser,
> enum
> tree_code code, tree clauses,
> 
>    /* Note that we saved the original contents of this flag when we entered
>       the structured block, and so we don't need to re-save it here.  */
> -  if (code == CILK_SIMD || code == CILK_FOR)
> -    parser->in_statement = IN_CILK_SIMD_FOR;
> -  else
>      parser->in_statement = IN_OMP_FOR;
> 
>    /* Note that the grammar doesn't call for a structured block here,
> 
> 
> 
> 
> I know it seems like a lot of issues, but I'm encouraged by the fact
> that they're all just formatting things.  The one or two correctness
> fragments I was worried about turned out to be non-issues when I looked
> at the larger context.
> 
> So if you could fix the things noted above and repost.  I'm pretty sure
> it'll be approved at that point.
> 
> jeff


[-- Attachment #2: cilk-plus.tar.xz --]
[-- Type: application/octet-stream, Size: 82028 bytes --]

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-22  8:47       ` Koval, Julia
@ 2017-11-22  9:15         ` Rainer Orth
  2017-11-22  9:22           ` Koval, Julia
  2017-11-22 17:33         ` Joseph Myers
  2017-11-23 17:58         ` Jeff Law
  2 siblings, 1 reply; 26+ messages in thread
From: Rainer Orth @ 2017-11-22  9:15 UTC (permalink / raw)
  To: Koval, Julia; +Cc: Jeff Law, Jakub Jelinek, GCC Patches

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

^ permalink raw reply	[flat|nested] 26+ messages in thread

* RE: [patch] remove cilk-plus
  2017-11-22  9:15         ` Rainer Orth
@ 2017-11-22  9:22           ` Koval, Julia
  2017-11-22 10:01             ` Koval, Julia
  0 siblings, 1 reply; 26+ messages in thread
From: Koval, Julia @ 2017-11-22  9:22 UTC (permalink / raw)
  To: Rainer Orth; +Cc: Jeff Law, Jakub Jelinek, GCC Patches

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:ro@CeBiTec.Uni-Bielefeld.DE]
> Sent: Wednesday, November 22, 2017 10:11 AM
> To: Koval, Julia <julia.koval@intel.com>
> Cc: Jeff Law <law@redhat.com>; Jakub Jelinek <jakub@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

^ permalink raw reply	[flat|nested] 26+ messages in thread

* RE: [patch] remove cilk-plus
  2017-11-22  9:22           ` Koval, Julia
@ 2017-11-22 10:01             ` Koval, Julia
  0 siblings, 0 replies; 26+ messages in thread
From: Koval, Julia @ 2017-11-22 10:01 UTC (permalink / raw)
  To: 'Rainer Orth'
  Cc: 'Jeff Law', 'Jakub Jelinek', 'GCC Patches'

[-- Attachment #1: Type: text/plain, Size: 2157 bytes --]

Added fix for gcc/doc/sourcebuild.texi

> -----Original Message-----
> From: Koval, Julia
> Sent: Wednesday, November 22, 2017 10:15 AM
> To: Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
> Cc: Jeff Law <law@redhat.com>; Jakub Jelinek <jakub@redhat.com>; GCC
> Patches <gcc-patches@gcc.gnu.org>
> Subject: RE: [patch] remove cilk-plus
> 
> 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:ro@CeBiTec.Uni-Bielefeld.DE]
> > Sent: Wednesday, November 22, 2017 10:11 AM
> > To: Koval, Julia <julia.koval@intel.com>
> > Cc: Jeff Law <law@redhat.com>; Jakub Jelinek <jakub@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

[-- Attachment #2: cilk-plus.tar.xz --]
[-- Type: application/octet-stream, Size: 82124 bytes --]

^ permalink raw reply	[flat|nested] 26+ messages in thread

* RE: [patch] remove cilk-plus
  2017-11-22  8:47       ` Koval, Julia
  2017-11-22  9:15         ` Rainer Orth
@ 2017-11-22 17:33         ` Joseph Myers
  2017-11-23 10:01           ` Koval, Julia
  2017-11-23 17:58         ` Jeff Law
  2 siblings, 1 reply; 26+ messages in thread
From: Joseph Myers @ 2017-11-22 17:33 UTC (permalink / raw)
  To: Koval, Julia; +Cc: Jeff Law, Jakub Jelinek, GCC Patches

This patch version does not appear to address my comment that you're 
leaving behind comments in c-parser.c relating to Cilk array notations 
while removing the subsequent code.  (Or in one case actually reindenting 
the comment that is no longer relevant, rather than removing it.)

-- 
Joseph S. Myers
joseph@codesourcery.com

^ permalink raw reply	[flat|nested] 26+ messages in thread

* RE: [patch] remove cilk-plus
  2017-11-22 17:33         ` Joseph Myers
@ 2017-11-23 10:01           ` Koval, Julia
  2017-11-27 18:06             ` Jeff Law
  0 siblings, 1 reply; 26+ messages in thread
From: Koval, Julia @ 2017-11-23 10:01 UTC (permalink / raw)
  To: Joseph Myers; +Cc: Jeff Law, Jakub Jelinek, GCC Patches

[-- Attachment #1: Type: text/plain, Size: 741 bytes --]

Sorry, I think in this version of this patch they are fixed.

> -----Original Message-----
> From: Joseph Myers [mailto:joseph@codesourcery.com]
> Sent: Wednesday, November 22, 2017 6:23 PM
> To: Koval, Julia <julia.koval@intel.com>
> Cc: Jeff Law <law@redhat.com>; Jakub Jelinek <jakub@redhat.com>; GCC
> Patches <gcc-patches@gcc.gnu.org>
> Subject: RE: [patch] remove cilk-plus
> 
> This patch version does not appear to address my comment that you're
> leaving behind comments in c-parser.c relating to Cilk array notations
> while removing the subsequent code.  (Or in one case actually reindenting
> the comment that is no longer relevant, rather than removing it.)
> 
> --
> Joseph S. Myers
> joseph@codesourcery.com

[-- Attachment #2: cilk-plus.tar.xz --]
[-- Type: application/octet-stream, Size: 82132 bytes --]

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-22  8:47       ` Koval, Julia
  2017-11-22  9:15         ` Rainer Orth
  2017-11-22 17:33         ` Joseph Myers
@ 2017-11-23 17:58         ` Jeff Law
  2 siblings, 0 replies; 26+ messages in thread
From: Jeff Law @ 2017-11-23 17:58 UTC (permalink / raw)
  To: Koval, Julia, Jakub Jelinek; +Cc: GCC Patches

On 11/22/2017 01:38 AM, Koval, Julia wrote:
> Hi,
> 
>> 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.
No need.  As I said, I can guess what those look like :-0

Jeff
> 

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-23 10:01           ` Koval, Julia
@ 2017-11-27 18:06             ` Jeff Law
  2017-11-30 11:55               ` Koval, Julia
  0 siblings, 1 reply; 26+ messages in thread
From: Jeff Law @ 2017-11-27 18:06 UTC (permalink / raw)
  To: Koval, Julia, Joseph Myers; +Cc: Jakub Jelinek, GCC Patches

On 11/23/2017 02:45 AM, Koval, Julia wrote:
> Sorry, I think in this version of this patch they are fixed.
> 
>> -----Original Message-----
>> From: Joseph Myers [mailto:joseph@codesourcery.com]
>> Sent: Wednesday, November 22, 2017 6:23 PM
>> To: Koval, Julia <julia.koval@intel.com>
>> Cc: Jeff Law <law@redhat.com>; Jakub Jelinek <jakub@redhat.com>; GCC
>> Patches <gcc-patches@gcc.gnu.org>
>> Subject: RE: [patch] remove cilk-plus
>>
>> This patch version does not appear to address my comment that you're
>> leaving behind comments in c-parser.c relating to Cilk array notations
>> while removing the subsequent code.  (Or in one case actually reindenting
>> the comment that is no longer relevant, rather than removing it.)
>>
>> --
>> Joseph S. Myers
>> joseph@codesourcery.com
This version is fine to commit.

Can you also remove struct inv_list from c-family/c-common.h.  It was
only used for array notation support.  You can include that in your main
commit or as a separate follow-up.

We may well find other nits like inv_list.  I'm comfortable addressing
those as we bump into them.

Thanks for taking care of this!

jeff

^ permalink raw reply	[flat|nested] 26+ messages in thread

* RE: [patch] remove cilk-plus
  2017-11-27 18:06             ` Jeff Law
@ 2017-11-30 11:55               ` Koval, Julia
  2017-11-30 12:02                 ` Jakub Jelinek
  2017-11-30 16:40                 ` Marek Polacek
  0 siblings, 2 replies; 26+ messages in thread
From: Koval, Julia @ 2017-11-30 11:55 UTC (permalink / raw)
  To: Jeff Law, Joseph Myers; +Cc: Jakub Jelinek, GCC Patches

[-- Attachment #1: Type: text/plain, Size: 1676 bytes --]

Hi, here is the followup patch. Ok for trunk?

gcc/c-family/
	* c-common.h (inv_list): Remove.

Thanks,
Julia

> -----Original Message-----
> From: Jeff Law [mailto:law@redhat.com]
> Sent: Monday, November 27, 2017 6:50 PM
> To: Koval, Julia <julia.koval@intel.com>; Joseph Myers
> <joseph@codesourcery.com>
> Cc: Jakub Jelinek <jakub@redhat.com>; GCC Patches <gcc-
> patches@gcc.gnu.org>
> Subject: Re: [patch] remove cilk-plus
> 
> On 11/23/2017 02:45 AM, Koval, Julia wrote:
> > Sorry, I think in this version of this patch they are fixed.
> >
> >> -----Original Message-----
> >> From: Joseph Myers [mailto:joseph@codesourcery.com]
> >> Sent: Wednesday, November 22, 2017 6:23 PM
> >> To: Koval, Julia <julia.koval@intel.com>
> >> Cc: Jeff Law <law@redhat.com>; Jakub Jelinek <jakub@redhat.com>; GCC
> >> Patches <gcc-patches@gcc.gnu.org>
> >> Subject: RE: [patch] remove cilk-plus
> >>
> >> This patch version does not appear to address my comment that you're
> >> leaving behind comments in c-parser.c relating to Cilk array notations
> >> while removing the subsequent code.  (Or in one case actually reindenting
> >> the comment that is no longer relevant, rather than removing it.)
> >>
> >> --
> >> Joseph S. Myers
> >> joseph@codesourcery.com
> This version is fine to commit.
> 
> Can you also remove struct inv_list from c-family/c-common.h.  It was
> only used for array notation support.  You can include that in your main
> commit or as a separate follow-up.
> 
> We may well find other nits like inv_list.  I'm comfortable addressing
> those as we bump into them.
> 
> Thanks for taking care of this!
> 
> jeff

[-- Attachment #2: 0001-cilk-followup.patch --]
[-- Type: application/octet-stream, Size: 1101 bytes --]

From 09ed2d6ceff7b1e51fa188057d311d1af4432308 Mon Sep 17 00:00:00 2001
From: julia <jkoval@gkticlel801.igk.intel.com>
Date: Thu, 30 Nov 2017 14:42:11 +0300
Subject: [PATCH] cilk followup

---
 gcc/c-family/c-common.h | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/gcc/c-family/c-common.h b/gcc/c-family/c-common.h
index 7561531..27b1de9 100644
--- a/gcc/c-family/c-common.h
+++ b/gcc/c-family/c-common.h
@@ -1358,17 +1358,6 @@ enum stv_conv {
 extern enum stv_conv scalar_to_vector (location_t loc, enum tree_code code,
 				       tree op0, tree op1, bool);
 
-/* This structure holds all the scalar values and its appropriate variable 
-   replacment.  It is mainly used by the function that pulls all the invariant
-   parts that should be executed only once, which comes with array notation 
-   expressions.  */
-struct inv_list
-{
-  vec<tree, va_gc> *list_values;
-  vec<tree, va_gc> *replacement;
-  vec<enum tree_code, va_gc> *additional_tcodes; 
-};
-
 extern tree find_inv_trees (tree *, int *, void *);
 extern tree replace_inv_trees (tree *, int *, void *);
 
-- 
2.5.5


^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-30 11:55               ` Koval, Julia
@ 2017-11-30 12:02                 ` Jakub Jelinek
  2017-11-30 16:40                 ` Marek Polacek
  1 sibling, 0 replies; 26+ messages in thread
From: Jakub Jelinek @ 2017-11-30 12:02 UTC (permalink / raw)
  To: Koval, Julia; +Cc: Jeff Law, Joseph Myers, GCC Patches

On Thu, Nov 30, 2017 at 11:54:11AM +0000, Koval, Julia wrote:
> Hi, here is the followup patch. Ok for trunk?
> 
> gcc/c-family/
> 	* c-common.h (inv_list): Remove.

Ok, thanks.

	Jakub

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-30 11:55               ` Koval, Julia
  2017-11-30 12:02                 ` Jakub Jelinek
@ 2017-11-30 16:40                 ` Marek Polacek
  2017-11-30 17:27                   ` Jeff Law
  1 sibling, 1 reply; 26+ messages in thread
From: Marek Polacek @ 2017-11-30 16:40 UTC (permalink / raw)
  To: Koval, Julia; +Cc: Jeff Law, Joseph Myers, Jakub Jelinek, GCC Patches

On Thu, Nov 30, 2017 at 11:54:11AM +0000, Koval, Julia wrote:
> Hi, here is the followup patch. Ok for trunk?
> 
> gcc/c-family/
> 	* c-common.h (inv_list): Remove.

In function.h I still see

  /* In a Cilk function, the VAR_DECL for the frame descriptor. */
  tree cilk_frame_decl;

isn't that redundant now, too?

	Marek

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-30 16:40                 ` Marek Polacek
@ 2017-11-30 17:27                   ` Jeff Law
  2017-11-30 17:29                     ` Jakub Jelinek
  0 siblings, 1 reply; 26+ messages in thread
From: Jeff Law @ 2017-11-30 17:27 UTC (permalink / raw)
  To: Marek Polacek, Koval, Julia; +Cc: Joseph Myers, Jakub Jelinek, GCC Patches

On 11/30/2017 09:32 AM, Marek Polacek wrote:
> On Thu, Nov 30, 2017 at 11:54:11AM +0000, Koval, Julia wrote:
>> Hi, here is the followup patch. Ok for trunk?
>>
>> gcc/c-family/
>> 	* c-common.h (inv_list): Remove.
> 
> In function.h I still see
> 
>   /* In a Cilk function, the VAR_DECL for the frame descriptor. */
>   tree cilk_frame_decl;
> 
> isn't that redundant now, too?
Yes.  I actually saw that today when looking at the issue Florian and
Carlos raised internally. :-)

Assuming a grep doesn't find any references, removal is pre-approved.

jeff

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-30 17:27                   ` Jeff Law
@ 2017-11-30 17:29                     ` Jakub Jelinek
  0 siblings, 0 replies; 26+ messages in thread
From: Jakub Jelinek @ 2017-11-30 17:29 UTC (permalink / raw)
  To: Jeff Law; +Cc: Marek Polacek, Koval, Julia, Joseph Myers, GCC Patches

On Thu, Nov 30, 2017 at 10:15:20AM -0700, Jeff Law wrote:
> On 11/30/2017 09:32 AM, Marek Polacek wrote:
> > On Thu, Nov 30, 2017 at 11:54:11AM +0000, Koval, Julia wrote:
> >> Hi, here is the followup patch. Ok for trunk?
> >>
> >> gcc/c-family/
> >> 	* c-common.h (inv_list): Remove.
> > 
> > In function.h I still see
> > 
> >   /* In a Cilk function, the VAR_DECL for the frame descriptor. */
> >   tree cilk_frame_decl;
> > 
> > isn't that redundant now, too?
> Yes.  I actually saw that today when looking at the issue Florian and
> Carlos raised internally. :-)
> 
> Assuming a grep doesn't find any references, removal is pre-approved.

grep finds actually tons of further stuff:

grep -i cilk * c*/* 2>/dev/null | grep -v ChangeLog
cgraph.h:  /* True if this is a Cilk Plus variant.  */
cgraph.h:  unsigned int cilk_elemental : 1;
function.h:  /* In a Cilk function, the VAR_DECL for the frame descriptor. */
function.h:  tree cilk_frame_decl;
function.h:  /* This will indicate whether a function is a cilk function */
function.h:  unsigned int is_cilk_function : 1;
function.h:  /* Nonzero if this is a Cilk function that spawns. */
function.h:  unsigned int calls_cilk_spawn : 1;
omp-simd-clone.c:  clone_info->cilk_elemental = false;
target.def:/* Functions relating to OpenMP and Cilk Plus SIMD clones.  */
tree-inline.h:  /* Cilk keywords currently need to replace some variables that
tree-inline.h:  bool remap_var_for_cilk;
c-family/c-attribs.c:  { "cilk simd function",     0, -1, true,  false, false,
c-family/c-attribs.c:      if (lookup_attribute ("cilk simd function",
c-family/c-attribs.c:		    "function marked as a Cilk Plus SIMD-enabled function");
c-family/c.opt:fcilkplus Undocumented
c-family/c.opt:C ObjC C++ ObjC++ LTO Report Var(flag_cilkplus) Init(0)
cp/cp-tree.h:/* In c-family/cilk.c */
cp/cp-tree.h:extern bool cilk_valid_spawn                    (tree);
cp/parser.c:  parser->cilk_simd_fn_info = NULL;
cp/parser.c:   clauses, Cilk Plus SIMD-enabled functions' vector attributes, and
cp/parser.c:   a Cilk Plus for loop.
cp/parser.h:/* Helper data structure for parsing #pragma omp declare simd, and Cilk Plus
cp/parser.h:  /* When parsing Cilk Plus SIMD-enabled functions' vector attributes,
cp/parser.h:  cp_omp_declare_simd_data * GTY((skip)) cilk_simd_fn_info;

Of course, when it is Cilk+ + something else, it should be carefully adjusted.

	Jakub

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-16 15:48 [patch] remove cilk-plus Koval, Julia
                   ` (2 preceding siblings ...)
  2017-11-16 16:34 ` Eric Gallager
@ 2017-12-01 10:28 ` Paolo Carlini
  2017-12-01 15:43   ` Jeff Law
  2018-03-02 16:37 ` Thomas Schwinge
  4 siblings, 1 reply; 26+ messages in thread
From: Paolo Carlini @ 2017-12-01 10:28 UTC (permalink / raw)
  To: Koval, Julia, GCC Patches; +Cc: law

Hi,

On 16/11/2017 16:33, Koval, Julia wrote:
> // I failed to send patch itself, it is too big even in gzipped form.  What is the right way to send such big patches?
>
> Hi, this patch removes cilkplus. Ok for trunk?
Now that cilkplus is gone I suppose we should clean-up Bugzilla about 
that. Shall I go ahead and essentially close all the bugs we got? As 
WONTFIX or what else? Let's agree on something. In principle we could 
keep the regressions for the sake of the existing release branches but I 
think we got very, very, few of those and anyway I don't see who gonna 
work on that...

Cheers,
Paolo.

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-12-01 10:28 ` Paolo Carlini
@ 2017-12-01 15:43   ` Jeff Law
  2017-12-01 16:19     ` Paolo Carlini
  0 siblings, 1 reply; 26+ messages in thread
From: Jeff Law @ 2017-12-01 15:43 UTC (permalink / raw)
  To: Paolo Carlini, Koval, Julia, GCC Patches

On 12/01/2017 03:28 AM, Paolo Carlini wrote:
> Hi,
> 
> On 16/11/2017 16:33, Koval, Julia wrote:
>> // I failed to send patch itself, it is too big even in gzipped form. 
>> What is the right way to send such big patches?
>>
>> Hi, this patch removes cilkplus. Ok for trunk?
> Now that cilkplus is gone I suppose we should clean-up Bugzilla about
> that. Shall I go ahead and essentially close all the bugs we got? As
> WONTFIX or what else? Let's agree on something. In principle we could
> keep the regressions for the sake of the existing release branches but I
> think we got very, very, few of those and anyway I don't see who gonna
> work on that...
Not sure if we have a policy in this space or not.

If we don't then my vote would be for CLOSE/WONTFIX now.  That seems to
most accurately reflect state -- we're not going to be fixing any Cilk+
stuff on the trunk or in the release branches.


jeff

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-12-01 15:43   ` Jeff Law
@ 2017-12-01 16:19     ` Paolo Carlini
  0 siblings, 0 replies; 26+ messages in thread
From: Paolo Carlini @ 2017-12-01 16:19 UTC (permalink / raw)
  To: Jeff Law, Koval, Julia, GCC Patches

Hi,

On 01/12/2017 16:43, Jeff Law wrote:
> On 12/01/2017 03:28 AM, Paolo Carlini wrote:
>> Hi,
>>
>> On 16/11/2017 16:33, Koval, Julia wrote:
>>> // I failed to send patch itself, it is too big even in gzipped form.
>>> What is the right way to send such big patches?
>>>
>>> Hi, this patch removes cilkplus. Ok for trunk?
>> Now that cilkplus is gone I suppose we should clean-up Bugzilla about
>> that. Shall I go ahead and essentially close all the bugs we got? As
>> WONTFIX or what else? Let's agree on something. In principle we could
>> keep the regressions for the sake of the existing release branches but I
>> think we got very, very, few of those and anyway I don't see who gonna
>> work on that...
> Not sure if we have a policy in this space or not.
>
> If we don't then my vote would be for CLOSE/WONTFIX now.  That seems to
> most accurately reflect state -- we're not going to be fixing any Cilk+
> stuff on the trunk or in the release branches.
Excellent, thanks Jeff. Thus, barring strong contrary opinions, I'll 
take care of that over the we.

Cheers,
Paolo.

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2017-11-16 15:48 [patch] remove cilk-plus Koval, Julia
                   ` (3 preceding siblings ...)
  2017-12-01 10:28 ` Paolo Carlini
@ 2018-03-02 16:37 ` Thomas Schwinge
  2018-03-02 16:43   ` Jeff Law
  4 siblings, 1 reply; 26+ messages in thread
From: Thomas Schwinge @ 2018-03-02 16:37 UTC (permalink / raw)
  To: GCC Patches; +Cc: Koval, Julia, law

Hi!

On Thu, 16 Nov 2017 15:33:40 +0000, "Koval, Julia" <julia.koval@intel.com> wrote:
> Hi, this patch removes cilkplus.

I noticed a few remaining bits, that I convinced myself are obvious
enough; committed to trunk in r258141:

commit bd571ec47012c4ee50ef028024276ab02f5c15ec
Author: tschwinge <tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4>
Date:   Fri Mar 2 16:35:26 2018 +0000

    More Cilk Plus removal
    
            contrib/
            * update-copyright.py: Remove "libcilkrts" reference.
            gcc/c-family/
            * c-attribs.c (c_common_attribute_table): Remove "cilk simd
            function".
            gcc/
            * doc/invoke.texi: Remove "Cilk Plus" references.
    
    git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@258141 138bc75d-0d04-0410-961f-82ee72b054a4
---
 contrib/ChangeLog           | 4 ++++
 contrib/update-copyright.py | 3 +--
 gcc/ChangeLog               | 4 ++++
 gcc/c-family/ChangeLog      | 5 +++++
 gcc/c-family/c-attribs.c    | 2 --
 gcc/doc/invoke.texi         | 4 ++--
 6 files changed, 16 insertions(+), 6 deletions(-)

diff --git contrib/ChangeLog contrib/ChangeLog
index 627cafd..6bb3b6f 100644
--- contrib/ChangeLog
+++ contrib/ChangeLog
@@ -1,3 +1,7 @@
+2018-03-02  Thomas Schwinge  <thomas@codesourcery.com>
+
+	* update-copyright.py: Remove "libcilkrts" reference.
+
 2018-02-19  Yury Gribov  <tetra2005@gmail.com>
 
 	* compare_tests: Use TMPDIR when set.
diff --git contrib/update-copyright.py contrib/update-copyright.py
index 85be1e1..9295c6b 100755
--- contrib/update-copyright.py
+++ contrib/update-copyright.py
@@ -1,6 +1,6 @@
 #!/usr/bin/python
 #
-# Copyright (C) 2013-2017 Free Software Foundation, Inc.
+# Copyright (C) 2013-2018 Free Software Foundation, Inc.
 #
 # This script is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -708,7 +708,6 @@ class GCCCmdLine (CmdLine):
         self.add_dir ('libatomic')
         self.add_dir ('libbacktrace')
         self.add_dir ('libcc1')
-        # libcilkrts is imported from upstream.
         self.add_dir ('libcpp', LibCppFilter())
         self.add_dir ('libdecnumber')
         # libffi is imported from upstream.
diff --git gcc/ChangeLog gcc/ChangeLog
index 447d563..662b72f 100644
--- gcc/ChangeLog
+++ gcc/ChangeLog
@@ -1,3 +1,7 @@
+2018-03-02  Thomas Schwinge  <thomas@codesourcery.com>
+
+	* doc/invoke.texi: Remove "Cilk Plus" references.
+
 2018-03-02  Jakub Jelinek  <jakub@redhat.com>
 	    Richard Biener  <rguenther@suse.de>
 
diff --git gcc/c-family/ChangeLog gcc/c-family/ChangeLog
index c10b7bf..27cc43e 100644
--- gcc/c-family/ChangeLog
+++ gcc/c-family/ChangeLog
@@ -1,3 +1,8 @@
+2018-03-02  Thomas Schwinge  <thomas@codesourcery.com>
+
+	* c-attribs.c (c_common_attribute_table): Remove "cilk simd
+	function".
+
 2018-03-01  Marek Polacek  <polacek@redhat.com>
 
 	PR c++/84639
diff --git gcc/c-family/c-attribs.c gcc/c-family/c-attribs.c
index 3ebb2d6..cebc0b2 100644
--- gcc/c-family/c-attribs.c
+++ gcc/c-family/c-attribs.c
@@ -439,8 +439,6 @@ const struct attribute_spec c_common_attribute_table[] =
 			      handle_returns_nonnull_attribute, NULL },
   { "omp declare simd",       0, -1, true,  false, false, false,
 			      handle_omp_declare_simd_attribute, NULL },
-  { "cilk simd function",     0, -1, true,  false, false, false,
-			      handle_omp_declare_simd_attribute, NULL },
   { "simd",		      0, 1, true,  false, false, false,
 			      handle_simd_attribute, NULL },
   { "omp declare target",     0, 0, true, false, false, false,
diff --git gcc/doc/invoke.texi gcc/doc/invoke.texi
index 8d366c6..ad0fec6 100644
--- gcc/doc/invoke.texi
+++ gcc/doc/invoke.texi
@@ -6710,7 +6710,7 @@ Requires @option{-flto-odr-type-merging} to be enabled.  Enabled by default.
 
 @item -Wopenmp-simd
 @opindex Wopenm-simd
-Warn if the vectorizer cost model overrides the OpenMP or the Cilk Plus
+Warn if the vectorizer cost model overrides the OpenMP
 simd directive set by user.  The @option{-fsimd-cost-model=unlimited}
 option can be used to relax the cost model.
 
@@ -8953,7 +8953,7 @@ either @samp{dynamic} or @samp{cheap}.
 @item -fsimd-cost-model=@var{model}
 @opindex fsimd-cost-model
 Alter the cost model used for vectorization of loops marked with the OpenMP
-or Cilk Plus simd directive.  The @var{model} argument should be one of
+simd directive.  The @var{model} argument should be one of
 @samp{unlimited}, @samp{dynamic}, @samp{cheap}.  All values of @var{model}
 have the same meaning as described in @option{-fvect-cost-model} and by
 default a cost model defined with @option{-fvect-cost-model} is used.


Grüße
 Thomas

^ permalink raw reply	[flat|nested] 26+ messages in thread

* Re: [patch] remove cilk-plus
  2018-03-02 16:37 ` Thomas Schwinge
@ 2018-03-02 16:43   ` Jeff Law
  0 siblings, 0 replies; 26+ messages in thread
From: Jeff Law @ 2018-03-02 16:43 UTC (permalink / raw)
  To: Thomas Schwinge, GCC Patches; +Cc: Koval, Julia

On 03/02/2018 09:36 AM, Thomas Schwinge wrote:
> Hi!
> 
> On Thu, 16 Nov 2017 15:33:40 +0000, "Koval, Julia" <julia.koval@intel.com> wrote:
>> Hi, this patch removes cilkplus.
> 
> I noticed a few remaining bits, that I convinced myself are obvious
> enough; committed to trunk in r258141:
> 
> commit bd571ec47012c4ee50ef028024276ab02f5c15ec
> Author: tschwinge <tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4>
> Date:   Fri Mar 2 16:35:26 2018 +0000
> 
>     More Cilk Plus removal
>     
>             contrib/
>             * update-copyright.py: Remove "libcilkrts" reference.
>             gcc/c-family/
>             * c-attribs.c (c_common_attribute_table): Remove "cilk simd
>             function".
>             gcc/
>             * doc/invoke.texi: Remove "Cilk Plus" references.
Thanks.  Yea, this certainly qualifies as obvious :-)

jeff

^ permalink raw reply	[flat|nested] 26+ messages in thread

end of thread, other threads:[~2018-03-02 16:43 UTC | newest]

Thread overview: 26+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-16 15:48 [patch] remove cilk-plus Koval, Julia
2017-11-16 15:49 ` Marek Polacek
2017-11-16 16:21 ` Jakub Jelinek
2017-11-16 17:06   ` Koval, Julia
2017-11-16 23:51     ` Joseph Myers
2017-11-21  7:46     ` Jeff Law
2017-11-22  8:47       ` Koval, Julia
2017-11-22  9:15         ` Rainer Orth
2017-11-22  9:22           ` Koval, Julia
2017-11-22 10:01             ` Koval, Julia
2017-11-22 17:33         ` Joseph Myers
2017-11-23 10:01           ` Koval, Julia
2017-11-27 18:06             ` Jeff Law
2017-11-30 11:55               ` Koval, Julia
2017-11-30 12:02                 ` Jakub Jelinek
2017-11-30 16:40                 ` Marek Polacek
2017-11-30 17:27                   ` Jeff Law
2017-11-30 17:29                     ` Jakub Jelinek
2017-11-23 17:58         ` Jeff Law
2017-11-16 16:34 ` Eric Gallager
2017-11-16 17:59   ` Jeff Law
2017-12-01 10:28 ` Paolo Carlini
2017-12-01 15:43   ` Jeff Law
2017-12-01 16:19     ` Paolo Carlini
2018-03-02 16:37 ` Thomas Schwinge
2018-03-02 16:43   ` Jeff Law

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).