This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Merge from gomp-4_1-branch to trunk
- From: Sebastian Huber <sebastian dot huber at embedded-brains dot de>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 14 Oct 2015 10:32:25 +0200
- Subject: Re: Merge from gomp-4_1-branch to trunk
- Authentication-results: sourceware.org; auth=none
- References: <20151013191214 dot GL478 at tucnak dot redhat dot com> <561E0598 dot 6030003 at embedded-brains dot de> <20151014080403 dot GV478 at tucnak dot redhat dot com>
On 14/10/15 10:04, Jakub Jelinek wrote:
No, both the above changes are wrong. There is not a single int32_t
written, but could be many more, it is an array of 32-bit integers.
I'd say you just want to cast explicitly,
omp_get_place_proc_ids (*place_num, (int *) ids);
and
omp_get_parition_place_nums ((int *) place_nums);
The reason for int32_t is that on the Fortran side it is integer(kind=4)
and everywhere else for that int32_t is used.
If this works, the patch is preapproved.
I checked this in:
https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=228805
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone : +49 89 189 47 41-16
Fax : +49 89 189 47 41-09
E-Mail : sebastian.huber@embedded-brains.de
PGP : Public key available on request.
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.