opencl-runtime compile on arm64

[0hanghang]

hi there! I've been compile rocm-opencl-runtime on arm64 platform ,following the README instruction procedure,but when make ,I got : ^~~~~~~~~~~~~~~ /home/ubuntu/opencl/rocm/opencl/runtime/platform/kernel.hpp:73:39: note: ‘const amd::KernelParameterDescriptor& amd::KernelSignature::at’ should be initialized const KernelParameterDescriptor& at(size_t index) const { ^~~~~~ /home/ubuntu/opencl/rocm/opencl/runtime/platform/kernel.hpp: At global scope: /home/ubuntu/opencl/rocm/opencl/runtime/platform/kernel.hpp:179:14: error: ‘size_t’ has not been declared void reset(size_t index) { ^~~~~~ /home/ubuntu/opencl/rocm/opencl/runtime/platform/kernel.hpp:185:12: error: ‘size_t’ has not been declared void set(size_t index, size_t size, const void* value, bool svmBound = false); ^~~~~~ /home/ubuntu/opencl/rocm/opencl/runtime/platform/kernel.hpp:185:26: error: ‘size_t’ has not been declared void set(size_t index, size_t size, const void* value, bool svmBound = false); ^~~~~~ /home/ubuntu/opencl/rocm/opencl/runtime/platform/kernel.hpp:188:13: error: ‘size_t’ has not been declared bool test(size_t index) const { return signature_.at(index).info_.defined_; } ^~~~~~ /home/ubuntu/opencl/rocm/opencl/runtime/platform/kernel.hpp:194:3: error: ‘size_t’ does not name a type; did you mean ‘ssize_t’? size_t localMemSize(size_t minDataTypeAlignment) const; ^~~~~~ many "size_t has not been declared "error! any help ? thanks in advance!

[0hanghang]

I install llvm by compiling source code with git clone --recursive -b clang_tot_upgrade https://github.com/RadeonOpenCompute/hcc.git is there any incompatible issue here ?

[0hanghang]

cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo -DLLVM_INCLUDES=/opt/rocm/llvm/lib/clang/11.0.0/include/ -DCMAKE_PREFIX_PATH=/home/ubuntu/opencl/rocm/ROCm-CompilerSupport/lib/comgr/build/lib/cmake/amd_comgr/ -DUSE_COMGR_LIBRARY=yes ..

[baryluk]

It looks the kernel.hpp was/is relaying on some implicit include that includes stdlib.h or stddef.h to define size_t.

It does include cstdlib, but that one provides it as std::size_t, not size_t (C style) itself.

So that should be fixed in kernel.hpp and other files audited for the same issue.