[PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Zamyatin, Igor
Hi! This patches fixes the ICE when array notation is used along with Cilk_spawn. We need to expand AN's loop using other routine for creating temporary variables. Bootstrapped/regtested on x86_64. Also it introduces no new failures and fixes couple of ICE for CilkPlus Conformance suite

Re: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Rainer Orth
Zamyatin, Igor igor.zamya...@intel.com writes: diff --git a/gcc/testsuite/c-c++-common/cilk-plus/CK/pr60469.c b/gcc/testsuite/c-c++-common/cilk-plus/CK/pr60469.c new file mode 100644 index 000..61e8014 --- /dev/null +++ b/gcc/testsuite/c-c++-common/cilk-plus/CK/pr60469.c @@ -0,0 +1,15

Re: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Jakub Jelinek
On Thu, Apr 10, 2014 at 05:19:33PM +, Zamyatin, Igor wrote: @@ -282,8 +283,7 @@ fix_builtin_array_notation_fn (tree an_builtin_fn, tree *new_var) for (ii = 0; ii rank; ii++) { - an_loop_info[ii].var = build_decl (location, VAR_DECL, NULL_TREE, -

RE: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Zamyatin, Igor
, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together On Thu, Apr 10, 2014 at 05:19:33PM +, Zamyatin, Igor wrote: @@ -282,8 +283,7 @@ fix_builtin_array_notation_fn (tree an_builtin_fn, tree *new_var) for (ii = 0; ii rank; ii++) { - an_loop_info[ii].var

Re: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Jakub Jelinek
On Thu, Apr 10, 2014 at 06:16:22PM +, Zamyatin, Igor wrote: Fixed patch is below. Just out of curiosity - why do you think this patch is not for 4.9? The one with just create_tmp_var (integer_type_node, NULL); is for 4.9. For the iterator var changes, I guess one needs to analyze all the

RE: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Zamyatin, Igor
-Original Message- From: Jakub Jelinek [mailto:ja...@redhat.com] Sent: Thursday, April 10, 2014 10:21 PM To: Zamyatin, Igor Cc: GCC Patches (gcc-patches@gcc.gnu.org); Iyer, Balaji V Subject: Re: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

Re: [PATCH, PR60469, Cilk+] Fix ICE of using Cilk_spawn and Array Notation together

2014-04-10 Thread Jakub Jelinek
On Thu, Apr 10, 2014 at 07:55:26PM +, Zamyatin, Igor wrote: So below is the patch for 4.9 trunk. Ok, thanks. Thanks, Igor gcc/c/Changelog: 2014-04-10 Igor Zamyatin igor.zamya...@intel.com PR middle-end/60469 * c-array-notation.c (fix_builtin_array_notation_fn):