Skip to content

Conversation

@avik-pal
Copy link
Collaborator

@avik-pal avik-pal commented Nov 9, 2025

No description provided.

@avik-pal
Copy link
Collaborator Author

avik-pal commented Nov 9, 2025

dont merge. Let me add reactant builder here just to be safe

@giordano
Copy link
Member

giordano commented Nov 9, 2025

This builds way too many platforms clogging CI

@giordano
Copy link
Member

giordano commented Nov 9, 2025

@wsmoses
Copy link
Member

wsmoses commented Nov 9, 2025

Relatedly we should also change that to include rocm too

@wsmoses
Copy link
Member

wsmoses commented Nov 9, 2025

longer term we probably should include a reactant_commit or something

@avik-pal
Copy link
Collaborator Author

avik-pal commented Nov 9, 2025

what I am messing up here?

https://github.com/EnzymeAD/Reactant.jl/actions/runs/19214699690/job/54922234558?pr=1844#step:9:16

Changing Reactant commit to a35f68fe372772371416dbc710c4c1ad9ca4be57...

this is clearly not the current commit

@avik-pal
Copy link
Collaborator Author

avik-pal commented Nov 9, 2025

@wsmoses builds are going through now

@avik-pal avik-pal merged commit 5462657 into main Nov 9, 2025
13 of 17 checks passed
@avik-pal avik-pal deleted the ap/hlo_ir_fixes branch November 9, 2025 22:27
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