Skip to content

Commit

Permalink
removing checks that are not exposed by scheduler
Browse files Browse the repository at this point in the history
  • Loading branch information
jjsjann123 committed Dec 11, 2024
1 parent 1d021c7 commit 742f7f3
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
1 change: 1 addition & 0 deletions csrc/scheduler/pointwise_utils.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -219,6 +219,7 @@ bool DomainMap::areAllTargetIdsCoveredBy(
// it's safe for target_tv to have them.
std::unordered_set<IterDomain*> covered_source_ids;
for (IterDomain* source_id_ref : get_source_iter_domains(reference_tv)) {
NVF_ERROR(source_id_ref->definition() == nullptr || id->definition()->isA<ResizeOp>());
covered_source_ids.insert(source_id_ref);
}
// It's safe to have unmapped broadcast IterDomain. There're quite a few tests
Expand Down
6 changes: 0 additions & 6 deletions tests/cpp/test_pointwise.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -826,9 +826,6 @@ TEST_F(PointwiseTest, DomainMapTestEg0) {
// tv4 is a valid reference
EXPECT_TRUE(domain_map.isValidReference(tv4));

// check reference tv selection
EXPECT_FALSE(domain_map.findReferenceTensorView() == tv4);

// validate generated kernel
auto options = at::TensorOptions().dtype(at::kFloat).device(at::kCUDA, 0);
at::Tensor t0 = at::randn({4, 7}, options);
Expand Down Expand Up @@ -873,9 +870,6 @@ TEST_F(PointwiseTest, DomainMapTestEg1) {
// tv4 is a valid reference
EXPECT_TRUE(domain_map.isValidReference(tv4));

// check reference tv selection
EXPECT_FALSE(domain_map.findReferenceTensorView() == tv4);

// validate generated kernel
auto options = at::TensorOptions().dtype(at::kFloat).device(at::kCUDA, 0);
at::Tensor t0 = at::randn({2, 4}, options);
Expand Down

0 comments on commit 742f7f3

Please sign in to comment.