Skip to content
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

Use BufferF32 instead of Buffer in GPU FFI example #13058

Closed
wants to merge 1 commit into from

Conversation

andportnoy
Copy link
Member

No description provided.

@andportnoy andportnoy requested a review from ezhulenev May 24, 2024 19:07
@github-actions github-actions bot added the kokoro:force-run Forces CI to rerun label May 24, 2024
@kokoro-team kokoro-team removed the kokoro:force-run Forces CI to rerun label May 24, 2024
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request May 27, 2024
Imported from GitHub PR openxla/xla#13058

Copybara import of the project:

--
5db8e9ecb4becd69cab0381c5142058697afadb5 by Andrey Portnoy <aportnoy@nvidia.com>:

Use `BufferF32` instead of `Buffer` in GPU FFI example

Merging this change closes #13058

FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#13058 from openxla:aportnoy/fix-ffi-docs-buffer 5db8e9ecb4becd69cab0381c5142058697afadb5
PiperOrigin-RevId: 636998665
@akuegel
Copy link
Member

akuegel commented Jun 3, 2024

I think there was a conflict when trying to merge this PR. Can you please rebase?

copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Jun 3, 2024
Imported from GitHub PR openxla/xla#13058

Copybara import of the project:

--
5db8e9ecb4becd69cab0381c5142058697afadb5 by Andrey Portnoy <aportnoy@nvidia.com>:

Use `BufferF32` instead of `Buffer` in GPU FFI example

Merging this change closes #13058

FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#13058 from openxla:aportnoy/fix-ffi-docs-buffer 5db8e9ecb4becd69cab0381c5142058697afadb5
PiperOrigin-RevId: 639796277
@andportnoy
Copy link
Member Author

@akuegel interesting, typically GitHub would show when there's a merge conflict, why is it not working in this case?

Also strange how GitHub is showing "Merging is blocked Merging can be performed automatically with 1 approving review." even though Eugene approved just recently.

@andportnoy
Copy link
Member Author

I'll try rebasing.

copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Jun 4, 2024
Imported from GitHub PR openxla/xla#13058

Copybara import of the project:

--
5db8e9ecb4becd69cab0381c5142058697afadb5 by Andrey Portnoy <aportnoy@nvidia.com>:

Use `BufferF32` instead of `Buffer` in GPU FFI example

Merging this change closes #13058

FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#13058 from openxla:aportnoy/fix-ffi-docs-buffer 5db8e9ecb4becd69cab0381c5142058697afadb5
PiperOrigin-RevId: 639796277
@copybara-service copybara-service bot closed this in 67f1815 Jun 4, 2024
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Jun 4, 2024
FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#13058 from openxla:aportnoy/fix-ffi-docs-buffer 5db8e9ecb4becd69cab0381c5142058697afadb5
PiperOrigin-RevId: 639744852
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Jun 4, 2024
FUTURE_COPYBARA_INTEGRATE_REVIEW=openxla/xla#13058 from openxla:aportnoy/fix-ffi-docs-buffer 5db8e9ecb4becd69cab0381c5142058697afadb5
PiperOrigin-RevId: 639586878
copybara-service bot pushed a commit to tensorflow/tensorflow that referenced this pull request Jun 4, 2024
Imported from GitHub PR openxla/xla#13058

Copybara import of the project:

--
5db8e9ecb4becd69cab0381c5142058697afadb5 by Andrey Portnoy <aportnoy@nvidia.com>:

Use `BufferF32` instead of `Buffer` in GPU FFI example

Merging this change closes #13058

PiperOrigin-RevId: 640072298
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.

None yet

6 participants