Skip to content

Conversation

@apaszke
Copy link
Contributor

@apaszke apaszke commented Jan 15, 2017

This comment was marked as off-topic.

@apaszke apaszke force-pushed the fixes_v2 branch 5 times, most recently from 1e239f7 to 1566e17 Compare January 15, 2017 22:26
@apaszke apaszke closed this Jan 15, 2017
@apaszke apaszke reopened this Jan 15, 2017
@soumith soumith merged commit df79631 into master Jan 16, 2017
ashishfarmer pushed a commit to ashishfarmer/pytorch that referenced this pull request Oct 3, 2019
Skip test failing only on hip-clang to unblock them.
mrshenli pushed a commit to mrshenli/pytorch that referenced this pull request Apr 11, 2020
jjsjann123 pushed a commit to jjsjann123/pytorch that referenced this pull request Nov 25, 2020
* Fix cache_before

Currently, cache_before seems to have two problems. First, when the
cached tensor has reductions, its IterDomain expression history is lost
as it gets a new TensorDomain with only its root domains minus the
reduction domains.

Second, using computeAt does not necessarily generate correct computeAt
relationships. For example, in the CacheBefore test, T1's computeAt
tensor is still T2 even after cache_before. It should be the cache
tensor instead.

A test case is also added. See pytorch#408 as well.

Closes pytorch#408

* clang-tidy

* Review feedback

* clang-format

* Fix computeAt setting with reduction domains

* Add a missing break
KyleCZH pushed a commit to KyleCZH/pytorch that referenced this pull request Sep 20, 2021
Signed-off-by: Eli Uriegas <eliuriegas@fb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants