Use const_data_ptr/mutable_data_ptr instead of deprecated data_ptr#20958
Use const_data_ptr/mutable_data_ptr instead of deprecated data_ptr#20958cyyever wants to merge 2 commits into
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/20958
Note: Links to docs will display an error until the docs builds have been completed. ❌ 6 New Failures, 1 Unrelated FailureAs of commit fb9f428 with merge base e16c048 ( NEW FAILURES - The following jobs have failed:
FLAKY - The following job failed but was likely due to flakiness present on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This PR needs a
|
|
Thanks @cyyever! This LGTM, just one cleanup on cuda_mutable_state.cpp |
cbda78d to
fb9f428
Compare
ExecuTorch
Tensor::data_ptrisET_DEPRECATED. Replace remaining call sites: read-only accesses useconst_data_ptr, the rest usemutable_data_ptr(sampler input, CMSIS-NN struct fields, JNI/mmap'd buffers that require non-const pointers).Left unchanged:
backends/aoti/slim(SlimTensor has no const/mutable variants) andat::Tensor::data_ptr()inextension/pybindings(not deprecated;for_blobtakesvoid*).Authored with Claude Code.