On Thu, 14 May 2015, Tom de Vries wrote:
> On 12-05-15 12:04, Richard Biener wrote:
> > > The fact that we have to handle this specially in both build_va_arg and
> > > >gimplify_va_arg makes me wonder whether we should be dealing with all
> > > this in
> > > >build_va_arg already.
> > > >
> > > >T
On 12-05-15 12:04, Richard Biener wrote:
The fact that we have to handle this specially in both build_va_arg and
>gimplify_va_arg makes me wonder whether we should be dealing with all this in
>build_va_arg already.
>
>That is, determine whether we take the address, and add the address operator
>i
On Wed, 13 May 2015, Andreas Krebbel wrote:
> On 05/12/2015 01:45 PM, Tom de Vries wrote:
> > On 12-05-15 12:04, Tom de Vries wrote:
> >> Committed with comments below added.
> >
> > Hmm, this causes an ice for s390 in gcc.dg/tree-ssa/stdarg-2.c:
> > ...
> > gimplification failed:
> > &ap >
On 05/12/2015 01:45 PM, Tom de Vries wrote:
> On 12-05-15 12:04, Tom de Vries wrote:
>> Committed with comments below added.
>
> Hmm, this causes an ice for s390 in gcc.dg/tree-ssa/stdarg-2.c:
> ...
> gimplification failed:
> &ap type type 0x7f132f46b888 __va_list_tag>
>
On 05/12/2015 06:45 AM, Tom de Vries wrote:
Hmm, this causes an ice for s390 in gcc.dg/tree-ssa/stdarg-2.c:
...
gimplification failed:
&ap type 0x7f132f46b888 __va_list_tag>
sizes-gimplified BLK
size
unit size
align 32 symtab 0 alias set
On 12-05-15 12:04, Tom de Vries wrote:
Committed with comments below added.
Hmm, this causes an ice for s390 in gcc.dg/tree-ssa/stdarg-2.c:
...
gimplification failed:
&ap type 0x7f132f46b888 __va_list_tag>
sizes-gimplified BLK
size
unit size
On 12-05-15 12:12, Michael Matz wrote:
Hi,
On Fri, 8 May 2015, Tom de Vries wrote:
III.
Using the patch, before inlining we can see the address operator has been
removed in va_arg:
...
f2_1 (struct * apD.1832)
{
intD.6 _4;
# .MEM_3 = VDEF <.MEM_1(D)>
# USE = anything
# CLB = any
Hi,
On Fri, 8 May 2015, Tom de Vries wrote:
> III.
>
> Using the patch, before inlining we can see the address operator has been
> removed in va_arg:
> ...
> f2_1 (struct * apD.1832)
> {
> intD.6 _4;
>
> # .MEM_3 = VDEF <.MEM_1(D)>
> # USE = anything
> # CLB = anything
>
> _4 = VA_A
On Tue, 12 May 2015, Tom de Vries wrote:
> On 12-05-15 09:45, Richard Biener wrote:
> > On Mon, 11 May 2015, Tom de Vries wrote:
> >
> > > On 11-05-15 09:47, Richard Biener wrote:
> > > > > Bootstrapped and reg-tested on x86_64, with and without -m32.
> > > > > >
> > > > > > OK for trunk?
> > >
On 12-05-15 09:45, Richard Biener wrote:
On Mon, 11 May 2015, Tom de Vries wrote:
On 11-05-15 09:47, Richard Biener wrote:
Bootstrapped and reg-tested on x86_64, with and without -m32.
OK for trunk?
[ FWIW, I suspect this patch will make life easier for the
reimplementation of
the pass_st
On Mon, 11 May 2015, Tom de Vries wrote:
> On 11-05-15 09:47, Richard Biener wrote:
> > > Bootstrapped and reg-tested on x86_64, with and without -m32.
> > > >
> > > >OK for trunk?
> > > >
> > > >[ FWIW, I suspect this patch will make life easier for the
> > > reimplementation of
> > > >the pass_s
On 11-05-15 09:47, Richard Biener wrote:
Bootstrapped and reg-tested on x86_64, with and without -m32.
>
>OK for trunk?
>
>[ FWIW, I suspect this patch will make life easier for the reimplementation of
>the pass_stdarg optimization using ifn_va_arg. ]
+ if (canon_va_type != NULL)
+{
+
On Fri, 8 May 2015, Tom de Vries wrote:
> Hi,
>
> this patch fixes PR66010.
>
>
> I.
>
> Consider this test-case, with a va_list passed from f2 to f2_1:
> ...
> #include
>
> inline int __attribute__((always_inline))
> f2_1 (va_list ap)
> {
> return va_arg (ap, int);
> }
>
> int
> f2 (int
Hi,
this patch fixes PR66010.
I.
Consider this test-case, with a va_list passed from f2 to f2_1:
...
#include
inline int __attribute__((always_inline))
f2_1 (va_list ap)
{
return va_arg (ap, int);
}
int
f2 (int i, ...)
{
int res;
va_list ap;
va_start (ap, i);
res = f2_1 (ap);
v
14 matches
Mail list logo
