[GitHub] [tvm] Hzfengsy opened a new pull request #9747: [TensorIR] fix compact_dataflow

2021-12-14 Thread GitBox
Hzfengsy opened a new pull request #9747: URL: https://github.com/apache/tvm/pull/9747 Fix a small bug in `require_subtree_compact_dataflow` Co-authored-by: Bohan Hou <32121147+spectrometer...@users.noreply.github.com> Co-authored-by: Hongyi Jin <3231950...@qq.com>

[GitHub] [tvm] masahi opened a new pull request #9746: [CUTLASS] Support conv2d activation fusion

2021-12-14 Thread GitBox
masahi opened a new pull request #9746: URL: https://github.com/apache/tvm/pull/9746 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 from

[GitHub] [tvm] jroesch commented on pull request #9529: [TVMC] Split common tvmc file into more specific files

2021-12-14 Thread GitBox
jroesch commented on pull request #9529: URL: https://github.com/apache/tvm/pull/9529#issuecomment-994372946 ping @leandron -- 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

[GitHub] [tvm] jroesch commented on a change in pull request #9723: [Frontend] Add Span filling for frontends to Relay

2021-12-14 Thread GitBox
jroesch commented on a change in pull request #9723: URL: https://github.com/apache/tvm/pull/9723#discussion_r769291698 ## File path: src/printer/relay_text_printer.cc ## @@ -389,12 +389,21 @@ Doc RelayTextPrinter::VisitExpr_(const TupleNode* op) { if (op->fields.size() ==

[tvm] branch main updated: [Relay] Support large constants saved/loaded outside of VM executable (#9734)

2021-12-14 Thread jroesch
This is an automated email from the ASF dual-hosted git repository. jroesch 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 e7b5d2d [Relay] Support large constants saved/loaded

[GitHub] [tvm] jroesch merged pull request #9734: [Relay] Support large constants saved/loaded outside of VM executable

2021-12-14 Thread GitBox
jroesch merged pull request #9734: URL: https://github.com/apache/tvm/pull/9734 -- 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:

[tvm] branch main updated: [TIR][Schedule] Analysis functions to check if compute_inline and com… (#9743)

2021-12-14 Thread syfeng
This is an automated email from the ASF dual-hosted git repository. syfeng 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 ecc2e56 [TIR][Schedule] Analysis functions to check

[GitHub] [tvm] Hzfengsy merged pull request #9743: [TIR][Schedule] Analysis functions to check if compute_inline and com…

2021-12-14 Thread GitBox
Hzfengsy merged pull request #9743: URL: https://github.com/apache/tvm/pull/9743 -- 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:

[tvm] branch main updated (dd42ef2 -> 5d47eff)

2021-12-14 Thread syfeng
This is an automated email from the ASF dual-hosted git repository. syfeng pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tvm.git. from dd42ef2 [CUTLASS] Add conv2d profiler (#9737) add 5d47eff [TIR] Add 'global_symbol' and 'tir.noalias' as default

[GitHub] [tvm] Hzfengsy merged pull request #9744: [TIR] Add 'global_symbol' and 'tir.noalias' as default attributes in script auto completion

2021-12-14 Thread GitBox
Hzfengsy merged pull request #9744: URL: https://github.com/apache/tvm/pull/9744 -- 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:

[tvm] branch main updated (21abb6e -> dd42ef2)

2021-12-14 Thread masahi
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 21abb6e Fix for tvm.build()'s name warning (#9678) add dd42ef2 [CUTLASS] Add conv2d profiler (#9737) No new

[GitHub] [tvm] masahi merged pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
masahi merged pull request #9737: URL: https://github.com/apache/tvm/pull/9737 -- 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:

[GitHub] [tvm] masahi commented on pull request #9724: [Frontend][PaddlePaddle] Enhance paddlepaddle frontend with more operators

2021-12-14 Thread GitBox
masahi commented on pull request #9724: URL: https://github.com/apache/tvm/pull/9724#issuecomment-994255628 CI had a problem, please kick a new job. -- 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

[GitHub] [tvm] ccjoechou commented on pull request #9730: [pre-RFC][BYOC] Marvell ML/AI Accelerator Integration

2021-12-14 Thread GitBox
ccjoechou commented on pull request #9730: URL: https://github.com/apache/tvm/pull/9730#issuecomment-994210222 Hello @jroesch, After done more self-discovery from reviewing other BYOC code files and CMake setup files, I think now I am aware of that, in the Mrvl.cmake file, the source

[GitHub] [tvm] junrushao1994 commented on pull request #9738: [TensorIR] Primitive "SetScope"

2021-12-14 Thread GitBox
junrushao1994 commented on pull request #9738: URL: https://github.com/apache/tvm/pull/9738#issuecomment-994192685 For context, this primitive is only used in one of our rule Cross-Thread-Reduction. Welcome everyone to review! CC: @Lunderberg @csullivan -- This is an automated message

[GitHub] [tvm] MasterJH5574 commented on a change in pull request #9738: [TensorIR] Primitive "SetScope"

2021-12-14 Thread GitBox
MasterJH5574 commented on a change in pull request #9738: URL: https://github.com/apache/tvm/pull/9738#discussion_r769165528 ## File path: src/tir/schedule/analysis.h ## @@ -393,6 +393,10 @@ std::vector> GetReducerGetters() bool FromIdentityCombiner(const PrimExpr& identity,

[GitHub] [tvm] MasterJH5574 commented on a change in pull request #9738: [TensorIR] Primitive "SetScope"

2021-12-14 Thread GitBox
MasterJH5574 commented on a change in pull request #9738: URL: https://github.com/apache/tvm/pull/9738#discussion_r769151080 ## File path: src/tir/schedule/primitive/block_annotate.cc ## @@ -16,6 +16,9 @@ * specific language governing permissions and limitations * under

[GitHub] [tvm] junrushao1994 commented on a change in pull request #9738: [TensorIR] Primitive "SetScope"

2021-12-14 Thread GitBox
junrushao1994 commented on a change in pull request #9738: URL: https://github.com/apache/tvm/pull/9738#discussion_r769150263 ## File path: src/tir/schedule/primitive/block_annotate.cc ## @@ -16,6 +16,9 @@ * specific language governing permissions and limitations * under

[GitHub] [tvm] MasterJH5574 commented on a change in pull request #9738: [TensorIR] Primitive "SetScope"

2021-12-14 Thread GitBox
MasterJH5574 commented on a change in pull request #9738: URL: https://github.com/apache/tvm/pull/9738#discussion_r769148372 ## File path: src/tir/schedule/primitive/block_annotate.cc ## @@ -16,6 +16,9 @@ * specific language governing permissions and limitations * under

[GitHub] [tvm] mbs-octoml commented on pull request #9745: [Relay] Add a unit test for structural equality

2021-12-14 Thread GitBox
mbs-octoml commented on pull request #9745: URL: https://github.com/apache/tvm/pull/9745#issuecomment-994115469 @comaniac might as well put this in. -- 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

[GitHub] [tvm] mbs-octoml opened a new pull request #9745: [Relay] Add a unit test for structural equality

2021-12-14 Thread GitBox
mbs-octoml opened a new pull request #9745: URL: https://github.com/apache/tvm/pull/9745 This is CORE-135 from the forums, which suggested structural equality was deeply broken. But unable to repro. No harm including unit test. -- This is an automated message from the Apache Git

[GitHub] [tvm] junrushao1994 commented on pull request #9527: [TIR] Allow compute_at create block predicate for non-trivial bounds and support floordiv pattern

2021-12-14 Thread GitBox
junrushao1994 commented on pull request #9527: URL: https://github.com/apache/tvm/pull/9527#issuecomment-994109980 CC @Hzfengsy -- 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

[GitHub] [tvm] junrushao1994 commented on pull request #9699: [TIR] Affine utility support freevars, iter lowerbound and diagnostics

2021-12-14 Thread GitBox
junrushao1994 commented on pull request #9699: URL: https://github.com/apache/tvm/pull/9699#issuecomment-994109651 Ping @spectrometerHBH -- 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

[tvm] branch main updated: Fix for tvm.build()'s name warning (#9678)

2021-12-14 Thread junrushao
This is an automated email from the ASF dual-hosted git repository. junrushao 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 21abb6e Fix for tvm.build()'s name warning (#9678)

[GitHub] [tvm] junrushao1994 commented on pull request #9602: [DRIVER] Specify name when build with primfunc

2021-12-14 Thread GitBox
junrushao1994 commented on pull request #9602: URL: https://github.com/apache/tvm/pull/9602#issuecomment-994109325 Thank you so much @wrongtest! -- 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

[GitHub] [tvm] junrushao1994 merged pull request #9678: Fix for tvm.build()'s name warning

2021-12-14 Thread GitBox
junrushao1994 merged pull request #9678: URL: https://github.com/apache/tvm/pull/9678 -- 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:

[GitHub] [tvm] junrushao1994 commented on pull request #9678: Fix for tvm.build()'s name warning

2021-12-14 Thread GitBox
junrushao1994 commented on pull request #9678: URL: https://github.com/apache/tvm/pull/9678#issuecomment-994109149 Sorry for the late reply! was a bit swamped last week. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [tvm] junrushao1994 commented on issue #9567: [Bug] Set `TVM_BACKTRACE=0` doesn't suppress C++ stacktrace

2021-12-14 Thread GitBox
junrushao1994 commented on issue #9567: URL: https://github.com/apache/tvm/issues/9567#issuecomment-994107868 @ZQPei Sorry for the late reply! That will be very helpful if we could potentially unify these environment variables. Please send an RFC and I'm more than happy to help! --

[GitHub] [tvm] junrushao1994 commented on a change in pull request #9743: [TIR][Schedule] Analysis functions to check if compute_inline and com…

2021-12-14 Thread GitBox
junrushao1994 commented on a change in pull request #9743: URL: https://github.com/apache/tvm/pull/9743#discussion_r769088046 ## File path: src/tir/schedule/primitive/compute_at.cc ## @@ -501,29 +503,59 @@ void ComputeAtOrReverseComputeAtImpl(ScheduleState self, const

[GitHub] [tvm] junrushao1994 commented on pull request #9744: [TIR] Add 'global_symbol' and 'tir.noalias' as default attributes in script auto completion

2021-12-14 Thread GitBox
junrushao1994 commented on pull request #9744: URL: https://github.com/apache/tvm/pull/9744#issuecomment-994091595 This is a trivial improvement over @Hzfengsy's TE-to-TIR bridge. CC @Lunderberg @mikepapadim -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [tvm] comaniac commented on a change in pull request #9743: [TIR][Schedule] Analysis functions to check if compute_inline and com…

2021-12-14 Thread GitBox
comaniac commented on a change in pull request #9743: URL: https://github.com/apache/tvm/pull/9743#discussion_r769076960 ## File path: src/tir/schedule/primitive/compute_inline.cc ## @@ -550,17 +567,32 @@ void ComputeInline(ScheduleState self, const StmtSRef&

[GitHub] [tvm] vinx13 opened a new pull request #9744: [TIR] Add 'global_symbol' and 'tir.noalias' as default attributes in script auto completion

2021-12-14 Thread GitBox
vinx13 opened a new pull request #9744: URL: https://github.com/apache/tvm/pull/9744 This PR adds `global_symbol` and `tir.noalias` as default attributes of PrimFunc from TIR. Co-authored-by: Siyuan Feng Co-authored-by: Bohan Hou

[GitHub] [tvm] masahi commented on a change in pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
masahi commented on a change in pull request #9737: URL: https://github.com/apache/tvm/pull/9737#discussion_r769075265 ## File path: python/tvm/contrib/cutlass/gen_conv2d.py ## @@ -121,27 +131,67 @@ def get_default(self, out_dtype): data_type =

[GitHub] [tvm] junrushao1994 commented on pull request #9742: [TIR][Schedule] Add Annotate/Unannotate primitive

2021-12-14 Thread GitBox
junrushao1994 commented on pull request #9742: URL: https://github.com/apache/tvm/pull/9742#issuecomment-994073515 This PR introduces two very simple schedule primitives: Annotate and Unannotate, which adds or removes block or loop attributes. CC: @mbs-octoml @jroesch @shingjan @YuchenJin

[GitHub] [tvm] junrushao1994 commented on pull request #9743: [TIR][Schedule] Analysis functions to check if compute_inline and com…

2021-12-14 Thread GitBox
junrushao1994 commented on pull request #9743: URL: https://github.com/apache/tvm/pull/9743#issuecomment-994063993 CC: @comaniac @jcf94 @wrongtest @Hzfengsy @csullivan -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [tvm] comaniac commented on a change in pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
comaniac commented on a change in pull request #9737: URL: https://github.com/apache/tvm/pull/9737#discussion_r769071320 ## File path: python/tvm/contrib/cutlass/gen_conv2d.py ## @@ -121,27 +131,67 @@ def get_default(self, out_dtype): data_type =

[tvm] branch main updated: [CI] Hotfix Jenkinsfile (#9739)

2021-12-14 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen 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 7279c9d [CI] Hotfix Jenkinsfile (#9739) 7279c9d is

[GitHub] [tvm] tqchen merged pull request #9739: [CI] Hotfix Jenkinsfile

2021-12-14 Thread GitBox
tqchen merged pull request #9739: URL: https://github.com/apache/tvm/pull/9739 -- 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:

[GitHub] [tvm] vinx13 commented on a change in pull request #9727: [TE][TIR] Implement layout transformations, non-flat memory buffers

2021-12-14 Thread GitBox
vinx13 commented on a change in pull request #9727: URL: https://github.com/apache/tvm/pull/9727#discussion_r768136729 ## File path: src/tir/ir/index_map.cc ## @@ -0,0 +1,153 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license

[GitHub] [tvm] junrushao1994 commented on a change in pull request #9738: [TensorIR] Primitive "SetScope"

2021-12-14 Thread GitBox
junrushao1994 commented on a change in pull request #9738: URL: https://github.com/apache/tvm/pull/9738#discussion_r769046705 ## File path: src/tir/schedule/primitive/block_annotate.cc ## @@ -16,6 +16,9 @@ * specific language governing permissions and limitations * under

[GitHub] [tvm] junrushao1994 commented on a change in pull request #9738: [TensorIR] Primitive "SetScope"

2021-12-14 Thread GitBox
junrushao1994 commented on a change in pull request #9738: URL: https://github.com/apache/tvm/pull/9738#discussion_r769040804 ## File path: src/tir/schedule/analysis.h ## @@ -393,6 +393,10 @@ std::vector> GetReducerGetters() bool FromIdentityCombiner(const PrimExpr&

[GitHub] [tvm] masahi commented on a change in pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
masahi commented on a change in pull request #9737: URL: https://github.com/apache/tvm/pull/9737#discussion_r769040090 ## File path: python/tvm/contrib/cutlass/gen_conv2d.py ## @@ -121,27 +131,70 @@ def get_default(self, out_dtype): data_type =

[GitHub] [tvm] comaniac commented on a change in pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
comaniac commented on a change in pull request #9737: URL: https://github.com/apache/tvm/pull/9737#discussion_r769005791 ## File path: python/tvm/contrib/cutlass/gen_conv2d.py ## @@ -121,27 +131,70 @@ def get_default(self, out_dtype): data_type =

[GitHub] [tvm] vinx13 opened a new pull request #9743: [TIR][Schedule] Analysis functions to check if compute_inline and com…

2021-12-14 Thread GitBox
vinx13 opened a new pull request #9743: URL: https://github.com/apache/tvm/pull/9743 …pute_inline is allowed Co-authored-by: Siyuan Feng Co-authored-by: Bohan Hou <32121147+spectrometer...@users.noreply.github.com> Co-authored-by: Hongyi Jin <3231950...@qq.com>

[GitHub] [tvm] tkonolige commented on a change in pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
tkonolige commented on a change in pull request #9737: URL: https://github.com/apache/tvm/pull/9737#discussion_r768999703 ## File path: python/tvm/contrib/cutlass/conv2d_profiler.py ## @@ -0,0 +1,163 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [tvm] vinx13 opened a new pull request #9742: [TIR][Schedule] Add Annotate/Unannotate primitive

2021-12-14 Thread GitBox
vinx13 opened a new pull request #9742: URL: https://github.com/apache/tvm/pull/9742 Co-authored-by: Siyuan Feng Co-authored-by: Bohan Hou <32121147+spectrometer...@users.noreply.github.com> Co-authored-by: Hongyi Jin <3231950...@qq.com> Co-authored-by: Ruihang Lai

[GitHub] [tvm] masahi edited a comment on pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
masahi edited a comment on pull request #9737: URL: https://github.com/apache/tvm/pull/9737#issuecomment-993909962 > @masahi Maybe the reason why the TVM script takes so long is that you are doing 100 iterations per benchmark where as the cutlass script is only doing 20? I believe

[GitHub] [tvm] masahi commented on pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
masahi commented on pull request #9737: URL: https://github.com/apache/tvm/pull/9737#issuecomment-993909962 > @masahi Maybe the reason why the TVM script takes so long is that you are doing 100 iterations per benchmark where as the cutlass script is only doing 20? I believe

[GitHub] [tvm] ryanstout edited a comment on issue #5580: [ONNX] check-fail with max_pool2d operator

2021-12-14 Thread GitBox
ryanstout edited a comment on issue #5580: URL: https://github.com/apache/tvm/issues/5580#issuecomment-993908033 @areusch did you figure this out, I'm seeing the same issue. I'm doing a lookup at one point: ``` out = lut[x.long()] ``` Were you doing something similar?

[GitHub] [tvm] ryanstout commented on issue #5580: [ONNX] check-fail with max_pool2d operator

2021-12-14 Thread GitBox
ryanstout commented on issue #5580: URL: https://github.com/apache/tvm/issues/5580#issuecomment-993908033 @areusch did you figure this out, I'm seeing the same issue. I'm doing a lookup at one point: ``` out = lut[x.long()] ``` Were you doing something similar? (my onnx

[GitHub] [tvm] masahi commented on a change in pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
masahi commented on a change in pull request #9737: URL: https://github.com/apache/tvm/pull/9737#discussion_r768971377 ## File path: python/tvm/contrib/cutlass/conv2d_profiler.py ## @@ -0,0 +1,163 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [tvm] masahi commented on a change in pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
masahi commented on a change in pull request #9737: URL: https://github.com/apache/tvm/pull/9737#discussion_r768970900 ## File path: python/tvm/contrib/cutlass/conv2d_profiler.py ## @@ -0,0 +1,163 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [tvm] masahi commented on a change in pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
masahi commented on a change in pull request #9737: URL: https://github.com/apache/tvm/pull/9737#discussion_r768968554 ## File path: python/tvm/contrib/cutlass/conv2d_profiler.py ## @@ -0,0 +1,163 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [tvm] masahi commented on a change in pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
masahi commented on a change in pull request #9737: URL: https://github.com/apache/tvm/pull/9737#discussion_r768961263 ## File path: python/tvm/contrib/cutlass/conv2d_profiler.py ## @@ -0,0 +1,163 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9727: [TE][TIR] Implement layout transformations, non-flat memory buffers

2021-12-14 Thread GitBox
mbs-octoml commented on a change in pull request #9727: URL: https://github.com/apache/tvm/pull/9727#discussion_r768910010 ## File path: include/tvm/te/schedule.h ## @@ -466,9 +498,27 @@ class StageNode : public Object { * while origin_op remains fixed. */

[GitHub] [tvm] masahi commented on a change in pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
masahi commented on a change in pull request #9737: URL: https://github.com/apache/tvm/pull/9737#discussion_r768960030 ## File path: python/tvm/relay/op/strategy/cuda.py ## @@ -324,7 +324,10 @@ def conv2d_strategy_cuda(attrs, inputs, out_type, target):

[GitHub] [tvm] manupa-arm commented on issue #9741: [CI Problem] Nightly docker re-build validations use different commits for CMake build and testing

2021-12-14 Thread GitBox
manupa-arm commented on issue #9741: URL: https://github.com/apache/tvm/issues/9741#issuecomment-993867843 cc : @leandron @areusch @tqchen @mbs-octoml @grant-arm -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [tvm] manupa-arm opened a new issue #9741: [CI Problem] Nightly docker re-build validations use different commits for CMake build and testing

2021-12-14 Thread GitBox
manupa-arm opened a new issue #9741: URL: https://github.com/apache/tvm/issues/9741 Where ? https://ci.tlcpack.ai/job/docker-images-ci/job/daily-docker-image-validate/45/ calls another Job that shows the issue:

[tvm] branch main updated (ceec0fc -> fb99383)

2021-12-14 Thread mbrookhart
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 ceec0fc [microNPU] Update Arm(R) Ethos(TM)-U55 NPU demo README (#9725) add fb99383 [Relay] Re-run PlanDevices

[GitHub] [tvm] mbrookhart merged pull request #9613: [Relay] Re-run PlanDevices after LowerTE to flow new memory scope constraints.

2021-12-14 Thread GitBox
mbrookhart merged pull request #9613: URL: https://github.com/apache/tvm/pull/9613 -- 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:

[GitHub] [tvm] mbrookhart commented on pull request #9613: [Relay] Re-run PlanDevices after LowerTE to flow new memory scope constraints.

2021-12-14 Thread GitBox
mbrookhart commented on pull request #9613: URL: https://github.com/apache/tvm/pull/9613#issuecomment-993851060 Thanks @mbs-octoml @jroesch -- 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

[GitHub] [tvm] tkonolige commented on pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
tkonolige commented on pull request #9737: URL: https://github.com/apache/tvm/pull/9737#issuecomment-993839104 @masahi Maybe the reason why the TVM script takes so long is that you are doing 100 iterations per benchmark where as the cutlass script is only doing 20? Also the TVM script is

[GitHub] [tvm] tkonolige commented on a change in pull request #9737: [CUTLASS] Add conv2d profiler

2021-12-14 Thread GitBox
tkonolige commented on a change in pull request #9737: URL: https://github.com/apache/tvm/pull/9737#discussion_r768913178 ## File path: python/tvm/contrib/cutlass/conv2d_profiler.py ## @@ -0,0 +1,163 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [tvm] mehrdadh commented on a change in pull request #9697: Add Module-Based Model Runtime Interface for AOT (support C++ runtime)

2021-12-14 Thread GitBox
mehrdadh commented on a change in pull request #9697: URL: https://github.com/apache/tvm/pull/9697#discussion_r766992034 ## File path: tests/python/relay/aot/test_cpp_aot.py ## @@ -0,0 +1,88 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [tvm] manupa-arm commented on pull request #9565: [TIR][USMP] Integrating USMP to AoT Executor

2021-12-14 Thread GitBox
manupa-arm commented on pull request #9565: URL: https://github.com/apache/tvm/pull/9565#issuecomment-993831166 @mbs-octoml @ekalda @lhutton1 @mbaret for reviews. The CI is broken for various other reasons. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [tvm-rfcs] vinx13 commented on pull request #39: [RFC][TIR] Layout transformations on buffer access

2021-12-14 Thread GitBox
vinx13 commented on pull request #39: URL: https://github.com/apache/tvm-rfcs/pull/39#issuecomment-993828128 @wrongtest I'm working on the TensorIR side and have a draft version of `transform_layout`. The current implementation is ```void TransformLayout(ScheduleState self, const

[GitHub] [tvm] junrushao1994 edited a comment on pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-14 Thread GitBox
junrushao1994 edited a comment on pull request #9729: URL: https://github.com/apache/tvm/pull/9729#issuecomment-993208836 Thank you everyone for raising the PR and providing important context on this PR! I really appreciate that we come together and discuss about architectural changes.

[GitHub] [tvm] mbs-octoml commented on pull request #9739: [CI] Hotfix Jenkinsfile

2021-12-14 Thread GitBox
mbs-octoml commented on pull request #9739: URL: https://github.com/apache/tvm/pull/9739#issuecomment-993816026 Thanks @driazati , no worries about the break, all for The Greater Good. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [tvm] mbs-octoml commented on a change in pull request #9723: [Frontend] Add Span filling for frontends to Relay

2021-12-14 Thread GitBox
mbs-octoml commented on a change in pull request #9723: URL: https://github.com/apache/tvm/pull/9723#discussion_r768889160 ## File path: python/tvm/relay/frontend/common.py ## @@ -954,3 +955,53 @@ def try_resolve_var_to_const(x, graph_params): return _op.const(value,

[GitHub] [tvm] areusch commented on issue #9705: [CI Problem] CI node out of memory

2021-12-14 Thread GitBox
areusch commented on issue #9705: URL: https://github.com/apache/tvm/issues/9705#issuecomment-993806967 from Noah: - docker prune script on static nodes (which I have running as a cron job) doesn't pass the 'volumes' flag - manually fixed 5 nodes which were full (think 246 was one of

[GitHub] [tvm] junrushao1994 edited a comment on pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-14 Thread GitBox
junrushao1994 edited a comment on pull request #9729: URL: https://github.com/apache/tvm/pull/9729#issuecomment-993208836 Thank you everyone for raising the PR and providing important context on this PR! I really appreciate that we come together and discuss about architectural changes.

[GitHub] [tvm] junrushao1994 commented on issue #9714: [Docs] An unclear definition of TensorIR

2021-12-14 Thread GitBox
junrushao1994 commented on issue #9714: URL: https://github.com/apache/tvm/issues/9714#issuecomment-993797839 CC @Hzfengsy -- 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.

[GitHub] [tvm] junrushao1994 edited a comment on pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-14 Thread GitBox
junrushao1994 edited a comment on pull request #9729: URL: https://github.com/apache/tvm/pull/9729#issuecomment-993208836 Thank you everyone for raising the PR and providing important context on this PR! I really appreciate that we come together and discuss architectural changes. These

[GitHub] [tvm] junrushao1994 edited a comment on pull request #9729: [CORE] Make external mods first class field in the IR module

2021-12-14 Thread GitBox
junrushao1994 edited a comment on pull request #9729: URL: https://github.com/apache/tvm/pull/9729#issuecomment-993208836 Thank you everyone for raising the PR and providing important context on this PR! I really appreciate that we come together and discuss architectural changes. These

[GitHub] [tvm] leandron merged pull request #9725: [microNPU] Update Arm(R) Ethos(TM)-U55 NPU demo README

2021-12-14 Thread GitBox
leandron merged pull request #9725: URL: https://github.com/apache/tvm/pull/9725 -- 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:

[tvm] branch main updated (af869bc -> ceec0fc)

2021-12-14 Thread leandron
This is an automated email from the ASF dual-hosted git repository. leandron pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tvm.git. from af869bc Merge Java unittests into GPU unittests (#9732) add ceec0fc [microNPU] Update Arm(R) Ethos(TM)-U55 NPU

[GitHub] [tvm] u99127 commented on issue #9703: [Bug][CI] Python 3.6 used in TVM CI is reaching EOL

2021-12-14 Thread GitBox
u99127 commented on issue #9703: URL: https://github.com/apache/tvm/issues/9703#issuecomment-993729017 I'd suggest python 3.8 as that seems to be the best bet for reasons as you suggest. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [tvm] grant-arm commented on a change in pull request #9725: [microNPU] Update Arm(R) Ethos(TM)-U55 NPU demo README

2021-12-14 Thread GitBox
grant-arm commented on a change in pull request #9725: URL: https://github.com/apache/tvm/pull/9725#discussion_r768840580 ## File path: apps/microtvm/ethosu/README.md ## @@ -43,6 +43,10 @@ If the demo is not run in the ci_cpu Docker container, then you will need the fo You

[GitHub] [tvm] junrushao1994 commented on pull request #9739: [CI] Hotfix Jenkinsfile

2021-12-14 Thread GitBox
junrushao1994 commented on pull request #9739: URL: https://github.com/apache/tvm/pull/9739#issuecomment-993709359 Yeah the recent jenkins change brought a lot of cool features that we all like a lot. Not sure why this regression is introduced, but will be great if we could follow the

[GitHub] [tvm] tqchen opened a new pull request #9739: [CI] Hotfix Jenkinsfile

2021-12-14 Thread GitBox
tqchen opened a new pull request #9739: URL: https://github.com/apache/tvm/pull/9739 Due to the current setup jenkins file changes needs to go through docker staging branch cc @driazati @areusch -- This is an automated message from the Apache Git Service. To respond to the

[tvm] 01/01: [CI] Hotfix Jenkinsfile

2021-12-14 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen pushed a commit to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git commit f960ab074044e04e7bf3bcfe27b74497ec5a967c Author: tqchen AuthorDate: Tue Dec 14 10:52:50 2021 -0500 [CI]

[tvm] branch ci-docker-staging updated (0710915 -> f960ab0)

2021-12-14 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen pushed a change to branch ci-docker-staging in repository https://gitbox.apache.org/repos/asf/tvm.git. omit 0710915 Add test for [skip ci] in commit but not PR title omit 1c3b526 Fix test case omit 70a9bcd

[GitHub] [tvm] MasterJH5574 commented on a change in pull request #9738: [TensorIR] Primitive "SetScope"

2021-12-14 Thread GitBox
MasterJH5574 commented on a change in pull request #9738: URL: https://github.com/apache/tvm/pull/9738#discussion_r768776048 ## File path: src/tir/schedule/primitive/block_annotate.cc ## @@ -233,14 +238,139 @@ class StorageAlignInvalidAnnotationError : public ScheduleError {

[GitHub] [tvm] MasterJH5574 commented on a change in pull request #9738: [TensorIR] Primitive "SetScope"

2021-12-14 Thread GitBox
MasterJH5574 commented on a change in pull request #9738: URL: https://github.com/apache/tvm/pull/9738#discussion_r768729478 ## File path: src/tir/schedule/primitive/block_annotate.cc ## @@ -16,6 +16,9 @@ * specific language governing permissions and limitations * under

[GitHub] [tvm] d-smirnov commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
d-smirnov commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768721244 ## File path: src/tir/usmp/algo/hill_climb.cc ## @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [tvm] d-smirnov commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
d-smirnov commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768698542 ## File path: src/tir/usmp/algo/hill_climb.cc ## @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [tvm] Hzfengsy commented on a change in pull request #9738: [TensorIR] Primitive "SetScope"

2021-12-14 Thread GitBox
Hzfengsy commented on a change in pull request #9738: URL: https://github.com/apache/tvm/pull/9738#discussion_r768682378 ## File path: src/tir/schedule/primitive/block_annotate.cc ## @@ -16,6 +16,9 @@ * specific language governing permissions and limitations * under the

[GitHub] [tvm] d-smirnov commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
d-smirnov commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768686617 ## File path: tests/python/unittest/test_tir_usmp_algo_hill_climb.py ## @@ -0,0 +1,409 @@ +# Licensed to the Apache Software Foundation (ASF) under one +#

[GitHub] [tvm] d-smirnov commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
d-smirnov commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768685234 ## File path: tests/python/unittest/test_tir_usmp_algo_hill_climb.py ## @@ -0,0 +1,409 @@ +# Licensed to the Apache Software Foundation (ASF) under one +#

[GitHub] [tvm] d-smirnov commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
d-smirnov commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768640273 ## File path: tests/python/unittest/test_tir_usmp_algo_hill_climb.py ## @@ -0,0 +1,409 @@ +# Licensed to the Apache Software Foundation (ASF) under one +#

[GitHub] [tvm] d-smirnov commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
d-smirnov commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768639871 ## File path: tests/python/unittest/test_tir_usmp_algo_hill_climb.py ## @@ -0,0 +1,409 @@ +# Licensed to the Apache Software Foundation (ASF) under one +#

[GitHub] [tvm] d-smirnov commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
d-smirnov commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768639434 ## File path: tests/python/unittest/test_tir_usmp_algo_hill_climb.py ## @@ -0,0 +1,409 @@ +# Licensed to the Apache Software Foundation (ASF) under one +#

[GitHub] [tvm] manupa-arm commented on pull request #8509: [TIR] Tir constants integration into compilation pipeline

2021-12-14 Thread GitBox
manupa-arm commented on pull request #8509: URL: https://github.com/apache/tvm/pull/8509#issuecomment-993484242 @d-smirnov , Since this is a follow up PR (thus, cannot be merged before the dependent one is merged), shall we put this to draft for now ? -- This is an automated message

[GitHub] [tvm] manupa-arm commented on pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
manupa-arm commented on pull request #9704: URL: https://github.com/apache/tvm/pull/9704#issuecomment-993475422 Should we merge the testing here into https://github.com/apache/tvm/blob/af869bc79cecdc18371862524ea241803dc78c7a/tests/python/unittest/test_tir_usmp_algo.py ? By doing

[GitHub] [tvm] manupa-arm commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
manupa-arm commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768598115 ## File path: src/tir/usmp/algo/hill_climb.cc ## @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [tvm] MasterJH5574 opened a new pull request #9738: [TensorIR] Primitive "SetScope"

2021-12-14 Thread GitBox
MasterJH5574 opened a new pull request #9738: URL: https://github.com/apache/tvm/pull/9738 This PR introduces "SetScope", a schedule primitive which helps update the storage scope of a buffer. cc @Hzfengsy @junrushao1994 -- This is an automated message from the Apache Git

[GitHub] [tvm] manupa-arm commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
manupa-arm commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768587455 ## File path: src/tir/usmp/algo/hill_climb.cc ## @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [tvm] d-smirnov commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
d-smirnov commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768581660 ## File path: tests/python/unittest/test_tir_usmp_algo_hill_climb.py ## @@ -0,0 +1,409 @@ +# Licensed to the Apache Software Foundation (ASF) under one +#

[GitHub] [tvm] d-smirnov commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
d-smirnov commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768573871 ## File path: src/tir/usmp/algo/hill_climb.cc ## @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [tvm] d-smirnov commented on a change in pull request #9704: [USMP] Hill Climb allocator

2021-12-14 Thread GitBox
d-smirnov commented on a change in pull request #9704: URL: https://github.com/apache/tvm/pull/9704#discussion_r768558039 ## File path: src/tir/usmp/algo/hill_climb.cc ## @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

  1   2   >