On Wed, Aug 13, 2014 at 4:17 PM, Ilya Enkovich <enkovich....@gmail.com> wrote: > Hi, > > Testing 'mpx' branch after merge with trunk I got a segfault in ipa-devirt.c. > It appears that cgraph_node cloning with indirect edge causes call to > get_polymorphic_call_info which uses current_function_decl. It happens in > IPA pass and therefore current_function_decl is NULL which causes segfault. > Also even within a GIMPLE pass it seems wrong to use current_function_decl > because examined call may belong to another function and passed fndecl should > be used instead. > > Proposed patch was bootstrapped and regtested on linux-x86_64. OK for trunk?
Ok. Thanks, Richard. > Thanks, > Ilya > -- > > 2014-08-13 Ilya Enkovich <ilya.enkov...@intel.com> > > * ipa-devirt.c (get_polymorphic_call_info): Use fndecl instead of > current_function_decl. > > > diff --git a/gcc/ipa-devirt.c b/gcc/ipa-devirt.c > index 3650b43..0f38655 100644 > --- a/gcc/ipa-devirt.c > +++ b/gcc/ipa-devirt.c > @@ -2319,7 +2319,7 @@ get_polymorphic_call_info (tree fndecl, > = decl_maybe_in_construction_p (base, > context->outer_type, > call, > - current_function_decl); > + fndecl); > return base; > } > else
