[patch, testsuite] Fix PR 49191 with test for STRICT_ALIGNMENT

Jakub Jelinek jakub@redhat.com
Wed Jun 8 20:45:00 GMT 2011


On Wed, Jun 08, 2011 at 01:16:26PM -0700, Steve Ellcey wrote:
> On Wed, 2011-06-08 at 22:08 +0200, Jakub Jelinek wrote:
> > On Wed, Jun 08, 2011 at 01:03:53PM -0700, Steve Ellcey wrote:
> > > 2011-06-08  Steve Ellcey  <sje@cup.hp.com>
> > > 
> > > 	PR middle-end/49191
> > > 	* lib/target-supports.exp (check_effective_target_strict_align): New.
> > > 	* gcc.dg/memcpy-3.c: Add dg-require-effective-target strict_align.
> > 
> > Isn't the test backwards, i.e. doesn't
> > { dg-require-effective-target strict_align }
> > actually mean !STRICT_ALIGNMENT?
> 
> You are right.  What do you think I should do, reverse the test or just
> rename it?  I would be inclined to just rename it, maybe:
> 
> dg-require-effective-target non_strict_align

Yeah, either that, or reverse the test and then
{ dg-do run { target { !strict_align } } } */
(if that is the right syntax).

	Jakub



More information about the Gcc-patches mailing list