Skip to content
This repository was archived by the owner on Apr 28, 2023. It is now read-only.

Commit 1096b56

Browse files
committed
ScheduleTree::make*: do not manually assign type_
After grafting ScheduleTreeElem*, we no longer need to assign ScheduleTree::type_ manually in make* static constructors. They call the actual ScheduleTreeElem* constructors, that assign type_ by calling the ScheduleTree constructor.
1 parent 8499a09 commit 1096b56

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

tc/core/polyhedral/schedule_tree.cc

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,6 @@ std::unique_ptr<ScheduleTree> ScheduleTree::makeBand(
227227
isl::multi_union_pw_aff mupa,
228228
std::vector<ScheduleTreeUPtr>&& children) {
229229
auto res = ScheduleTreeElemBand::fromMultiUnionPwAff(mupa);
230-
res->type_ = detail::ScheduleTreeType::Band;
231230
res->appendChildren(std::move(children));
232231
return res;
233232
}
@@ -245,7 +244,6 @@ std::unique_ptr<ScheduleTree> ScheduleTree::makeDomain(
245244
isl::union_set domain,
246245
std::vector<ScheduleTreeUPtr>&& children) {
247246
auto res = std::unique_ptr<ScheduleTree>(new ScheduleTreeElemDomain(domain));
248-
res->type_ = detail::ScheduleTreeType::Domain;
249247
res->appendChildren(std::move(children));
250248
return res;
251249
}
@@ -255,7 +253,6 @@ std::unique_ptr<ScheduleTree> ScheduleTree::makeContext(
255253
std::vector<ScheduleTreeUPtr>&& children) {
256254
auto res =
257255
std::unique_ptr<ScheduleTree>(new ScheduleTreeElemContext(context));
258-
res->type_ = detail::ScheduleTreeType::Context;
259256
res->appendChildren(std::move(children));
260257
return res;
261258
}
@@ -264,7 +261,6 @@ std::unique_ptr<ScheduleTree> ScheduleTree::makeFilter(
264261
isl::union_set filter,
265262
std::vector<ScheduleTreeUPtr>&& children) {
266263
auto res = std::unique_ptr<ScheduleTree>(new ScheduleTreeElemFilter(filter));
267-
res->type_ = detail::ScheduleTreeType::Filter;
268264
res->appendChildren(std::move(children));
269265
return res;
270266
}
@@ -294,7 +290,6 @@ std::unique_ptr<ScheduleTree> ScheduleTree::makeExtension(
294290
std::vector<ScheduleTreeUPtr>&& children) {
295291
auto res =
296292
std::unique_ptr<ScheduleTree>(new ScheduleTreeElemExtension(extension));
297-
res->type_ = detail::ScheduleTreeType::Extension;
298293
res->appendChildren(std::move(children));
299294
return res;
300295
}
@@ -304,7 +299,6 @@ std::unique_ptr<ScheduleTree> ScheduleTree::makeThreadSpecificMarker(
304299
std::vector<ScheduleTreeUPtr>&& children) {
305300
auto res = std::unique_ptr<ScheduleTree>(
306301
new ScheduleTreeElemThreadSpecificMarker(ctx));
307-
res->type_ = detail::ScheduleTreeType::ThreadSpecificMarker;
308302
res->appendChildren(std::move(children));
309303
return res;
310304
}

0 commit comments

Comments
 (0)