diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2010-04-20 17:36:45 +0200 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2010-04-20 17:36:45 +0200 |
| commit | 5221d7a91b9d6b674fb473675274099f1da939dd (patch) | |
| tree | 7440afed755684acfb2774ba45ae6709add2d3a0 /libgomp | |
| parent | a837268b2b7f2b2623392c376144c5f343f23011 (diff) | |
| download | gcc-5221d7a91b9d6b674fb473675274099f1da939dd.zip gcc-5221d7a91b9d6b674fb473675274099f1da939dd.tar.gz gcc-5221d7a91b9d6b674fb473675274099f1da939dd.tar.bz2 | |
re PR libgomp/43569 (libgomp: Conditional jump or move depends on uninitialised value)
PR libgomp/43569
* sections.c (gomp_sections_init): Initialize ws->mode.
From-SVN: r158564
Diffstat (limited to 'libgomp')
| -rw-r--r-- | libgomp/ChangeLog | 5 | ||||
| -rw-r--r-- | libgomp/sections.c | 20 |
2 files changed, 23 insertions, 2 deletions
diff --git a/libgomp/ChangeLog b/libgomp/ChangeLog index 5534cd6..7756ac1 100644 --- a/libgomp/ChangeLog +++ b/libgomp/ChangeLog @@ -1,3 +1,8 @@ +2010-04-20 Jakub Jelinek <jakub@redhat.com> + + PR libgomp/43569 + * sections.c (gomp_sections_init): Initialize ws->mode. + 2010-04-14 Uros Bizjak <ubizjak@gmail.com> * acinclude.m4 (LIBGOMP_CHECK_SYNC_BUILTINS): Remove set but diff --git a/libgomp/sections.c b/libgomp/sections.c index 7acd441..c7f49b7 100644 --- a/libgomp/sections.c +++ b/libgomp/sections.c @@ -1,4 +1,4 @@ -/* Copyright (C) 2005, 2007, 2008, 2009 Free Software Foundation, Inc. +/* Copyright (C) 2005, 2007, 2008, 2009, 2010 Free Software Foundation, Inc. Contributed by Richard Henderson <rth@redhat.com>. This file is part of the GNU OpenMP Library (libgomp). @@ -34,9 +34,25 @@ gomp_sections_init (struct gomp_work_share *ws, unsigned count) { ws->sched = GFS_DYNAMIC; ws->chunk_size = 1; - ws->end = count + 1; + ws->end = count + 1L; ws->incr = 1; ws->next = 1; +#ifdef HAVE_SYNC_BUILTINS + /* Prepare things to make each iteration faster. */ + if (sizeof (long) > sizeof (unsigned)) + ws->mode = 1; + else + { + struct gomp_thread *thr = gomp_thread (); + struct gomp_team *team = thr->ts.team; + long nthreads = team ? team->nthreads : 1; + + ws->mode = ((nthreads | ws->end) + < 1UL << (sizeof (long) * __CHAR_BIT__ / 2 - 1)); + } +#else + ws->mode = 0; +#endif } /* This routine is called when first encountering a sections construct |