Skip to content

multiple fixes for XLA bump#190

Merged
mofeing merged 12 commits into
mainfrom
ss/explicit-arm-aes
Jul 8, 2026
Merged

multiple fixes for XLA bump#190
mofeing merged 12 commits into
mainfrom
ss/explicit-arm-aes

Conversation

@mofeing

@mofeing mofeing commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

required for fixing https://github.com/EnzymeAD/Enzyme-JAX/actions/runs/28517017952/job/84531806243?pr=2588#step:10:1953

changes

  • explicitly set ARM target features AES and SHA2 (required for aarch64 builds)
  • wrap type_traits and bits/move.h C++ std headers to be marked as __host__ __device__ for device-side compilation with hipcc (required for ROCm builds)
  • stop exporting libnvshmem_device.bc: I have searched around and it no longer seems to exist (required for CUDA builds)

Comment thread R/Reactant/build_tarballs.jl Outdated
Co-authored-by: Mosè Giordano <765740+giordano@users.noreply.github.com>
@mofeing mofeing changed the title explicitly set -march=armv8+aes explicitly set ARM target features Jul 2, 2026
@mofeing mofeing changed the title explicitly set ARM target features multiple fixes for XLA bump Jul 3, 2026
@mofeing mofeing requested review from giordano and wsmoses July 3, 2026 08:14
Comment thread R/Reactant/build_tarballs.jl Outdated
@mofeing mofeing merged commit db0aafe into main Jul 8, 2026
31 of 32 checks passed
@mofeing mofeing deleted the ss/explicit-arm-aes branch July 8, 2026 11:23
repo = "https://github.com/EnzymeAD/Reactant.jl.git"
reactant_commit = "ea9253ba28fcd81b13caf78faa13f11a38e441ba"
reactant_commit = "8e91f8791c2b3155777818325b541d05bf6135db"
version = v"0.0.391"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why wasn't the version changed? This produced v0.0.391+2: JuliaRegistries/General#160350

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.

2 participants