Skip to content

Conversation

@keryell
Copy link
Contributor

@keryell keryell commented Dec 5, 2023

Fix a test compilation error with g++ 13 on Ubuntu 23.10. Replace also a C header include with the official C++ one.

Bug context:

/home/rkeryell/Xilinx/Projects/LLVM/worktrees/intel/llvm/clang/test/Driver/clang-offload-wrapper-exe.cpp:55:3: error: unknown type name 'uint32_t'
   55 |   uint32_t Type;    // pi_property_type
      |   ^
/home/rkeryell/Xilinx/Projects/LLVM/worktrees/intel/llvm/clang/test/Driver/clang-offload-wrapper-exe.cpp:56:3: error: unknown type name 'uint64_t'
   56 |   uint64_t ValSize; // size of property value in bytes
      |   ^
/home/rkeryell/Xilinx/Projects/LLVM/worktrees/intel/llvm/clang/test/Driver/clang-offload-wrapper-exe.cpp:70:3: error: unknown type name 'uint16_t'
   70 |   uint16_t Version;
      |   ^
/home/rkeryell/Xilinx/Projects/LLVM/worktrees/intel/llvm/clang/test/Driver/clang-offload-wrapper-exe.cpp:71:3: error: unknown type name 'uint8_t'
   71 |   uint8_t Kind;   // 4 for SYCL
      |   ^
/home/rkeryell/Xilinx/Projects/LLVM/worktrees/intel/llvm/clang/test/Driver/clang-offload-wrapper-exe.cpp:72:3: error: unknown type name 'uint8_t'
   72 |   uint8_t Format; // 1 for native
      |   ^
/home/rkeryell/Xilinx/Projects/LLVM/worktrees/intel/llvm/clang/test/Driver/clang-offload-wrapper-exe.cpp:88:3: error: unknown type name 'uint16_t'
   88 |   uint16_t Version;
      |   ^
/home/rkeryell/Xilinx/Projects/LLVM/worktrees/intel/llvm/clang/test/Driver/clang-offload-wrapper-exe.cpp:89:3: error: unknown type name 'uint16_t'
   89 |   uint16_t NumDeviceBinaries;
      |   ^
7 errors generated.

Fix a test compilation error with g++ 13 on Ubuntu 23.10.
Replace also a C header include with the official C++ one.
@keryell keryell requested a review from a team as a code owner December 5, 2023 23:21
@bader
Copy link
Contributor

bader commented Dec 6, 2023

@intel/dpcpp-devops-reviewers, please, disable broken "SYCL E2E on AWS CUDA" until it's fixed.

@bader bader merged commit b1adf03 into intel:sycl Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants