On 8/20/07, Andrew Pinski <[EMAIL PROTECTED]> wrote: > On 8/20/07, Thomas Veith <[EMAIL PROTECTED]> wrote: > > Hi, > > > > when bootstrapping gcc trunk rev. 127646 with > > > > ./contrib/gcc_build -c "--prefix=/home/xtv > > --enable-languages=c,c++,objc,fortran,obj-c++ --enable-bootstrap > > --enable-decimal-float=bid --enable-checking" -d /home/xtv/gcc-devel/gcc > > -o obj bootstrap > > This was caused by: > 2007-08-20 Jakub Jelinek <[EMAIL PROTECTED]> > > PR c++/33025 > * init.c (build_new_1): Rename placement_var variable to > placement_expr. > Initialize it with save_expr rather than get_temp_regvar. > > > The statement which we are gimplifying: > <<<change_dynamic_type (struct __recursive_mutex *) &fake_mutex)>>> > > 5887 case CHANGE_DYNAMIC_TYPE_EXPR: > 5888 ret = gimplify_expr (&CHANGE_DYNAMIC_TYPE_LOCATION > (*expr_p), > 5889 pre_p, post_p, is_gimple_reg, > fb_lvalue); > > > So why fb_lvalue?
Actually this is correct, we need a decl here as we need to mark that for pointing to anything. Jakub, how did you test your patch again? Thanks, Andrew Pinski
