Commit 7c16c49
[ET-VK][ez] Update requirements for partitioning to_dim_order_copy (#7949)
Pull Request resolved: #7859
## Context
The previous registration of the to dim order copy op is incorrect. Currently, there is no implementation for the op in the Vulkan backend, but since Vulkan manages memory layout internally the op node can be removed as long as the only thing being changed is dim order.
In some instances the op can be used to modify the dtype, in which case it will not be removed and the Vulkan delegate cannot execute the op correctly. Therefore, update the registration of the op to reflect this restriction.
This diff should unblock enabling dim order ops for Vulkan.
ghstack-source-id: 262710507
@exported-using-ghexport
Differential Revision: [D68528213](https://our.internmc.facebook.com/intern/diff/D68528213/)
Co-authored-by: Stephen Jia <[email protected]>
(cherry picked from commit 5ee5f2f)1 parent 374f034 commit 7c16c49
1 file changed
+31
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
228 | 228 | | |
229 | 229 | | |
230 | 230 | | |
231 | | - | |
232 | | - | |
233 | 231 | | |
234 | 232 | | |
235 | 233 | | |
| |||
322 | 320 | | |
323 | 321 | | |
324 | 322 | | |
| 323 | + | |
| 324 | + | |
| 325 | + | |
| 326 | + | |
| 327 | + | |
| 328 | + | |
| 329 | + | |
| 330 | + | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
| 336 | + | |
| 337 | + | |
| 338 | + | |
| 339 | + | |
| 340 | + | |
| 341 | + | |
| 342 | + | |
| 343 | + | |
| 344 | + | |
| 345 | + | |
| 346 | + | |
| 347 | + | |
| 348 | + | |
| 349 | + | |
| 350 | + | |
| 351 | + | |
| 352 | + | |
| 353 | + | |
325 | 354 | | |
326 | 355 | | |
327 | 356 | | |
| |||
0 commit comments