inducer / inducer/loopy

Remove Array.offset

Open
#432 0 comments 0 reactions 0 assignees View on GitHub
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.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.