Skip to content

ggml: support ggml_rope_set_offset on opencl, sycl, wgpu, hexagon - #27345

Merged
ngxson merged 2 commits into
masterfrom
xsn/ggml_rope_off_rest
Aug 20, 2026
Merged

ggml: support ggml_rope_set_offset on opencl, sycl, wgpu, hexagon#27345
ngxson merged 2 commits into
masterfrom
xsn/ggml_rope_off_rest

Conversation

@ngxson

@ngxson ngxson commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

Overview

Ref #27120

Pretty much mechanical change, same logic copied to other backends

Requirements

@ngxson
ngxson requested review from a team as code owners August 18, 2026 22:31
@github-actions github-actions Bot added ggml changes relating to the ggml tensor library for machine learning SYCL https://en.wikipedia.org/wiki/SYCL - GPU programming language OpenCL Issues specific to the OpenCL backend Hexagon WebGPU labels Aug 18, 2026
@ngxson
ngxson changed the base branch from xsn/ggml_rope_off to master August 19, 2026 12:05
@ngxson
ngxson requested review from a team, ggerganov and marty1885 as code owners August 19, 2026 12:05
@ngxson
ngxson removed request for a team, ggerganov and marty1885 August 19, 2026 12:06

@max-krasnyansky max-krasnyansky left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hexagon changes look good to me

@Titaniumtown Titaniumtown left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sycl changes LGTM

@lhez lhez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

opencl changes look good.

@yomaytk yomaytk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

webgpu changes looks good to me.

@arthw arthw left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's OK for SYCL backend.
All related UT cases (ROPE,ROPE_BACK) are passed.

Thank you!

@ngxson
ngxson merged commit 749f688 into master Aug 20, 2026
38 of 41 checks passed
@ngxson

ngxson commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator Author

thanks everyone, this unblocked #27382

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ggml changes relating to the ggml tensor library for machine learning Hexagon OpenCL Issues specific to the OpenCL backend SYCL https://en.wikipedia.org/wiki/SYCL - GPU programming language WebGPU

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants