hipMallocPitch requires cast to void**

[jakub-homola]

Hello,
this is a minor issue, but it is quite inconvenient -- one needs to cast the first parameter of hipMallocPitch to void** in HIP. In CUDA (cudaMallocPitch), the cast is not needed.

The CUDA program

#include <cstdio>


int main()
{
    int width = 42, height = 69;
    size_t pitch;
    float * data;

    cudaMallocPitch(&data, &pitch, width * sizeof(float), height);
    cudaFree(data);

    return 0;
}

is compiled with nvcc source.cu -o program.cuda.x without any issues.

However, the equivalent HIP program

#include <cstdio>
#include "hip/hip_runtime.h"

int main()
{
    int width = 42, height = 69;
    size_t pitch;
    float * data;

    hipMallocPitch(&data, &pitch, width * sizeof(float), height);
    hipFree(data);

    return 0;
}

fails on compilation (hipcc source.hip.cpp -o program.hip.x) on the hipMallocPitch function:

source.hip.cpp(10): error: argument of type "float **" is incompatible with parameter of type "void **"

Not a huge problem, since casting the first parameter to void** solves it, but in CUDA we do not need to perform the cast, so I think that neither should we in HIP.

I am using nvcc V11.4.152, HIP version 4.4.21401-bedc5f61

[ppanchad-amd]

@jakub-homola Apologies for the lack of response. Can you please test with latest ROCm 6.0.2 (HIP 6.0.32831)? If resolved, please close ticket. Thanks!

[jakub-homola]

I don't have access to any machine with the newest version of HIP. Using 5.2.3, the issue is still there.

Installing the newest version myself would be too complicated for me right now.

Could you please test it? It's really just copy-pasting the code above and trying to compile it.

[ppanchad-amd]

@jakub-homola We have confirmed it still occurs with ROCm 6.1.1. Internal ticket has been created to fix issue. Thanks!