Re: Handle non-SSA arguments in ipa-inline-analysis better

2011-09-25 Thread Richard Guenther
On Sun, Sep 25, 2011 at 12:25 PM, Jan Hubicka wrote: >> >> You don't care for a NULL get_base_address return in the other >> place you added it.  I think you should instead bail out on a >> NULL return from said. >> >> Note that get_base_address will not strip things down to an >> SSA name pointer

Re: Handle non-SSA arguments in ipa-inline-analysis better

2011-09-25 Thread Jan Hubicka
> > You don't care for a NULL get_base_address return in the other > place you added it. I think you should instead bail out on a > NULL return from said. > > Note that get_base_address will not strip things down to an > SSA name pointer but will return a MEM[ptr, offset] for that. > But I suppo

Re: Handle non-SSA arguments in ipa-inline-analysis better

2011-09-24 Thread Richard Guenther
On Thu, Sep 22, 2011 at 12:51 AM, Jan Hubicka wrote: > Hi, > this patch extends handling of non-SSA arguments to bultin_constant_p and > execution predicates. > > Bootstrapped/regtested x86_64-linux, will commit it shortly. > > Honza > >        * ipa-inline-analysis.c (set_cond_stmt_execution_pred

Handle non-SSA arguments in ipa-inline-analysis better

2011-09-21 Thread Jan Hubicka
Hi, this patch extends handling of non-SSA arguments to bultin_constant_p and execution predicates. Bootstrapped/regtested x86_64-linux, will commit it shortly. Honza * ipa-inline-analysis.c (set_cond_stmt_execution_predicate): Allow handled components in parameter of builtin_con