iree: CPU ukernel breaks byo LLVM builds
What happened?
runtime/src/iree/builtins/ukernel/arch/x86_64/common_x86_64.h relies on immintrin.h which is included from Clang via set(_BUILTIN_HEADERS_PATH "${IREE_BINARY_DIR}/llvm-project/lib/clang/${_CLANG_VERSION_MAJOR}/include/") in build_tools/cmake/iree_bitcode_library.cmake.
However, when building IREE using build_tools/scripts/byo_llvm.sh, the system’s LLVM is used instead of being built, so ${IREE_BINARY_DIR} does not contain llvm-project.
Trying to build IREE using the byo_llvm.sh breaks due to this issue. To fix, check if IREE_BUILD_BUNDLED_LLVM=OFF and look for the installed Clang headers.
Steps to reproduce your issue
No response
What component(s) does this issue relate to?
Runtime
Version information
686860c6ba561401f8df1b86f193723470703954
Additional context
No response
About this issue
- Original URL
- State: closed
- Created a year ago
- Comments: 16 (11 by maintainers)
Commits related to this issue
- Unbreak the `byo_llvm.sh` build with `iree_bitcode_library`. (#13968) The byo-llvm build is broken by the way `iree_bitcode_library` assumes it's dealing with our own bundled `clang` (#13957). This ... — committed to iree-org/iree by bjacob a year ago
- Unbreak the `byo_llvm.sh` build with `iree_bitcode_library`. (#13968) The byo-llvm build is broken by the way `iree_bitcode_library` assumes it's dealing with our own bundled `clang` (#13957). This ... — committed to NatashaKnk/iree by bjacob a year ago
- Unbreak the `byo_llvm.sh` build with `iree_bitcode_library`. (#13968) The byo-llvm build is broken by the way `iree_bitcode_library` assumes it's dealing with our own bundled `clang` (#13957). This ... — committed to plaidml/iree by bjacob a year ago
also is byo_llvm.sh in the CI ? If not we probably should add it.
Since this is nontrivial to fix, I think we should at least for now unblock this by disabling the bitcode build in the non-bundled-Clang case.