Comments

Unless I get an objection I will check in this testsuite patch as
obvious. The test includes stdint.h and uses uint32_t so it should only
be run on targets that support the stdint types.
Tested on IA64 HP-UX and Linux.
Steve Ellcey
sje@cup.hp.com
2010-09-02 Steve Ellcey <sje@cup.hp.com>
* gcc.dg/torture/pr44806.c: Require stdint_types.

On Thu, 2010-09-02 at 09:29 -0700, Andrew Pinski wrote:
> Shouldn't target stdint_types be true now for all targets? Sinc
I think stdint.h should exist for all targets now but I am not sure if
the types like uint32_t will always exist. The failure I am getting
is because uint32_t is not defined.
It looks like (at least on IA64 HP-UX) uint32_t is only defined if I
specify -std=c99.
Steve Ellcey
sje@cup.hp.com

On 09/02/2010 09:37 AM, Steve Ellcey wrote:
> On Thu, 2010-09-02 at 09:29 -0700, Andrew Pinski wrote:>> Shouldn't target stdint_types be true now for all targets? Sinc> > I think stdint.h should exist for all targets now but I am not sure if> the types like uint32_t will always exist. The failure I am getting> is because uint32_t is not defined.> > It looks like (at least on IA64 HP-UX) uint32_t is only defined if I> specify -std=c99.
The test case would not be hurt by either adding -std=c99 to dg-options,
or switching the types to "unsigned" and "unsigned long long".
I'm fine with either of those changes.
r~

On Thu, 2010-09-02 at 12:42 -0700, Richard Henderson wrote:
> On 09/02/2010 09:37 AM, Steve Ellcey wrote:> > On Thu, 2010-09-02 at 09:29 -0700, Andrew Pinski wrote:> >> Shouldn't target stdint_types be true now for all targets? Sinc> > > > I think stdint.h should exist for all targets now but I am not sure if> > the types like uint32_t will always exist. The failure I am getting> > is because uint32_t is not defined.> > > > It looks like (at least on IA64 HP-UX) uint32_t is only defined if I> > specify -std=c99.> > The test case would not be hurt by either adding -std=c99 to dg-options,> or switching the types to "unsigned" and "unsigned long long".> > I'm fine with either of those changes.> > > r~
I checked in a patch to add -std=c99.
Steve Ellcey
sje@cup.hp.com