PR #39201: [xla:gpu] Split legacy custom calls into LegacyCustomCallThunk#41072
Open
copybara-service[bot] wants to merge 1 commit intomainfrom
Open
PR #39201: [xla:gpu] Split legacy custom calls into LegacyCustomCallThunk#41072copybara-service[bot] wants to merge 1 commit intomainfrom
copybara-service[bot] wants to merge 1 commit intomainfrom
Conversation
copybara-service
bot
force-pushed
the
test_900699293
branch
4 times, most recently
from
f872317 to
582f91a
Compare
…hunk Imported from GitHub PR #39201 Split `CustomCallThunk` into two separate classes to cleanly separate FFI from legacy custom call APIs (to be deleted). This change is **NFC**, simply splitting existing thunk/command into two classes to clearly separate legacy calling conventions from FFI. - **`CustomCallThunk`** — now exclusively handles XLA FFI custom calls. Simplified by removing all legacy code paths, optional bundle checks, and the `ExecuteCustomCall()` fallback. The `bundle_` field is now a plain `std::variant` (always present) instead of `std::optional<std::variant>`. - **`LegacyCustomCallThunk`** (new) — handles deprecated legacy custom calls (`API_VERSION_STATUS_RETURNING`, etc.) using `CustomCallTargetRegistry`. Marked as deprecated in its class comment, directing users to FFI via `CustomCallThunk`. Also split `CustomCallCmd` in the command buffer layer into: - **`CustomCallCmd`** — FFI only, builds `ffi::CallFrame` and invokes FFI handlers directly. - **`LegacyCustomCallCmd`** — legacy only, traces custom call execution via `TraceCommandBufferFactory`. Unified `CustomCallThunk` accumulated too much complexity related to dispatching between two unrelated custom all ABIs, by splitting them into separate files we can keep on improving FFI without having to deal with legacy custom calls. Copybara import of the project: -- 9f2fb95 by Eugene Zhulenev <ezhulenev@openxla.org>: [xla:gpu] Split legacy custom calls into LegacyCustomCallThunk Merging this change closes #39201 FUTURE_COPYBARA_INTEGRATE_REVIEW=#39201 from ezhulenev:split-custom-call-thunk 9f2fb95 PiperOrigin-RevId: 900699293
copybara-service
bot
force-pushed
the
test_900699293
branch
from
582f91a to
850b157
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR #39201: [xla:gpu] Split legacy custom calls into LegacyCustomCallThunk
Imported from GitHub PR #39201
Split
CustomCallThunkinto two separate classes to cleanly separate FFI from legacy custom call APIs (to be deleted). This change is NFC, simply splitting existing thunk/command into two classes to clearly separate legacy calling conventions from FFI.CustomCallThunk— now exclusively handles XLA FFI custom calls. Simplified by removing all legacy code paths, optional bundle checks, and theExecuteCustomCall()fallback. Thebundle_field is now a plainstd::variant(always present) instead ofstd::optional<std::variant>.LegacyCustomCallThunk(new) — handles deprecated legacy custom calls (API_VERSION_STATUS_RETURNING, etc.) usingCustomCallTargetRegistry. Marked as deprecated in its class comment, directing users to FFI viaCustomCallThunk.Also split
CustomCallCmdin the command buffer layer into:CustomCallCmd— FFI only, buildsffi::CallFrameand invokes FFI handlers directly.LegacyCustomCallCmd— legacy only, traces custom call execution viaTraceCommandBufferFactory.Unified
CustomCallThunkaccumulated too much complexity related to dispatching between two unrelated custom all ABIs, by splitting them into separate files we can keep on improving FFI without having to deal with legacy custom calls.Copybara import of the project:
--
9f2fb95 by Eugene Zhulenev ezhulenev@openxla.org:
[xla:gpu] Split legacy custom calls into LegacyCustomCallThunk
Merging this change closes #39201
FUTURE_COPYBARA_INTEGRATE_REVIEW=#39201 from ezhulenev:split-custom-call-thunk 9f2fb95