On Fri, Oct 5, 2012 at 7:19 AM, Jakub Jelinek wrote:
> On Fri, Oct 05, 2012 at 03:59:55PM +0200, Richard Guenther wrote:
>> I don't think we want to rely on that ... so just keep the push/pop_cfun.
>
> Ok, so this is what I'm retesting (basically just comments added and the two
> lines (subcode an
On Fri, Oct 05, 2012 at 03:59:55PM +0200, Richard Guenther wrote:
> I don't think we want to rely on that ... so just keep the push/pop_cfun.
Ok, so this is what I'm retesting (basically just comments added and the two
lines (subcode and set) swapped:
2012-10-05 Jakub Jelinek
PR debug
On Fri, Oct 5, 2012 at 2:49 PM, Jakub Jelinek wrote:
> On Fri, Oct 05, 2012 at 02:20:13PM +0200, Richard Guenther wrote:
>> The following could use a comment on what you are doing ...
>
> Will add something.
>
>> > + if (args_to_skip)
>> > +for (parm = DECL_ARGUMENTS (current_function_decl),
On Fri, Oct 05, 2012 at 02:49:07PM +0200, Jakub Jelinek wrote:
> I believe that gsi_insert_before in another function
> isn't going to work well.
> E.g. update_modified_stmt starts with
> if (!ssa_operands_active (cfun))
> return;
>
> Or is it ok to use gsi_insert_before_without_update and e
On Fri, Oct 05, 2012 at 02:20:13PM +0200, Richard Guenther wrote:
> The following could use a comment on what you are doing ...
Will add something.
> > + if (args_to_skip)
> > +for (parm = DECL_ARGUMENTS (current_function_decl), num = 0;
> > +parm; parm = DECL_CHAIN (parm), num++)
> > +
On Wed, 3 Oct 2012, Jakub Jelinek wrote:
> On Tue, Sep 11, 2012 at 03:59:56PM +0200, Jakub Jelinek wrote:
> > As discussed in the PR, right now we do a very bad job for debug info
> > of partially inlined functions (both when they are kept only partially
> > inlined, or when partial inlining is pe
On Thu, Oct 04, 2012 at 01:31:08AM -0300, Alexandre Oliva wrote:
> On Oct 3, 2012, Jakub Jelinek wrote:
>
> > basically there is a non-addressable parameter in stack slot, and
> > vt_canon_true_dep -> canon_true_dependence thinks an argument push
> > insn might alias with it, because it doesn't
On Oct 3, 2012, Jakub Jelinek wrote:
> basically there is a non-addressable parameter in stack slot, and
> vt_canon_true_dep -> canon_true_dependence thinks an argument push
> insn might alias with it, because it doesn't have a MEM_EXPR and
> ao_ref_from_mem fails.
I have a pending (still unrev
On Tue, Sep 11, 2012 at 04:41:24PM +0200, Steven Bosscher wrote:
> > + if (args_to_skip)
> > +for (parm = DECL_ARGUMENTS (current_function_decl), num = 0;
> > +parm; parm = DECL_CHAIN (parm), num++)
> > + if (bitmap_bit_p (args_to_skip, num)
> > + && is_gimple_reg (parm))
> + if (args_to_skip)
> +for (parm = DECL_ARGUMENTS (current_function_decl), num = 0;
> +parm; parm = DECL_CHAIN (parm), num++)
> + if (bitmap_bit_p (args_to_skip, num)
> + && is_gimple_reg (parm))
> + {
> + tree ddecl;
> + gimple def_temp;
> +
> +
10 matches
Mail list logo
