search

KhronosGroup / SPIRV-LLVM-Translator

A tool and a library for bi-directional translation between SPIR-V and LLVM IR
Other
468 stars 209 forks source link

[Backport to 14] [DebugInfo] Round up #elts of TypeVector when calculating memory size #2649

Closed maarquitos14 closed 1 month ago

MrSidims commented 1 month ago

@maarquitos14 please check the build failure: "Translator/lib/SPIRV/SPIRVToLLVMDbgTran.cpp:499:56: error: ‘bit_ceil’ is not a member of ‘llvm’ 499 | uint64_t Size = getDerivedSizeInBits(BaseTy) * llvm::bit_ceil(Count);"

maarquitos14 commented 1 month ago

@maarquitos14 please check the build failure: "Translator/lib/SPIRV/SPIRVToLLVMDbgTran.cpp:499:56: error: ‘bit_ceil’ is not a member of ‘llvm’ 499 | uint64_t Size = getDerivedSizeInBits(BaseTy) * llvm::bit_ceil(Count);"

llvm::bit_ceil was introduced in LLVM 16, and std::bit_ceil requires C++20. Should I implement our own bit_ceil function for backports to 14 and 15?