https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64972
Thomas Schwinge <tschwinge at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |openacc
CC| |tschwinge at gcc dot gnu.org
--- Comment #12 from Thomas Schwinge <tschwinge at gcc dot gnu.org> ---
Sorry for introducing this breakage; and thanks for working on fixing it.
(In reply to Jakub Jelinek from comment #11)
> I wouldn't call %zd gnu-style, as it is POSIX.
> Anyway, libgomp already uses inttypes.h, but only conditionally:
> #ifdef HAVE_INTTYPES_H
> fprintf (stderr, " GOMP_SPINCOUNT = '%"PRIu64"'\n",
> (uint64_t) gomp_spin_count_var);
> #else
> fprintf (stderr, " GOMP_SPINCOUNT = '%lu'\n",
> (unsigned long) gomp_spin_count_var);
> #endif
> So, if you'd like to do the same (but then cast to uint64_t and use PRIu64,
> dunno how portable is PRIuPTR compared to PRIu64), that is also an option,
> but IMNSHO not worth the trouble.
> All these messages are unimportant OpenACC debugging messages or a fancy
> abort that will very likely not have sizes of anything > 2GB - trying to
> offload 2GB+ of data to offloading device in one chunk is hardly a case to
> worry about.
..., and »640 K ought to be enough for anybody«? ;-)
Please do not change the code -- even if only "unimportant" debugging messages
-- so that it might print incorrect data.
