You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Enabled TensorIndexer for the reshape tests.
I temporarily added a codegen diff result to this PR. This one is more
concise as I disabled index hoisting. As far as I can see, there's no
concerning change. I haven't verified everything, but I believe most of
them are because TensorIndexer can detect more divisible splits, which
helps generate simplified indices through more aggressive contig
indexing.
Once approved, I'll remove the html file.
### Context
Part of #4175.
I'm planning to enable the new indexer globally by default once we are
sufficiently confident with it. I'm going to enable it for some of the
C++ tests for now. Just manually checking the diff results seems to be
the only way to gain some confidence.
All the tests are passing in my local branch, but just having green test
results don't necessarily mean everything is properly ported to the new
indexer. I'll also check perf changes with the benchmarks, but they may
not give clear signals as indexing is just one piece of performance
bottlenecks.
0 commit comments