mbs-octoml commented on a change in pull request #8974:
URL: https://github.com/apache/tvm/pull/8974#discussion_r708454807
##########
File path: src/relay/backend/interpreter.cc
##########
@@ -906,17 +906,16 @@ IRModule Prepare(IRModule mod, Device device, Target
target) {
backend::StaticMemoryPlan memory_plan; /*=nullptr*/
// Run minimal transforms on module to establish invariants needed by
interpreter.
- transform::Sequential seq(
- {transform::SimplifyInference(),
- // FuseOps will mark wrapped calls to prim-ops with the 'Primitive'
- // attribute.
- transform::FuseOps(/*fuse_opt_level=*/0), transform::ToANormalForm(),
- // eta expand to support constructors in argument position
- transform::EtaExpand(
- /*expand_constructor=*/true, /*expand_global_var=*/false),
- transform::InferType(),
- tec::LowerTEPass(targets, device_map, memory_plan,
/*module_name=*/"intrp",
- [](Function func) { /* no-op */ })});
+ transform::Sequential seq({transform::SimplifyInference(),
Review comment:
Seems we have inconsistent formatters? In any case I'd revert this
whitespace change.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
