Skip to content

Commit

Permalink
[checkpoint] Address PR comments
Browse files Browse the repository at this point in the history
Also add back SplitArgs pass in build_module.cc which somehow got lost in the shuffle.

(try again -- flaky test_crt.py test_autotune?)
  • Loading branch information
mbs-octoml committed Nov 11, 2021
1 parent 8d79842 commit 1860b7a
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
6 changes: 6 additions & 0 deletions src/relay/backend/build_module.cc
Original file line number Diff line number Diff line change
Expand Up @@ -323,6 +323,12 @@ class RelayBuildModule : public runtime::ModuleNode {
/*is_homogenous=*/config_->optional_homogeneous_target.defined(), /*is_vm=*/false);
transform::PassContext pass_ctx = PassContext::Current();

if (config_->optional_homogeneous_target.defined()) {
// This pass currently only supports the homogeneous case.
pass_seqs.push_back(transform::SplitArgs(
config_->optional_homogeneous_target->GetAttr<Integer>("max_function_args", -1).value()));
}

// Always plan devices so the remaining passes don't need to distinguish homogeneous vs
// hetrogenous execution.
pass_seqs.push_back(transform::PlanDevices(config_));
Expand Down
3 changes: 1 addition & 2 deletions tests/python/unittest/test_micro_model_library_format.py
Original file line number Diff line number Diff line change
Expand Up @@ -412,5 +412,4 @@ def test_export_byoc_c_module():
if __name__ == "__main__":
import sys

# sys.exit(pytest.main([__file__] + sys.argv[1:]))
test_export_operator_model_library_format()
sys.exit(pytest.main([__file__] + sys.argv[1:]))

0 comments on commit 1860b7a

Please sign in to comment.