junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797347960
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797345809
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797342927
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
cyx-6 commented on pull request #51:
URL: https://github.com/apache/tvm-rfcs/pull/51#issuecomment-1027640390
Thanks for @areusch and @junrushao1994 's suggestions. I have updated the
RFC according to discussion here and on the forum.
--
This is an automated message from the Apache Git Se
LyricZhao commented on issue #10134:
URL: https://github.com/apache/tvm/issues/10134#issuecomment-1027638506
Same error with me (CUDA 11.2 + Miniconda). Downgrading to 10.2 works but
not a fancy solution.
--
This is an automated message from the Apache Git Service.
To respond to the mess
This is an automated email from the ASF dual-hosted git repository.
andrewzhaoluo pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git
The following commit(s) were added to refs/heads/main by this push:
new efe662f [Relay][Pass] Add a relay pass to extra
AndrewZhaoLuo merged pull request #10089:
URL: https://github.com/apache/tvm/pull/10089
--
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: commits-un
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797247187
##
File path: src/tir/transforms/common_subexpr_elim.cc
##
@@ -0,0 +1,601 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more c
masahi commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797245853
##
File path: src/tir/transforms/common_subexpr_elim.cc
##
@@ -0,0 +1,601 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more con
masahi commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797245853
##
File path: src/tir/transforms/common_subexpr_elim.cc
##
@@ -0,0 +1,601 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more con
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797243979
##
File path: src/tir/transforms/common_subexpr_elim.cc
##
@@ -0,0 +1,601 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more c
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797243979
##
File path: src/tir/transforms/common_subexpr_elim.cc
##
@@ -0,0 +1,601 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more c
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797243979
##
File path: src/tir/transforms/common_subexpr_elim.cc
##
@@ -0,0 +1,601 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more c
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797243979
##
File path: src/tir/transforms/common_subexpr_elim.cc
##
@@ -0,0 +1,601 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more c
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797243979
##
File path: src/tir/transforms/common_subexpr_elim.cc
##
@@ -0,0 +1,601 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more c
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797243979
##
File path: src/tir/transforms/common_subexpr_elim.cc
##
@@ -0,0 +1,601 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more c
masahi commented on a change in pull request #10124:
URL: https://github.com/apache/tvm/pull/10124#discussion_r797227357
##
File path: src/relay/transforms/device_planner.cc
##
@@ -489,51 +489,71 @@ class DeviceAnalyzer : public ExprVisitor {
void VisitExpr_(const CallNode
ccjoechou commented on pull request #48:
URL: https://github.com/apache/tvm-rfcs/pull/48#issuecomment-1027512160
Hi @areusch:
I have updated RFC-#48 based on most of your feedback. New RFC md file and 4
new figures have been uploaded.
Please take a look again (in a view document mode
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797219586
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
junrushao1994 commented on issue #10107:
URL: https://github.com/apache/tvm/issues/10107#issuecomment-1027510782
That's good point! Thanks for pointing this out :-)
@wrongtest so looks like our bound is probably too tight. Would you mind
sending a PR tweaking it a little bit? Thanks!
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797214394
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
areusch opened a new issue #10142:
URL: https://github.com/apache/tvm/issues/10142
microTVM currently reports some memory usage numbers, but a true estimate of
the RAM footprint needs to be made by examining the output of the Unified
Static Memory Planner being implemented in #8404. This i
areusch opened a new issue #10141:
URL: https://github.com/apache/tvm/issues/10141
We added initial support for RISC-V in microTVM by proving that we can run
regression tests against the Zephyr RISC-V target (backed by QEMU). However, we
have not yet added support for any V- or P- extensio
srkreddy1238 opened a new pull request #10140:
URL: https://github.com/apache/tvm/pull/10140
Thanks for contributing to TVM! Please refer to guideline
https://tvm.apache.org/docs/contribute/ for useful information and tips. After
the pull request is submitted, please request code reviews
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797208786
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
prateek9623 opened a new issue #10139:
URL: https://github.com/apache/tvm/issues/10139
Hi, can we add tvm cmake targets in an export group? As current targets when
linked with another target with an export target create issues. Also, it would
be great if we add support cmake "find_package"
mehrdadh commented on issue #10126:
URL: https://github.com/apache/tvm/issues/10126#issuecomment-1027488653
working on this: https://github.com/apache/tvm/pull/10138
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
mehrdadh opened a new pull request #10138:
URL: https://github.com/apache/tvm/pull/10138
This PR:
- Updates Zephyr version on RVM to 2.7-branch
- Refactor duplication of zephyr branch and zephyr sdk version
- Fix tests for version upgrade
cc @Mousius @areusch @gromero
--
ccjoechou commented on a change in pull request #48:
URL: https://github.com/apache/tvm-rfcs/pull/48#discussion_r797201727
##
File path: rfcs/0048-BYOC-Marvell-ML-accelerator-integration.md
##
@@ -0,0 +1,547 @@
+- Feature Name: (fill me in with a unique identifier, `my_awesome_
ccjoechou commented on a change in pull request #48:
URL: https://github.com/apache/tvm-rfcs/pull/48#discussion_r797201350
##
File path: rfcs/0048-BYOC-Marvell-ML-accelerator-integration.md
##
@@ -0,0 +1,547 @@
+- Feature Name: (fill me in with a unique identifier, `my_awesome_
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797200354
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
areusch opened a new issue #10137:
URL: https://github.com/apache/tvm/issues/10137
This is a tracking issue for accepted legacy RFC:
https://discuss.tvm.apache.org/t/rfc-python-dependencies-in-tvm-ci-containers/9011
--
This is an automated message from the Apache Git Service.
To respond
ccjoechou commented on a change in pull request #48:
URL: https://github.com/apache/tvm-rfcs/pull/48#discussion_r797198719
##
File path: rfcs/0048-BYOC-Marvell-ML-accelerator-integration.md
##
@@ -0,0 +1,547 @@
+- Feature Name: (fill me in with a unique identifier, `my_awesome_
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797198386
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
ccjoechou commented on a change in pull request #48:
URL: https://github.com/apache/tvm-rfcs/pull/48#discussion_r797197944
##
File path: rfcs/0048-BYOC-Marvell-ML-accelerator-integration.md
##
@@ -0,0 +1,547 @@
+- Feature Name: (fill me in with a unique identifier, `my_awesome_
ccjoechou commented on a change in pull request #48:
URL: https://github.com/apache/tvm-rfcs/pull/48#discussion_r797197725
##
File path: rfcs/0048-BYOC-Marvell-ML-accelerator-integration.md
##
@@ -0,0 +1,547 @@
+- Feature Name: (fill me in with a unique identifier, `my_awesome_
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797197301
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
ccjoechou commented on a change in pull request #48:
URL: https://github.com/apache/tvm-rfcs/pull/48#discussion_r797197238
##
File path: rfcs/0048-BYOC-Marvell-ML-accelerator-integration.md
##
@@ -0,0 +1,547 @@
+- Feature Name: (fill me in with a unique identifier, `my_awesome_
ccjoechou commented on a change in pull request #48:
URL: https://github.com/apache/tvm-rfcs/pull/48#discussion_r797193790
##
File path: rfcs/0048-BYOC-Marvell-ML-accelerator-integration.md
##
@@ -0,0 +1,547 @@
+- Feature Name: (fill me in with a unique identifier, `my_awesome_
mehrdadh commented on issue #10136:
URL: https://github.com/apache/tvm/issues/10136#issuecomment-1027473778
I can help with this. Feel free to assign it to me.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
ccjoechou commented on a change in pull request #48:
URL: https://github.com/apache/tvm-rfcs/pull/48#discussion_r797193262
##
File path: rfcs/0048-BYOC-Marvell-ML-accelerator-integration.md
##
@@ -0,0 +1,547 @@
+- Feature Name: (fill me in with a unique identifier, `my_awesome_
areusch opened a new issue #10136:
URL: https://github.com/apache/tvm/issues/10136
There have been a few instances where we've discovered that the microTVM
Reference VMs can't be rebuilt from the
[Vagrantfile](https://github.com/apache/tvm/blob/main/apps/microtvm/reference-vm/zephyr/Vagran
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797182546
##
File path: src/tir/transforms/common_subexpr_elim_tools.h
##
@@ -0,0 +1,216 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or m
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797181949
##
File path: src/tir/transforms/common_subexpr_elim_tools.h
##
@@ -0,0 +1,216 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or m
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797171727
##
File path: src/tir/transforms/common_subexpr_elim_tools.cc
##
@@ -0,0 +1,836 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797162445
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797148809
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797148517
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797147754
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797146441
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
altanh commented on a change in pull request #10026:
URL: https://github.com/apache/tvm/pull/10026#discussion_r797142059
##
File path: src/relay/backend/vm/manifest_lifetimes.cc
##
@@ -0,0 +1,611 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797139585
##
File path: src/tir/transforms/common_subexpr_elim_tools.h
##
@@ -0,0 +1,216 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or m
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797137808
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797137370
##
File path: src/tir/transforms/inject_software_pipeline.cc
##
@@ -0,0 +1,785 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797134900
##
File path: src/tir/transforms/ir_utils.h
##
@@ -267,6 +267,24 @@ class ConditionalBoundsContext {
std::unordered_map origin_map_;
};
+// Info
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797134900
##
File path: src/tir/transforms/ir_utils.h
##
@@ -267,6 +267,24 @@ class ConditionalBoundsContext {
std::unordered_map origin_map_;
};
+// Info
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797132652
##
File path: include/tvm/tir/transform.h
##
@@ -492,6 +492,112 @@ TVM_DLL Pass ConvertForLoopsToSerial();
*/
TVM_DLL Pass UnifiedStaticMemoryPlann
FranckQC commented on a change in pull request #9482:
URL: https://github.com/apache/tvm/pull/9482#discussion_r797131033
##
File path: include/tvm/tir/transform.h
##
@@ -458,6 +458,14 @@ TVM_DLL Pass FlattenBuffer();
*/
TVM_DLL Pass TextureFlatten();
+/*!
+ * \brief Implem
mei-ye commented on a change in pull request #10124:
URL: https://github.com/apache/tvm/pull/10124#discussion_r797130983
##
File path: src/relay/transforms/device_planner.cc
##
@@ -489,51 +489,71 @@ class DeviceAnalyzer : public ExprVisitor {
void VisitExpr_(const CallNode
This is an automated email from the ASF dual-hosted git repository.
masahi pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git
The following commit(s) were added to refs/heads/main by this push:
new ac4815c [AutoScheduler] Allow device specification for
mei-ye opened a new issue #10135:
URL: https://github.com/apache/tvm/issues/10135
On windows, the max stack size is 1 MB, whereas it is 8 MB on Linux. So
stack overflow occurs more often on windows for large graphs. We run into a
stack overflow problem on windows. The cause is recursi
masahi merged pull request #10123:
URL: https://github.com/apache/tvm/pull/10123
--
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: commits-unsubscr.
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797128964
##
File path: include/tvm/tir/transform.h
##
@@ -492,6 +492,112 @@ TVM_DLL Pass ConvertForLoopsToSerial();
*/
TVM_DLL Pass UnifiedStaticMemoryPlann
junrushao1994 commented on a change in pull request #10066:
URL: https://github.com/apache/tvm/pull/10066#discussion_r797127287
##
File path: include/tvm/tir/transform.h
##
@@ -492,6 +492,112 @@ TVM_DLL Pass ConvertForLoopsToSerial();
*/
TVM_DLL Pass UnifiedStaticMemoryPlann
hubert0527 opened a new issue #10134:
URL: https://github.com/apache/tvm/issues/10134
### Expected behavior
I followed the [install/build
instruction](https://tvm.apache.org/docs/install/from_source.html) to build TVM
with CUDA11.1. And failed at the following step:
```
conda
jwfromm commented on pull request #10123:
URL: https://github.com/apache/tvm/pull/10123#issuecomment-1027370076
@masahi can you also check to make sure this all looks ok?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
mbs-octoml commented on a change in pull request #10026:
URL: https://github.com/apache/tvm/pull/10026#discussion_r797098041
##
File path: src/relay/backend/vm/manifest_lifetimes.cc
##
@@ -0,0 +1,611 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or m
Mousius commented on pull request #10129:
URL: https://github.com/apache/tvm/pull/10129#issuecomment-1027355629
Thanks @mbaret !
--
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 comm
Mousius merged pull request #10129:
URL: https://github.com/apache/tvm/pull/10129
--
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: commits-unsubscr
This is an automated email from the ASF dual-hosted git repository.
mousius pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git
The following commit(s) were added to refs/heads/main by this push:
new faf63f9 Update ethos-u-vela for demo app (#10129)
faf
This is an automated email from the ASF dual-hosted git repository.
comaniac pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git.
from a1f51aa [CUTLASS] Conv2d dgrad (#10110)
add 780f88a [FIX,AUTOTVM] Add backtraces to tuning errors (#9901)
No new
comaniac merged pull request #9901:
URL: https://github.com/apache/tvm/pull/9901
--
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: commits-unsubscr.
comaniac commented on pull request #9901:
URL: https://github.com/apache/tvm/pull/9901#issuecomment-1027344681
Thanks @tkonolige
--
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 com
tkonolige commented on pull request #9901:
URL: https://github.com/apache/tvm/pull/9901#issuecomment-1027333490
@comaniac I think I've addressed all your concerns. Could you verify that
this PR is looking good to you?
--
This is an automated message from the Apache Git Service.
To respon
MargaretQian commented on a change in pull request #10089:
URL: https://github.com/apache/tvm/pull/10089#discussion_r797044095
##
File path: src/relay/analysis/extract_fake_quantized_ops.cc
##
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
MargaretQian commented on a change in pull request #10089:
URL: https://github.com/apache/tvm/pull/10089#discussion_r797044095
##
File path: src/relay/analysis/extract_fake_quantized_ops.cc
##
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
MargaretQian commented on a change in pull request #10089:
URL: https://github.com/apache/tvm/pull/10089#discussion_r797012395
##
File path: src/relay/analysis/extract_fake_quantized_ops.cc
##
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
lazycal commented on pull request #10118:
URL: https://github.com/apache/tvm/pull/10118#issuecomment-1027277470
So the 2x2 is actually encoding 2 transforms: one is the producer's change,
and the other is the consumer's (expected) change. For a weird example, before
the change: `(A) conv_N
This is an automated email from the ASF dual-hosted git repository.
masahi pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git.
from 72b22f8 [Relay][VM] Fix loading late bound consts when none exist
(#10087)
add a1f51aa [CUTLASS] Conv2d dgrad (#10
masahi commented on pull request #10110:
URL: https://github.com/apache/tvm/pull/10110#issuecomment-1027275237
Merging, I'll follow up with wgrad + parallel split-k support.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
masahi merged pull request #10110:
URL: https://github.com/apache/tvm/pull/10110
--
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: commits-unsubscr.
AndrewZhaoLuo commented on a change in pull request #10089:
URL: https://github.com/apache/tvm/pull/10089#discussion_r796995152
##
File path: src/relay/transforms/fake_quantization_to_integer.h
##
@@ -0,0 +1,56 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under o
masahi commented on pull request #10118:
URL: https://github.com/apache/tvm/pull/10118#issuecomment-1027266790
> Same here. I have 2 proposals:
>
> * replace `_in` with `_in_input` and `_in2` with `_in_infer`.
>
> * Or more explicit, `_in` with `_in_producer` and `_in2`
lazycal commented on a change in pull request #10118:
URL: https://github.com/apache/tvm/pull/10118#discussion_r796979604
##
File path: src/relay/transforms/transform_layout.h
##
@@ -371,6 +375,17 @@ Expr LayoutRewriter(const Call& ref_call, const
Array& new_args, const Obj
masahi commented on a change in pull request #10118:
URL: https://github.com/apache/tvm/pull/10118#discussion_r796974298
##
File path: src/relay/transforms/transform_layout.h
##
@@ -371,6 +375,17 @@ Expr LayoutRewriter(const Call& ref_call, const
Array& new_args, const Obj
altanh commented on pull request #10026:
URL: https://github.com/apache/tvm/pull/10026#issuecomment-1027244014
cc @jroesch @mbs-octoml @mbrookhart
this PR is ready for full review
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
altanh commented on pull request #10026:
URL: https://github.com/apache/tvm/pull/10026#issuecomment-1027242901
I wrote this as a comment in the code, but reposting here for future
reference:
```
Current Limitations
---
1. For simplicity, we only insert kills when
lazycal commented on a change in pull request #10118:
URL: https://github.com/apache/tvm/pull/10118#discussion_r796951191
##
File path: src/relay/op/nn/nn.cc
##
@@ -283,14 +279,6 @@ InferCorrectLayoutOutput DensePackInferCorrectLayout(const
Attrs& attrs,
areusch commented on pull request #10056:
URL: https://github.com/apache/tvm/pull/10056#issuecomment-1027198966
one problem here is that a tag doesn't automatically update with `git
fetch`. A branch would. it's a little confusing to have a branch instead of a
tag, but a branch is confined
This is an automated email from the ASF dual-hosted git repository.
mbrookhart pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git.
from 339f888 [USMP] Add performance characteristics to PoolInfo (#10005)
add 72b22f8 [Relay][VM] Fix loading late bo
mbrookhart merged pull request #10087:
URL: https://github.com/apache/tvm/pull/10087
--
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: commits-unsub
mei-ye commented on a change in pull request #10124:
URL: https://github.com/apache/tvm/pull/10124#discussion_r796913274
##
File path: src/relay/transforms/device_planner.cc
##
@@ -489,51 +489,71 @@ class DeviceAnalyzer : public ExprVisitor {
void VisitExpr_(const CallNode
mbs-octoml commented on a change in pull request #10124:
URL: https://github.com/apache/tvm/pull/10124#discussion_r796905096
##
File path: src/relay/transforms/device_planner.cc
##
@@ -489,51 +489,71 @@ class DeviceAnalyzer : public ExprVisitor {
void VisitExpr_(const Call
AndrewZhaoLuo commented on pull request #8668:
URL: https://github.com/apache/tvm/pull/8668#issuecomment-1027172187
Closing in favor of #10085
--
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 th
AndrewZhaoLuo closed pull request #8668:
URL: https://github.com/apache/tvm/pull/8668
--
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: commits-unsu
anwang2009 commented on a change in pull request #10086:
URL: https://github.com/apache/tvm/pull/10086#discussion_r796287746
##
File path: src/relay/transforms/pattern_utils.h
##
@@ -520,8 +525,8 @@ inline Expr FastSoftmax(Expr e, tvm::Attrs attr) {
return Call(op, {e}, attr
AndrewZhaoLuo commented on pull request #10133:
URL: https://github.com/apache/tvm/pull/10133#issuecomment-1027152890
cc @mbrookhart can you re-review this?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
AndrewZhaoLuo closed pull request #8081:
URL: https://github.com/apache/tvm/pull/8081
--
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: commits-unsu
AndrewZhaoLuo commented on pull request #8081:
URL: https://github.com/apache/tvm/pull/8081#issuecomment-1027152626
Closing in favor of https://github.com/apache/tvm/pull/10133
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
AndrewZhaoLuo opened a new pull request #10133:
URL: https://github.com/apache/tvm/pull/10133
This is a shepherding of https://github.com/apache/tvm/pull/8081. All credit
goes to @icemelon.
I slightly simplified some stuff and added some comments
--
This is an automated message fr
1 - 100 of 160 matches
Mail list logo