Skip to content

[pull] master from tensorflow:master#1989

Merged
pull[bot] merged 4 commits into
GesuBackups:masterfrom
tensorflow:master
Jul 1, 2026
Merged

[pull] master from tensorflow:master#1989
pull[bot] merged 4 commits into
GesuBackups:masterfrom
tensorflow:master

Conversation

@pull

@pull pull Bot commented Jul 1, 2026

Copy link
Copy Markdown

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 : )

dmiltr3 and others added 4 commits July 1, 2026 08:06
…_size() validation.

PiperOrigin-RevId: 941098195
Updates LLVM usage to match
[c7e2809a631a](llvm/llvm-project@c7e2809a631a)

PiperOrigin-RevId: 941147999
---

When exporting StableHLOComposite attributes, flexbuffers::Builder was being called with the attribute name as a key for every vector element. In flexbuffers, passing a key to a Vector element pushes both the key and the value to the stack, causing the vector to be parsed with twice its intended length (with the key interleaved).

This change removes the key from the vector element calls, and only uses the key when starting the parent vector, which matches the flexbuffers::Builder API expectation for Vectors.

- Also fixed a typo where `isa<StringAttr>(attr)` was checking the Array attribute instead of the element.

- Updated test to pass actual vector values for serialization

PiperOrigin-RevId: 941177131
@pull pull Bot locked and limited conversation to collaborators Jul 1, 2026
@pull pull Bot added the ⤵️ pull label Jul 1, 2026
@pull pull Bot merged commit 756be76 into GesuBackups:master Jul 1, 2026
2 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.

4 participants