Skip to content

Conversation

@wsmoses
Copy link
Member

@wsmoses wsmoses commented Oct 26, 2025

we need to local jll build test against JuliaPackaging/Yggdrasil#12363

? not sure if the current builder stuff takes a branch

@wsmoses wsmoses requested a review from giordano October 26, 2025 09:07
@wsmoses
Copy link
Member Author

wsmoses commented Oct 26, 2025

this test isn't doing anything since we need the bazelrc change in reactant in EnzymeAD/Reactant.jl#1787

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EnzymeJAX Benchmarks

Benchmark suite Current: 24c086b Previous: 992f348 Ratio
scatter_sum / JaX / cpu / Primal 0.000004288459301460535 s 0.000004391519189812243 s 0.98
scatter_sum / JaXPipe / cpu / Primal 0.000004264789295848459 s 0.000004307569190859795 s 0.99
scatter_sum / JaX / tpu / Primal 0.0001560258985031 s 0.0001399941653944 s 1.11
scatter_sum / JaXPipe / tpu / Primal 0.0001554112866986 s 0.000138530330907 s 1.12

This comment was automatically generated by workflow using github-action-benchmark.

@wsmoses wsmoses merged commit 8f6fcba into main Oct 28, 2025
4 of 7 checks passed
@wsmoses wsmoses deleted the scuda branch October 28, 2025 05:51
@giordano
Copy link
Member

This broke compilation with CUDA 13

ERROR: /__w/Enzyme-JAX/Enzyme-JAX/Reactant.jl/deps/ReactantExtra/BUILD:1224:10: Linking libReactantExtra.so failed: (Exit 1): process-wrapper failed: error executing CppLink command 
  (cd /root/.bazel/sandbox/processwrapper-sandbox/14131/execroot/__main__ && \
  exec env - \
    GRPC_BAZEL_RUNTIME=1 \
    JULIA=/__w/_tool/julia/1.11.7/x64/bin/julia \
    PATH=/__w/.cache/bazel/bazelisk/downloads/sha256/c97f02133adce63f0c28678ac1f21d65fa8255c80429b588aeeba8a1fac6202b/bin:/__w/_tool/julia/1.11.7/x64/bin:/__w/_tool/bazelisk/1.x/x64:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin \
    PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=upb \
    *** \
    TMPDIR=/tmp \
  /__w/.cache/bazel/bazel/_bazel_root/install/81618c1cfcf8a55fe29d247a9003bce4/process-wrapper '--timeout=0' '--kill_delay=15' '--stats=/root/.bazel/sandbox/processwrapper-sandbox/14131/stats.out' external/local_config_cuda/crosstool/clang/bin/crosstool_wrapper_driver_is_not_gcc @bazel-out/k8-opt/bin/libReactantExtra.so-2.params)
ld.lld: error: relocation R_X86_64_PC32 cannot be used against symbol '_nv024841cupti'; recompile with -fPIC
>>> defined in external/cuda_cupti/lib/libcupti_static.a(libcupti_static_partial_mangled.o)
>>> referenced by libcupti_static_partial_mangled.o:(_nv024859cupti) in archive external/cuda_cupti/lib/libcupti_static.a

ld.lld: error: relocation R_X86_64_PC32 cannot be used against symbol '_nv024841cupti'; recompile with -fPIC
>>> defined in external/cuda_cupti/lib/libcupti_static.a(libcupti_static_partial_mangled.o)
>>> referenced by libcupti_static_partial_mangled.o:(_nv024836cupti) in archive external/cuda_cupti/lib/libcupti_static.a

ld.lld: error: relocation R_X86_64_PC32 cannot be used against symbol '_nv024841cupti'; recompile with -fPIC
>>> defined in external/cuda_cupti/lib/libcupti_static.a(libcupti_static_partial_mangled.o)
>>> referenced by libcupti_static_partial_mangled.o:(_nv024836cupti) in archive external/cuda_cupti/lib/libcupti_static.a

ld.lld: error: relocation R_X86_64_PC32 cannot be used against symbol '_nv024840cupti'; recompile with -fPIC
>>> defined in external/cuda_cupti/lib/libcupti_static.a(libcupti_static_partial_mangled.o)
>>> referenced by libcupti_static_partial_mangled.o:(_nv024839cupti) in archive external/cuda_cupti/lib/libcupti_static.a
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Target //:libReactantExtra.so failed to build

@wsmoses
Copy link
Member Author

wsmoses commented Oct 28, 2025

The reactant Pr forcing the use of lld fixes that

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