[Bug libgomp/90585] libgomp hsa plugin ftbfs in the x32 multilib variant
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90585 Jakub Jelinek changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|--- |FIXED Target Milestone|--- |7.5 --- Comment #8 from Jakub Jelinek --- Fixed.
[Bug libgomp/90585] libgomp hsa plugin ftbfs in the x32 multilib variant
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90585 --- Comment #7 from Jakub Jelinek --- Author: jakub Date: Fri Aug 30 12:45:31 2019 New Revision: 275158 URL: https://gcc.gnu.org/viewcvs?rev=275158=gcc=rev Log: Backported from mainline 2019-05-24 Jakub Jelinek PR libgomp/90585 * plugin/plugin-hsa.c (print_kernel_dispatch, run_kernel): Use PRIu64 macro instead of "lu". (release_kernel_dispatch): Likewise. Cast shadow->debug to uintptr_t before casting to void *. Modified: branches/gcc-7-branch/libgomp/ChangeLog branches/gcc-7-branch/libgomp/plugin/plugin-hsa.c
[Bug libgomp/90585] libgomp hsa plugin ftbfs in the x32 multilib variant
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90585 --- Comment #6 from Jakub Jelinek --- Author: jakub Date: Thu Aug 29 15:00:20 2019 New Revision: 275039 URL: https://gcc.gnu.org/viewcvs?rev=275039=gcc=rev Log: Backported from mainline 2019-05-24 Jakub Jelinek PR libgomp/90585 * plugin/plugin-hsa.c (print_kernel_dispatch, run_kernel): Use PRIu64 macro instead of "lu". (release_kernel_dispatch): Likewise. Cast shadow->debug to uintptr_t before casting to void *. Modified: branches/gcc-8-branch/libgomp/ChangeLog branches/gcc-8-branch/libgomp/plugin/plugin-hsa.c
[Bug libgomp/90585] libgomp hsa plugin ftbfs in the x32 multilib variant
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90585 --- Comment #5 from Jakub Jelinek --- Author: jakub Date: Wed Jun 5 12:18:23 2019 New Revision: 271957 URL: https://gcc.gnu.org/viewcvs?rev=271957=gcc=rev Log: Backported from mainline 2019-05-24 Jakub Jelinek PR libgomp/90585 * plugin/plugin-hsa.c (print_kernel_dispatch, run_kernel): Use PRIu64 macro instead of "lu". (release_kernel_dispatch): Likewise. Cast shadow->debug to uintptr_t before casting to void *. Modified: branches/gcc-9-branch/libgomp/ChangeLog branches/gcc-9-branch/libgomp/plugin/plugin-hsa.c
[Bug libgomp/90585] libgomp hsa plugin ftbfs in the x32 multilib variant
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90585 --- Comment #4 from Jakub Jelinek --- Fixed on the trunk, guess for the backport we could do a simpler thing (as we included inttypes.h unconditionally and nobody complained on the branch, just use the PRIu64 and nothing else). On the trunk it is better to stay being more portable.
[Bug libgomp/90585] libgomp hsa plugin ftbfs in the x32 multilib variant
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90585 --- Comment #3 from Jakub Jelinek --- Author: jakub Date: Fri May 24 08:59:37 2019 New Revision: 271597 URL: https://gcc.gnu.org/viewcvs?rev=271597=gcc=rev Log: PR libgomp/90585 * plugin/plugin-hsa.c: Include gstdint.h. Include inttypes.h only if HAVE_INTTYPES_H is defined. (print_uint64_t): New typedef. (PRIu64): Define if HAVE_INTTYPES_H is not defined. (print_kernel_dispatch, run_kernel): Use PRIu64 macro instead of "lu", cast uint64_t HSA_DEBUG and fprintf arguments to print_uint64_t. (release_kernel_dispatch): Likewise. Cast shadow->debug to uintptr_t before casting to void *. * plugin/plugin-nvptx.c: Include gstdint.h instead of stdint.h. * oacc-mem.c: Don't include config.h nor stdint.h. * target.c: Don't include config.h. * oacc-cuda.c: Likewise. * oacc-host.c: Don't include stdint.h. Modified: trunk/libgomp/ChangeLog trunk/libgomp/oacc-cuda.c trunk/libgomp/oacc-host.c trunk/libgomp/oacc-mem.c trunk/libgomp/plugin/plugin-hsa.c trunk/libgomp/plugin/plugin-nvptx.c trunk/libgomp/target.c
[Bug libgomp/90585] libgomp hsa plugin ftbfs in the x32 multilib variant
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90585 --- Comment #2 from Jakub Jelinek --- Created attachment 46400 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46400=edit gcc10-pr90585.patch Untested fix. Note, the rest of libgomp uses PRIu64 only conditionally on HAVE_INTTYPES_H, so this patch does that too, but as it is in many spots, using a typedef.
[Bug libgomp/90585] libgomp hsa plugin ftbfs in the x32 multilib variant
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90585 --- Comment #1 from Matthias Klose --- looks like libgomp/configure.ac always sets -Werror, not respecting the --disable-werror configure option.
