google / google/heir

Improve support for InsertSlice in RotationAnalysis

Open
#2,877 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
MLIR
Stars
906
Forks
171
Avg merge
4d 12h
Merged PRs (30d)
32

Description

Adding a TODO for minor techdebt.

The RotationAnalysis builds a dag of operations to use for statically inferring a set of rotation indices given as input to rotation ops. If the computation of those indices involves an insert_slice op, it will likely be incorrect due to this issue. However, it seems unlikely that this will occur, since rotation indices are usually scalar computations.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.