Skip to content

Replace .data_ptr with .mutable_data_ptr or .const_data_ptr#5267

Closed
cyyever wants to merge 4 commits into
pytorch:mainfrom
cyyever:data_ptr
Closed

Replace .data_ptr with .mutable_data_ptr or .const_data_ptr#5267
cyyever wants to merge 4 commits into
pytorch:mainfrom
cyyever:data_ptr

Conversation

@cyyever

@cyyever cyyever commented Dec 20, 2025

Copy link
Copy Markdown
Contributor

.const_data_ptr is more performant than .data_ptr

Signed-off-by: Yuanyuan Chen <cyyever@outlook.com>
Signed-off-by: Yuanyuan Chen <cyyever@outlook.com>
@meta-cla meta-cla Bot added the cla signed label Dec 20, 2025
@cyyever cyyever changed the title Replace .data_ptr with .mutable_data_ptr or .const_data_ptr. Replace .data_ptr with .mutable_data_ptr or .const_data_ptr Dec 20, 2025
@meta-codesync

meta-codesync Bot commented Dec 22, 2025

Copy link
Copy Markdown
Contributor

@q10 has imported this pull request. If you are a Meta employee, you can view this in D89675247.

@q10

q10 commented Dec 22, 2025

Copy link
Copy Markdown
Contributor

@cyyever looks like the builds are failing

Signed-off-by: Yuanyuan Chen <cyyever@outlook.com>
@cyyever

cyyever commented Dec 23, 2025

Copy link
Copy Markdown
Contributor Author

@q10 fixed

@meta-codesync

meta-codesync Bot commented Jan 2, 2026

Copy link
Copy Markdown
Contributor

@q10 merged this pull request in 2e4e480.

@cyyever cyyever deleted the data_ptr branch January 3, 2026 01:13
jensenhwa pushed a commit to jensenhwa/FBGEMM that referenced this pull request Mar 18, 2026
…5276)

Summary:
Pull Request resolved: pytorch#5276

X-link: https://github.com/facebookresearch/FBGEMM/pull/2270

`.const_data_ptr` is more performant than `.data_ptr`

Pull Request resolved: pytorch#5267

Reviewed By: gchalump

Differential Revision: D89675247

Pulled By: q10

fbshipit-source-id: 16065d0f9739c9c5e01bd4c30a1ae3cc746e24eb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants