Skip to content

[pull] master from tensorflow:master#1632

Merged
pull[bot] merged 2 commits into
makesoftwaresafe:masterfrom
tensorflow:master
May 10, 2026
Merged

[pull] master from tensorflow:master#1632
pull[bot] merged 2 commits into
makesoftwaresafe:masterfrom
tensorflow:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented May 10, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

metaflow and others added 2 commits May 10, 2026 01:04
SymbolicTileAnalysis::ToString didn't handle regions correctly. Use spaces instead of tabs.
PiperOrigin-RevId: 913214711
Imported from GitHub PR openxla/xla#42259

When setting FFI state via `TypeId` require passing `TypeInfo` as well. Templated APIs are supposed to be used in statically linked binaries, and when different FFI handlers can be in separate object files we rely on opaque type ids and state, and always lookup them through FFI API.

This is one more fix required for making "internal" FFI headers "safe" when linked as multiple object files. This is still not safe in general and not recommended, but this is an escape hatch that makes it work correctly when XLA and FFI handler compiled with the same toolchain.
Copybara import of the project:

--
b3e513f56c2233a5c59ac5328de644df3e00d2ef by Eugene Zhulenev <ezhulenev@openxla.org>:

[xla:ffi] When setting opaque state require TypeInfo

Merging this change closes #42259

PiperOrigin-RevId: 913250041
@pull pull Bot locked and limited conversation to collaborators May 10, 2026
@pull pull Bot added the ⤵️ pull label May 10, 2026
@pull pull Bot merged commit 0d8f4d8 into makesoftwaresafe:master May 10, 2026
0 of 5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants