On 06/11/2012 12:15 PM, H.J. Lu wrote: > On Mon, Jun 11, 2012 at 12:00 PM, Janis Johnson > <janis_john...@mentor.com> wrote: >> The tests in gcc.dg/torture/stackalign use two sets of torture options: >> the usual optimization sets used as default for torture tests, and up to >> four sets of options that are specific to stack alignment. The stack >> alignment options are passed via an option that is used by the dg-test >> option dg_extra_tool_flags, which can be overridden by dg-options. This >> means that the seven tests in that test directory that use dg-option do >> not use the other alignment options. All seven of these tests are >> limited to x86 targets. Four of them use -msse2, and four use different >> values for -mpreferred-stack-boundary (one does both), so it doesn't >> look as if they are intended to use the four sets of stackalign torture >> options. >> >> This patch moves those seven tests out of the stackalign directory up to >> the general gcc.dg/torture directory. With them out of the way I'll be >> able to use clean up the remaining stackalign tests to use torture test >> support to combine the stack align options with other torture options so >> they'll show up in test summary lines, eliminating lots of duplicate >> lines in test summaries. >> >> Tested on i686-pc-linux-gnu and arm-none-eabi. OK for mainline? >> >> 2012-06-11 Janis Johnson <jani...@codesourcery.com> >> >> * gcc.dg/torture/stackalign/alloca-2.c: Move to ... >> * gcc.dg/torture/alloca-2.c: ... here. >> * gcc.dg/torture/stackalign/alloca-3.c: Move to ... >> * gcc.dg/torture/alloca-3.c: ... here. >> * gcc.dg/torture/stackalign/alloca-4.c: Move to ... >> * gcc.dg/torture/alloca-4.c: ... here. >> * gcc.dg/torture/stackalign/alloca-5.c: Move to ... >> * gcc.dg/torture/alloca-5.c: ... here. >> * gcc.dg/torture/stackalign/alloca-6.c: Move to ... >> * gcc.dg/torture/alloca-6.c: ... here. >> * gcc.dg/torture/stackalign/push-1.c: Move to ... >> * gcc.dg/torture/push-1.c: ... here. >> * gcc.dg/torture/stackalign/vararg-3.c: Move to ... >> * gcc.dg/torture/vararg-3.c: ... here. > > stackalign.exp has > > gcc-dg-runtest [lsort [glob $srcdir/$subdir/*.c]] $additional_flags > if { [check_effective_target_fpic] } then { > set pic_additional_flags $additional_flags > lappend pic_additional_flags "-fpic" > gcc-dg-runtest [lsort [glob $srcdir/$subdir/*.c]] $pic_additional_flags > } > > It adds PIC tests. Will this change remove PIC tests? > >
The options for PIC tests are overridden by dg-options. This change will make no difference at all to how these 7 tests are compiled. They are currently compiled four times with exactly the same options. Janis