Hello, Martin, Jakub, community,
This part of the patch:
On Mon, 7 Dec 2015, Martin Jambor wrote:
> include/
> * gomp-constants.h (GOMP_DEVICE_HSA): New macro.
[snip]
> (GOMP_kernel_launch_attributes): New type.
> (GOMP_hsa_kernel_dispatch): New type.
is going to break build of NVPTX cross-compiler, because it uses uint32_t,
uint64_t types like below, but those types will not be available when building
nvptx libgcc. gomp-constants.h is #include'd in libgcc via tm.h and
offload.h.
Note how other files in include/ need to do a special dance with #ifdef
HAVE_STDINT_H to include <stdint.h> and obtain uint64_t.
Shall I move the problematic structs into a separate file, gomp-types.h?
Thanks.
Alexander
> diff --git a/include/gomp-constants.h b/include/gomp-constants.h
> index dffd631..1dae474 100644
> --- a/include/gomp-constants.h
> +++ b/include/gomp-constants.h
[snip]
> +/* Structure describing the run-time and grid properties of an HSA kernel
> + lauch. */
> +
> +struct GOMP_kernel_launch_attributes
> +{
> + /* Number of dimensions the workload has. Maximum number is 3. */
> + uint32_t ndim;
> + /* Size of the grid in the three respective dimensions. */
> + uint32_t gdims[3];
> + /* Size of work-groups in the respective dimensions. */
> + uint32_t wdims[3];
> +};
