I forgot to mention that the change was approved by Jian-Xin Lai.

Thanks,

Min

On Tue, Oct 5, 2010 at 1:27 PM, <s...@open64.net> wrote:

> Author: minz
> Date: 2010-10-05 16:27:56 -0400 (Tue, 05 Oct 2010)
> New Revision: 3366
>
> Modified:
>   trunk/osprey/wgen/wgen_expr.cxx
> Log:
> Fix for bug664:
>
> The assertion is because incorrect IR is generated for  comma expression.
> The root cause for the problem is in WGEN_Expand_Expr, when processing
> GS_COMPOUND_EXPR case, it does not pass the correct information
> (e.g., field_id) to the following WGEN_Expand_Expr call (for its child).
>
>
>
> Modified: trunk/osprey/wgen/wgen_expr.cxx
> ===================================================================
> --- trunk/osprey/wgen/wgen_expr.cxx     2010-10-04 17:46:27 UTC (rev 3365)
> +++ trunk/osprey/wgen/wgen_expr.cxx     2010-10-05 20:27:56 UTC (rev 3366)
> @@ -9810,7 +9810,8 @@
>  #ifdef KEY
>         // bug 11238: pass on the target
>         wn = WGEN_Expand_Expr (gs_tree_operand (exp, 1), need_result,
> -                               0, 0, 0, 0, FALSE, FALSE, target_wn);
> +                               nop_ty_idx, component_ty_idx,
> component_offset,
> +                               field_id, is_bit_field,
> is_aggr_init_via_ctor, target_wn);
>  #else
>         wn = WGEN_Expand_Expr (gs_tree_operand (exp, 1), need_result);
>  #endif
>
>
>
> ------------------------------------------------------------------------------
> Beautiful is writing same markup. Internet Explorer 9 supports
> standards for HTML5, CSS3, SVG 1.1,  ECMAScript5, and DOM L2 & L3.
> Spend less time writing and  rewriting code and more time creating great
> experiences on the web. Be a part of the beta today.
> http://p.sf.net/sfu/beautyoftheweb
> _______________________________________________
> Open64-devel mailing list
> Open64-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/open64-devel
>
------------------------------------------------------------------------------
Beautiful is writing same markup. Internet Explorer 9 supports
standards for HTML5, CSS3, SVG 1.1,  ECMAScript5, and DOM L2 & L3.
Spend less time writing and  rewriting code and more time creating great
experiences on the web. Be a part of the beta today.
http://p.sf.net/sfu/beautyoftheweb
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel

Reply via email to