Skip to content

[Tile] Use unpacked vector field for Tile16x16/Tile32x32 register storage#722

Open
hughperkins wants to merge 5 commits into
mainfrom
hp/tiles-use-unpacked-vector
Open

[Tile] Use unpacked vector field for Tile16x16/Tile32x32 register storage#722
hughperkins wants to merge 5 commits into
mainfrom
hp/tiles-use-unpacked-vector

[Tile] Fix two perf/correctness regressions introduced during the unp…

c705326
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
GitHub Actions / PR change report succeeded Jun 29, 2026 in 0s

PR change report

Per-file totals + per-function breakdown of code-line additions and removals.

Details

PR change report (c70532649)

Per-file totals + per-function breakdown of code-line additions and removals.
Code lines exclude blank lines, comment-only lines, and Python multi-line strings.

File LoC Added Removed
python/quadrants/lang/simt/_tile.py 0 +303
tests/python/test_tile.py 1559 +14 -9
python/quadrants/lang/simt/__init__.py 20 +7 -11
python/quadrants/lang/simt/tile_slicing.py 47 +4 -7
python/quadrants/__init__.py 63 +1 -1
python/quadrants/lang/simt/_tile32.py 774 -774
python/quadrants/lang/simt/_tile16.py 566 -566

Total: 7 file(s) changed, +329 -1368 code lines.

Per-function breakdown

python/quadrants/lang/simt/_tile.py 0 +303
    New:
      _make_tile_class()                       0     +172
      _TileProxy::_resolve()                   0      +12
      _TileRefProxy::_assign()                 0      +10
      _TileSliceProxy::__init__()              0       +9
      _make_tile()                             0       +9
      <module>()                               0       +7
      _VecSliceProxy::__init__()               0       +6
      _TileSliceProxy::_assign()               0       +5
      _DeferredProxyMixin::_misuse()           0       +3
      _OuterProduct::__init__()                0       +3
      _TileProto::_load3d()                    0       +3
      _TileProto::_store()                     0       +3
      _TileProto::_store3d()                   0       +3
      _TileProxy::__init__()                   0       +3
      _DeferredProxyMixin::__add__()           0       +2
      _DeferredProxyMixin::__getitem__()       0       +2
      _DeferredProxyMixin::__mul__()           0       +2
      _DeferredProxyMixin::__radd__()          0       +2
      _DeferredProxyMixin::__repr__()          0       +2
      _DeferredProxyMixin::__sub__()           0       +2
      _OuterProduct::__add__()                 0       +2
      _OuterProduct::__radd__()                0       +2
      _TileProxy::eye()                        0       +2
      _TileProxy::zeros()                      0       +2
      _TileRefProxy::__init__()                0       +2
      outer()                                  0       +2
      _TileProto::__getitem__()                0       +1
      _TileProto::__init__()                   0       +1
      _TileProto::__isub__()                   0       +1
      _TileProto::__setitem__()                0       +1
      _TileProto::_ger_sub()                   0       +1
      _TileProto::_get_col()                   0       +1
      _TileProto::_load()                      0       +1
      _TileProto::_set_col()                   0       +1
      _TileProto::_trsm()                      0       +1
      _TileProto::cholesky_()                  0       +1
      _TileProto::eye()                        0       +1
      _TileProto::eye_()                       0       +1
      _TileProto::solve_triangular_()          0       +1
      _TileProto::zeros()                      0       +1
    # note: per-function +/- differs from file totals by added_drift=+17 removed_drift=+0

tests/python/test_tile.py 1559 +14 -9
    Existing:
      <module>()                               26      +10       -6
      test_vec_proxy_non_identity_dtype()      51       +4       -3

python/quadrants/lang/simt/__init__.py 20 +7 -11
    Existing:
      __getattr__()      14       +5       -9
      <module>()          3       +2       -2

python/quadrants/lang/simt/tile_slicing.py 47 +4 -7
    Existing:
      <module>()              8       +2       -3
      _is_tile()              4       +1       -3
      _any_tile_built()       2       +1       -1

python/quadrants/__init__.py 63 +1 -1
    Existing:
      __getattr__()       7       +1       -1

python/quadrants/lang/simt/_tile32.py 774 -774
    # entire file deleted (per-function breakdown skipped)

python/quadrants/lang/simt/_tile16.py 566 -566
    # entire file deleted (per-function breakdown skipped)

Notes:
  * The number columns (without a + or - sign) are code-line counts in the BASE (pre-PR) version: file size before this PR (0 for newly-added files), function body size before this PR (0 for new functions; original body size for deleted functions).
  * +<n> / -<n> are code lines added / removed by this PR.
  * Code lines exclude blank lines, comment-only lines, and Python multi-line strings.