Skip to content

Commit

Permalink
[SYCL] Temporarily turn off zstd, except on CI (#15833)
Browse files Browse the repository at this point in the history
It's a temporary fix for local build issues with intel/llvm when zstd is
not available.
I'll revert this after triaging and fixing the build issue.
  • Loading branch information
uditagarwal97 authored Oct 23, 2024
1 parent de4efa0 commit 12ece03
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion buildbot/configure.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ def do_configure(args):

sycl_enable_xpti_tracing = "ON"
xpti_enable_werror = "OFF"
llvm_enable_zstd = "OFF"

if sys.platform != "darwin":
sycl_enabled_backends.append("level_zero")
Expand Down Expand Up @@ -133,6 +134,8 @@ def do_configure(args):

# For clang-format, clang-tidy and code coverage
llvm_enable_projects += ";clang-tools-extra;compiler-rt"
# Build with zstd enabled on CI.
llvm_enable_zstd = "ON"
if sys.platform != "darwin":
# libclc is required for CI validation
libclc_enabled = True
Expand Down Expand Up @@ -177,7 +180,7 @@ def do_configure(args):
"-DLLVM_ENABLE_PROJECTS={}".format(llvm_enable_projects),
"-DSYCL_BUILD_PI_HIP_PLATFORM={}".format(sycl_build_pi_hip_platform),
"-DLLVM_BUILD_TOOLS=ON",
"-DLLVM_ENABLE_ZSTD=ON",
"-DLLVM_ENABLE_ZSTD={}".format(llvm_enable_zstd),
"-DLLVM_USE_STATIC_ZSTD=ON",
"-DSYCL_ENABLE_WERROR={}".format(sycl_werror),
"-DCMAKE_INSTALL_PREFIX={}".format(install_dir),
Expand Down

0 comments on commit 12ece03

Please sign in to comment.