RE: [PATCH][Cilkplus] Remove unwanted static chain.

["Iyer, Balaji V" <balaji dot v dot iyer at intel dot com>]

Hello Aldy,
	The only time we will get into this function (cp_parser_cilk_for) is when the fcilkplus is turned on.

Here is the original call for this function (line #9983) :

    if (!flag_enable_cilk)
        fatal_error ("-fcilkplus must be enabled to use %<cilk_for%>");
      else
        statement = cp_parser_cilk_for (parser, (tree) NULL_TREE, parser->in_statement);


Thanks,

Balaji V. Iyer.

-----Original Message-----
From: Aldy Hernandez [mailto:aldyh@redhat.com] 
Sent: Wednesday, August 08, 2012 2:23 PM
To: Iyer, Balaji V
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH][Cilkplus] Remove unwanted static chain.


> Index: gcc/cp/parser.c
> ===================================================================
> --- gcc/cp/parser.c	(revision 190195)
> +++ gcc/cp/parser.c	(working copy)
> @@ -28351,6 +28351,13 @@
>    FOR_EXPR (statement) = decl;
>    CILK_FOR_GRAIN (statement) = grain;
>  
> +  /*  If an initial value is available, and it is of type integer, then we
> +      save it in CILK_FOR_INIT.  */
> +  if (init && TREE_TYPE (init) && INTEGRAL_TYPE_P (TREE_TYPE (init))) 
> +    CILK_FOR_INIT (statement) = init;  else
> +    CILK_FOR_INIT (statement) = NULL_TREE;
> +

Shouldn't you only set this for flag_cilkplus (?), or does it need to be set for non Cilk instances of the compiler?