[PATCH 6/6 v4]: Merge from Stack Branch - Test Cases

Janis Johnson janis187@us.ibm.com
Thu Jun 5 22:56:00 GMT 2008


On Thu, 2008-06-05 at 16:03 +0800, Ye, Joey wrote:
> Updated version.
> 
> 2008-06-05  H.J. Lu  <hongjiu.lu@intel.com>
> 	    Joey Ye  <joey.ye@intel.com>
> 
> 	* gcc.target/i386/20060512-1.c: Add
>         -mpreferred-stack-boundary=4.
> 	(main): Move "popl" after check.
> 	* gcc.target/i386/20060512-3.c: Likewise.
[lots more]

When a test has been copied from another part of the testsuite, add a
comment giving the path of the original.

Avoid using C library functions and their headers whenever possible.
For example, instead of including <string.h> and calling strncpy, call
__builtin_strncpy.

Add comments to tests in which the purpose isn't clear, as with
  gcc.target/i386/stackalign/asm-1.c
  gcc.target/i386/stackalign/return-2.c
They compile with no additional checks; did they cause ICEs in earlier
versions on the branch?

Torture option handling was cleaned up recently, and it should be
possible to make gcc.dg/torture/stackalign/stackalign.exp much
simpler by combining the existing C_TORTURE_OPTIONS with your
sets of additional options.

Just FYI (I don't mind if it stays the way it is), this:

  /* { dg-do run } */
  /* { dg-require-effective-target ilp32 } */
  /* { dg-require-effective-target dfp } */

is the same as this:

  /* { dg-do run { target { ilp32 && dfp } } } */

or this:

  /* { dg-do run } */
  /* { dg-require-effective-target { ilp32 && dfp } } */

Unless there are other objections you can add these tests when the
functionality has been committed.

Janis




More information about the Gcc-patches mailing list