inducer / inducer/loopy

Get rid of `applied_iname_rewrites`

Open
#641 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

Suggested by @kaushikcfd in https://github.com/inducer/loopy/pull/631#discussion_r907887383.

The original thinking behind it was transform usability. Suppose one splits an iname. The one thing that this currently enables is the use of the old iname in `add_prefetch`. That's less awkward than having to manually specify the split iname expression, but I do agree that this is kind of broken as a concept, and definitely not strong enough justification to drag this around in the IR.

Usage site for reference: https://github.com/inducer/loopy/blob/7d2f7d1eb43e0ebf923e141f68879ea0e10d9ac9/loopy/transform/data.py#L98-L101=

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.