Remove Array.offset
Open
- Dominant language
- Python
- Stars
- 636
- Forks
- 81
- Avg merge
- 1d 19h
- Merged PRs (30d)
- 7
Description
`offset` has no business being in the IR. It's a concern that's specific entrypoints in OpenCL and *nothing* else. So it should be a concern for *just that* target.
How does this sound:
* [ ] As a first step, change the default for `offset` to `lp.auto`, deprecate passing anything else. This is a change in behavior, but it should be safe. For `offset == 0`, steer people towards adding a `lp.NoOffsetInCL` tag, which avoids the cost of the additional offset argument.
* [ ] Deprecate accessing `ary.offset`.
* [ ] After the deprecation expires, remove `ary.offset`
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.