Message ID | 001d01cdb001$59be9310$0d3bb930$@cheng@arm.com |
---|---|
State | New |
Headers | show |
On Mon, Oct 22, 2012 at 11:00:08AM +0800, Bin Cheng wrote: > The test case "gcc/testsuite/gcc.dg/hoist-register-pressure.c" is failed on > x86_64-apple-darwin because it uses more registers than x86_64-linux. This > can be fixed by simplifying the case using fewer registers. > > Tested on x86_64-apple-darwin/x86_64-linux, is it OK? I'd say it is better to do the scan-rtl-dump only on nonpic targets, that way it won't be done on darwin or for testing with --target_board=unix/-fpic where it would fail too. You can add the test with smaller register pressure as a new test (hoist-register-pressure2.c). Jakub
Index: gcc/testsuite/gcc.dg/hoist-register-pressure.c =================================================================== --- gcc/testsuite/gcc.dg/hoist-register-pressure.c (revision 192604) +++ gcc/testsuite/gcc.dg/hoist-register-pressure.c (working copy) @@ -13,13 +13,14 @@ because its rtx_cost is too small. */ if (z) { - a[1] = a[0] + a[2]; - a[2] = a[1] + a[3]; - a[3] = a[2] + a[4]; - a[4] = a[3] + a[5]; - a[5] = a[4] + a[6]; - a[6] = a[5] + a[7]; - a[7] = a[6] + a[8]; + a[1] = a[0]; + a[2] = a[1]; + a[3] = a[3]; + a[4] = a[5]; + a[5] = a[7]; + a[6] = a[11]; + a[7] = a[13]; + a[8] = a[17]; com (x+y); } else