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
>
> PR middle-end/60469
> * c-array-notation.c (fix_builtin_array_notation_fn): Use
> cr
> -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
&
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 th
fine ALEN 1024
+
+int main(int argc, char* argv[])
+{
+ int b[ALEN];
+ b[:] = 100;
+ _Cilk_spawn foo();
+ return 0;
+
> -Original Message-
> From: Jakub Jelinek [mailto:ja...@redhat.com]
> Sent: Thursday, April 10, 2014 9:29 PM
> To: Zamyatin, Igor
> Cc: GCC Patches (g
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,
> -
"Zamyatin, Igor" 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 @@
> +/* middl
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 which