Skip to content

[mosaic_gpu] Added support for using cf.assert in Mosaic GPU kernels #28215

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 1, 2025

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Apr 24, 2025

[mosaic_gpu] Added support for using cf.assert in Mosaic GPU kernels

The XLA GPU runtime does not yet handle device assertions well and will hang
if the assert is triggered. However, the assertion output still appears in
stderr, so I think having cf.assert support is still useful.

@copybara-service copybara-service bot force-pushed the test_740717697 branch 2 times, most recently from 0bd8f2c to bcf8c69 Compare May 1, 2025 19:44
@copybara-service copybara-service bot changed the title [mosaic_gpu] Added debug_assert [mosaic_gpu] Added support for using cf.assert in Mosaic GPU kernels May 1, 2025
@copybara-service copybara-service bot force-pushed the test_740717697 branch 5 times, most recently from b06035d to cf86ec6 Compare May 1, 2025 21:42
The XLA GPU runtime does not yet handle device assertions well and will hang
if the assert is triggered. However, the assertion output still appears in
stderr, so I think having `cf.assert` support is still useful.

PiperOrigin-RevId: 753742121
@copybara-service copybara-service bot merged commit f725249 into main May 1, 2025
1 of 2 checks passed
@copybara-service copybara-service bot deleted the test_740717697 branch May 1, 2025 22:02
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.

1 participant