* [PATCH]: Reduce array size in libgomp.graphite/force-parallel-2.c to fix PR42135
@ 2009-11-26 1:58 Jack Howarth
2009-11-30 17:45 ` Sebastian Pop
2009-12-16 17:37 ` [PATCH][[PING]: " Jack Howarth
0 siblings, 2 replies; 4+ messages in thread
From: Jack Howarth @ 2009-11-26 1:58 UTC (permalink / raw)
To: gcc-patches; +Cc: sebpop
The attached patch reduces the array size in the libgomp.graphite/force-parallel-2.c
testcase from 10000 to 500 so that it will fit into stacksize's as small as 2MB per
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42135#c2. This revised testcase now passes
on x86_64-apple-darwin10. Okay for gcc trunk?
Jack
2009-11-25 Jack Howarth <howarth@bromo.med.uc.edu>
PR testsuite/42135
* libgomp.graphite/force-parallel-2.c: Reduce array size.
Index: libgomp/testsuite/libgomp.graphite/force-parallel-2.c
===================================================================
--- libgomp/testsuite/libgomp.graphite/force-parallel-2.c (revision 154653)
+++ libgomp/testsuite/libgomp.graphite/force-parallel-2.c (working copy)
@@ -3,7 +3,7 @@
void parloop (int N)
{
int i, j;
- int x[10000][10000];
+ int x[500][500];
for (i = 0; i < N; i++)
for (j = 0; j < N; j++)
@@ -17,7 +17,7 @@
int main(void)
{
- parloop(10000);
+ parloop(500);
return 0;
}
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH]: Reduce array size in libgomp.graphite/force-parallel-2.c to fix PR42135
2009-11-26 1:58 [PATCH]: Reduce array size in libgomp.graphite/force-parallel-2.c to fix PR42135 Jack Howarth
@ 2009-11-30 17:45 ` Sebastian Pop
2009-12-16 17:37 ` [PATCH][[PING]: " Jack Howarth
1 sibling, 0 replies; 4+ messages in thread
From: Sebastian Pop @ 2009-11-30 17:45 UTC (permalink / raw)
To: Jack Howarth; +Cc: gcc-patches
On Wed, Nov 25, 2009 at 19:34, Jack Howarth <howarth@bromo.med.uc.edu> wrote:
> The attached patch reduces the array size in the libgomp.graphite/force-parallel-2.c
> testcase from 10000 to 500 so that it will fit into stacksize's as small as 2MB per
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42135#c2. This revised testcase now passes
> on x86_64-apple-darwin10. Okay for gcc trunk?
Yes.
Thanks,
Sebastian
> Jack
>
> 2009-11-25 Jack Howarth <howarth@bromo.med.uc.edu>
>
> PR testsuite/42135
> * libgomp.graphite/force-parallel-2.c: Reduce array size.
>
> Index: libgomp/testsuite/libgomp.graphite/force-parallel-2.c
> ===================================================================
> --- libgomp/testsuite/libgomp.graphite/force-parallel-2.c (revision 154653)
> +++ libgomp/testsuite/libgomp.graphite/force-parallel-2.c (working copy)
> @@ -3,7 +3,7 @@
> void parloop (int N)
> {
> int i, j;
> - int x[10000][10000];
> + int x[500][500];
>
> for (i = 0; i < N; i++)
> for (j = 0; j < N; j++)
> @@ -17,7 +17,7 @@
>
> int main(void)
> {
> - parloop(10000);
> + parloop(500);
>
> return 0;
> }
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH][[PING]: Reduce array size in libgomp.graphite/force-parallel-2.c to fix PR42135
2009-11-26 1:58 [PATCH]: Reduce array size in libgomp.graphite/force-parallel-2.c to fix PR42135 Jack Howarth
2009-11-30 17:45 ` Sebastian Pop
@ 2009-12-16 17:37 ` Jack Howarth
2009-12-16 18:09 ` Sebastian Pop
1 sibling, 1 reply; 4+ messages in thread
From: Jack Howarth @ 2009-12-16 17:37 UTC (permalink / raw)
To: gcc-patches; +Cc: jakub
Jakub,
Can we get this committed to gcc trunk? Sebastian Pop has already
approved this change to solve PR42135...
http://gcc.gnu.org/ml/gcc-patches/2009-11/msg01721.html
Thanks in advance.
Jack
On Wed, Nov 25, 2009 at 08:34:19PM -0500, Jack Howarth wrote:
> The attached patch reduces the array size in the libgomp.graphite/force-parallel-2.c
> testcase from 10000 to 500 so that it will fit into stacksize's as small as 2MB per
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42135#c2. This revised testcase now passes
> on x86_64-apple-darwin10. Okay for gcc trunk?
> Jack
>
> 2009-11-25 Jack Howarth <howarth@bromo.med.uc.edu>
>
> PR testsuite/42135
> * libgomp.graphite/force-parallel-2.c: Reduce array size.
>
> Index: libgomp/testsuite/libgomp.graphite/force-parallel-2.c
> ===================================================================
> --- libgomp/testsuite/libgomp.graphite/force-parallel-2.c (revision 154653)
> +++ libgomp/testsuite/libgomp.graphite/force-parallel-2.c (working copy)
> @@ -3,7 +3,7 @@
> void parloop (int N)
> {
> int i, j;
> - int x[10000][10000];
> + int x[500][500];
>
> for (i = 0; i < N; i++)
> for (j = 0; j < N; j++)
> @@ -17,7 +17,7 @@
>
> int main(void)
> {
> - parloop(10000);
> + parloop(500);
>
> return 0;
> }
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH][[PING]: Reduce array size in libgomp.graphite/force-parallel-2.c to fix PR42135
2009-12-16 17:37 ` [PATCH][[PING]: " Jack Howarth
@ 2009-12-16 18:09 ` Sebastian Pop
0 siblings, 0 replies; 4+ messages in thread
From: Sebastian Pop @ 2009-12-16 18:09 UTC (permalink / raw)
To: Jack Howarth; +Cc: gcc-patches, jakub
On Wed, Dec 16, 2009 at 08:51, Jack Howarth <howarth@bromo.med.uc.edu> wrote:
> Jakub,
> Can we get this committed to gcc trunk? Sebastian Pop has already
> approved this change to solve PR42135...
>
> http://gcc.gnu.org/ml/gcc-patches/2009-11/msg01721.html
>
> Thanks in advance.
> Jack
>
I will commit this to trunk. Thanks for pinging this patch.
Sebastian
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2009-12-16 15:50 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-11-26 1:58 [PATCH]: Reduce array size in libgomp.graphite/force-parallel-2.c to fix PR42135 Jack Howarth
2009-11-30 17:45 ` Sebastian Pop
2009-12-16 17:37 ` [PATCH][[PING]: " Jack Howarth
2009-12-16 18:09 ` Sebastian Pop
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).