[GitHub] [incubator-tvm] kparzysz-quic commented on pull request #5487: [Hexagon] Change "scalar" and "stack" in IDL from "inrout" to "in"

kparzysz-quic commented on pull request #5487: URL: https://github.com/apache/incubator-tvm/pull/5487#issuecomment-621974419 Yes, we have tested this change. This is an automated message from the Apache Git Service. To

[GitHub] [incubator-tvm] tqchen commented on pull request #5436: [TFLite Runtime] Re-enable test for remote execution via RPC

tqchen commented on pull request #5436: URL: https://github.com/apache/incubator-tvm/pull/5436#issuecomment-621962718 Please check the ci error https://ci.tvm.ai/blue/organizations/jenkins/tvm/detail/PR-5436/4/pipeline seems was due to changes to the cpu build config file.

[incubator-tvm] branch master updated: [Fix] Add ConstantNode to IsAtomic (#5457)

This is an automated email from the ASF dual-hosted git repository. kevinthesun pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git The following commit(s) were added to refs/heads/master by this push: new ae89afe [Fix] Add ConstantNode to

[GitHub] [incubator-tvm] kevinthesun commented on pull request #5457: [Fix] Add ConstantNode to IsAtomic

kevinthesun commented on pull request #5457: URL: https://github.com/apache/incubator-tvm/pull/5457#issuecomment-621979744 Thanks @zhiics @MarisaKirisame This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-tvm] kazum opened a new pull request #5489: [Rust] Fixes for wasm32 target

kazum opened a new pull request #5489: URL: https://github.com/apache/incubator-tvm/pull/5489 This PR fixes warnings and errors when targeting wasm32. - Update the BackendPackedCFunc signature which was changed in #4637. - Use derive_default() for bindgen to handle the generated

[GitHub] [incubator-tvm] tqchen commented on issue #5221: TextPrinter for PrimFunc in IRModule

tqchen commented on issue #5221: URL: https://github.com/apache/incubator-tvm/issues/5221#issuecomment-622051555 close as we have a printable version, see also https://github.com/apache/incubator-tvm/pull/5483 This is an

[GitHub] [incubator-tvm] tqchen commented on issue #5396: [DOCS] Sphinx docs warning fixes

tqchen commented on issue #5396: URL: https://github.com/apache/incubator-tvm/issues/5396#issuecomment-622050912 All the docs change are landed by now This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-tvm-vta] huajsj commented on pull request #5: Add c++ and python local deploy example

huajsj commented on pull request #5: URL: https://github.com/apache/incubator-tvm-vta/pull/5#issuecomment-621996958 @tmoreau89 , thanks for the kindly follow up, the code& test done, if you have time, could you help to have a review? Thanks.

[GitHub] [incubator-tvm] kazum commented on issue #5464: [OpenCL] `directly 4 8 bit int in integer` causes compiling error

kazum commented on issue #5464: URL: https://github.com/apache/incubator-tvm/issues/5464#issuecomment-622013436 It looks wrong and should be fixed, I think. This is an automated message from the Apache Git Service. To

[GitHub] [incubator-tvm] tqchen opened a new issue #5490: [REFACTOR] std::string -> String Migration in IR nodes

tqchen opened a new issue #5490: URL: https://github.com/apache/incubator-tvm/issues/5490 This is an issue to track the progress of std::string -> String migration in IR Nodes. - [ ] Merge initial example https://github.com/apache/incubator-tvm/pull/5438 - [ ] Convert relay nodes

[GitHub] [incubator-tvm] tqchen commented on issue #5490: [REFACTOR] std::string -> String Migration in IR nodes

tqchen commented on issue #5490: URL: https://github.com/apache/incubator-tvm/issues/5490#issuecomment-622050597 cc @icemelon9 @jroesch @zhiics please chime in if you would like to take a stab This is an automated message

[GitHub] [incubator-tvm-vta] huajsj commented on pull request #7: [pynq_driver] fix device early return

huajsj commented on pull request #7: URL: https://github.com/apache/incubator-tvm-vta/pull/7#issuecomment-621981787 @remotego , thanks for the detailed explain, the logic make sense, code LGTM. This is an automated

[GitHub] [incubator-tvm] tmoreau89 commented on pull request #5417: [RUNTIME][uTVM] AutoTVM + uTVM for Cortex-M7

tmoreau89 commented on pull request #5417: URL: https://github.com/apache/incubator-tvm/pull/5417#issuecomment-622010729 Thanks @areusch , @liangfu @weberlo @u99127 the PR has been merged This is an automated message from

[incubator-tvm] branch master updated (8d72496 -> 9bbf58a)

This is an automated email from the ASF dual-hosted git repository. tqchen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from 8d72496 [RUNTIME][uTVM] AutoTVM + uTVM for Cortex-M7 (#5417) add 9bbf58a Removing older Object

[GitHub] [incubator-tvm] kparzysz-quic opened a new pull request #5491: Make "none" DataType explicit

kparzysz-quic opened a new pull request #5491: URL: https://github.com/apache/incubator-tvm/pull/5491 The `None` data type is created when converting an empty string to `DataType`. Add functions to create it and recognize it. Convert it to the `void` LLVM type in LLVM codegen (it

[GitHub] [incubator-tvm] zhiics opened a new pull request #5493: [REFACTOR][BOYC] Non recursive partitioning

zhiics opened a new pull request #5493: URL: https://github.com/apache/incubator-tvm/pull/5493 This PR refactors the partitioning pass by using non-recursive mutator. It also removes the unnecessary mutators as we only need to look at begin/end annotations which are definitely wrapped in

[GitHub] [incubator-tvm] zhiics commented on pull request #5493: [REFACTOR][BOYC] Non recursive partitioning

zhiics commented on pull request #5493: URL: https://github.com/apache/incubator-tvm/pull/5493#issuecomment-622129716 @mbrookhart please take a look for the mixed mutator pattern. BTW, we will still need to refactor the infertype pass as it is the most frequently used pass.

[GitHub] [incubator-tvm] kparzysz-quic commented on a change in pull request #5491: Make "none" DataType explicit

kparzysz-quic commented on a change in pull request #5491: URL: https://github.com/apache/incubator-tvm/pull/5491#discussion_r418270642 ## File path: include/tvm/runtime/data_type.h ## @@ -211,6 +215,13 @@ class DataType { static DataType Handle(int bits = 64, int lanes =

[GitHub] [incubator-tvm] jroesch commented on a change in pull request #5467: [Relay]Improve Shape Func handling for Tuple inputs

jroesch commented on a change in pull request #5467: URL: https://github.com/apache/incubator-tvm/pull/5467#discussion_r418270872 ## File path: src/relay/op/memory/memory.cc ## @@ -360,12 +360,26 @@ bool ShapeFuncRel(const Array& types, int num_inputs, const Attrs& attrs,

[GitHub] [incubator-tvm] comaniac commented on pull request #5493: [REFACTOR][BOYC] Non recursive partitioning

comaniac commented on pull request #5493: URL: https://github.com/apache/incubator-tvm/pull/5493#issuecomment-622139182 Ah I think that's because I manually ran clang-format for the file. We should definitely build style checker in CI.

[GitHub] [incubator-tvm] lixiaoquan commented on a change in pull request #5429: [RELAY][TF] Support symbolic newshape for Reshape

lixiaoquan commented on a change in pull request #5429: URL: https://github.com/apache/incubator-tvm/pull/5429#discussion_r418341379 ## File path: include/tvm/relay/op_attr_types.h ## @@ -81,10 +81,16 @@ using TOpIsStateful = bool; */ using TNonComputational = bool; +enum

[GitHub] [incubator-tvm] tqchen commented on a change in pull request #5491: Make "none" DataType explicit

tqchen commented on a change in pull request #5491: URL: https://github.com/apache/incubator-tvm/pull/5491#discussion_r418253950 ## File path: include/tvm/runtime/data_type.h ## @@ -211,6 +215,13 @@ class DataType { static DataType Handle(int bits = 64, int lanes = 1) {

[GitHub] [incubator-tvm] kevinthesun commented on a change in pull request #5467: [Relay]Improve Shape Func handling for Tuple inputs

kevinthesun commented on a change in pull request #5467: URL: https://github.com/apache/incubator-tvm/pull/5467#discussion_r418284406 ## File path: src/relay/op/memory/memory.cc ## @@ -360,12 +360,26 @@ bool ShapeFuncRel(const Array& types, int num_inputs, const Attrs& attrs,

[incubator-tvm] branch master updated (9bbf58a -> 3aa103e)

This is an automated email from the ASF dual-hosted git repository. tqchen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from 9bbf58a Removing older Object detection TFlite test (#5477) add 3aa103e [IR] Initial stab at

[GitHub] [incubator-tvm] kparzysz-quic opened a new pull request #5492: [RUNTIME] Hexagon driver for offloading kernels to simulator

kparzysz-quic opened a new pull request #5492: URL: https://github.com/apache/incubator-tvm/pull/5492 The driver (`sim_dev` executable) is the process running on the Hexagon simulator that handles the Hexagon-side communication with the TVM runtime running on x86. The x86-side is

[GitHub] [incubator-tvm] ehsanmok commented on pull request #5489: [Rust] Fixes for wasm32 target

ehsanmok commented on pull request #5489: URL: https://github.com/apache/incubator-tvm/pull/5489#issuecomment-622074069 Thanks @kazum! LGTM. This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-tvm] kparzysz-quic commented on a change in pull request #5491: Make "none" DataType explicit

kparzysz-quic commented on a change in pull request #5491: URL: https://github.com/apache/incubator-tvm/pull/5491#discussion_r418263860 ## File path: include/tvm/runtime/data_type.h ## @@ -211,6 +215,13 @@ class DataType { static DataType Handle(int bits = 64, int lanes =

[GitHub] [incubator-tvm] jroesch commented on pull request #5489: [Rust] Fixes for wasm32 target

jroesch commented on pull request #5489: URL: https://github.com/apache/incubator-tvm/pull/5489#issuecomment-622091138 Could you actually add some kind of test which checks that we compile correctly for WASM target? This

[GitHub] [incubator-tvm] tqchen commented on a change in pull request #5491: Make "none" DataType explicit

tqchen commented on a change in pull request #5491: URL: https://github.com/apache/incubator-tvm/pull/5491#discussion_r418268635 ## File path: include/tvm/runtime/data_type.h ## @@ -211,6 +215,13 @@ class DataType { static DataType Handle(int bits = 64, int lanes = 1) {

[GitHub] [incubator-tvm] tqchen commented on pull request #5483: [TIR][Printer] text format printer considering future parsing use

tqchen commented on pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#issuecomment-622071330 - Allocate `float32x32` is not the actual data type. Perhaps we can just show allocate the flattened size since that is the semantics. - eval is not necessary since

[GitHub] [incubator-tvm] tqchen edited a comment on pull request #5483: [TIR][Printer] text format printer considering future parsing use

tqchen edited a comment on pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#issuecomment-622071330 - Allocate `float32x32` is not the actual data type. Perhaps we can just show allocate the flattened size since that is the semantics. - eval is not necessary

[GitHub] [incubator-tvm] tqchen edited a comment on pull request #5483: [TIR][Printer] text format printer considering future parsing use

tqchen edited a comment on pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#issuecomment-622071330 - Allocate `float32x32` is not the actual data type. Perhaps we can just show allocate the flattened size since that is the semantics. - eval is not necessary

[GitHub] [incubator-tvm] kevinthesun commented on a change in pull request #5467: [Relay]Improve Shape Func handling for Tuple inputs

kevinthesun commented on a change in pull request #5467: URL: https://github.com/apache/incubator-tvm/pull/5467#discussion_r418284406 ## File path: src/relay/op/memory/memory.cc ## @@ -360,12 +360,26 @@ bool ShapeFuncRel(const Array& types, int num_inputs, const Attrs& attrs,

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

junrushao1994 commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418371862 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] spectrometerHBH commented on pull request #5483: [TIR][Printer] text format printer considering future parsing use

spectrometerHBH commented on pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#issuecomment-622203710 > Some comments: > > 1. There are more than one spaces before the left brace in the allocation line >``` >allocate(B.local, float32, [64]) {

[GitHub] [incubator-tvm] spectrometerHBH edited a comment on pull request #5483: [TIR][Printer] text format printer considering future parsing use

spectrometerHBH edited a comment on pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#issuecomment-622203710 > Some comments: > > 1. There are more than one spaces before the left brace in the allocation line >``` >allocate(B.local, float32,

[GitHub] [incubator-tvm] spectrometerHBH commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

spectrometerHBH commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418378401 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

[GitHub] [incubator-tvm] spectrometerHBH commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

spectrometerHBH commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418406985 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

[incubator-tvm-vta] branch master updated: [pynq_driver] fix device early return (#7)

This is an automated email from the ASF dual-hosted git repository. moreau pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm-vta.git The following commit(s) were added to refs/heads/master by this push: new 21937a0 [pynq_driver] fix device

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

junrushao1994 commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418368333 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] spectrometerHBH edited a comment on pull request #5483: [TIR][Printer] text format printer considering future parsing use

spectrometerHBH edited a comment on pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#issuecomment-622203710 > Some comments: > > 1. There are more than one spaces before the left brace in the allocation line >``` >allocate(B.local, float32,

[GitHub] [incubator-tvm] spectrometerHBH edited a comment on pull request #5483: [TIR][Printer] text format printer considering future parsing use

spectrometerHBH edited a comment on pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#issuecomment-622238350 To avoid making this page too long, I will edit the examples for reference in the top comment if I change the format.

[GitHub] [incubator-tvm] spectrometerHBH commented on pull request #5483: [TIR][Printer] text format printer considering future parsing use

spectrometerHBH commented on pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#issuecomment-622238350 To avoid making this page too long, I will edit the examples for reference in the top if I change the format.

[GitHub] [incubator-tvm-vta] tmoreau89 commented on a change in pull request #5: Add c++ and python local deploy example

tmoreau89 commented on a change in pull request #5: URL: https://github.com/apache/incubator-tvm-vta/pull/5#discussion_r418425846 ## File path: apps/deploy/Makefile ## @@ -0,0 +1,71 @@ +#licensed to the Apache Software Foundation (ASF) under one Review comment:

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

junrushao1994 commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418367759 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

junrushao1994 commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418371548 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

junrushao1994 commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418401901 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

junrushao1994 commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418365372 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

junrushao1994 commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418370164 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] spectrometerHBH commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

spectrometerHBH commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418400818 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

[GitHub] [incubator-tvm] spectrometerHBH commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

spectrometerHBH commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418401170 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

[GitHub] [incubator-tvm] tqchen commented on pull request #5491: Make "none" DataType explicit

tqchen commented on pull request #5491: URL: https://github.com/apache/incubator-tvm/pull/5491#issuecomment-622250434 Thanks @kparzysz-quic ! This is an automated message from the Apache Git Service. To respond to the

[incubator-tvm] branch master updated: Make "none" DataType explicit (#5491)

This is an automated email from the ASF dual-hosted git repository. tqchen pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git The following commit(s) were added to refs/heads/master by this push: new 12e737f Make "none" DataType explicit

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

junrushao1994 commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418364252 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

junrushao1994 commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418368333 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5483: [TIR][Printer] text format printer considering future parsing use

junrushao1994 commented on a change in pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#discussion_r418367759 ## File path: src/printer/tir_text_printer.cc ## @@ -0,0 +1,735 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

[GitHub] [incubator-tvm] spectrometerHBH edited a comment on pull request #5483: [TIR][Printer] text format printer considering future parsing use

spectrometerHBH edited a comment on pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#issuecomment-622203710 > Some comments: > > 1. There are more than one spaces before the left brace in the allocation line >``` >allocate(B.local, float32,

[GitHub] [incubator-tvm] kparzysz-quic opened a new pull request #5487: [Hexagon] Change "scalar" and "stack" in IDL from "inrout" to "in"

kparzysz-quic opened a new pull request #5487: URL: https://github.com/apache/incubator-tvm/pull/5487 This changes the `scalar` and `stack` parameters to `kernel` to be input parameters only, as suggested by @FrozenGene .

[GitHub] [incubator-tvm] siju-samuel opened a new pull request #5486: [TFLITE]Select op support for tflite frontend

siju-samuel opened a new pull request #5486: URL: https://github.com/apache/incubator-tvm/pull/5486 Added the support of `select` op in tflite frontend. @FrozenGene @masahi please help me to review and merge this PR. TIA Note: tflite `where` op is same as `select` in tflite.

[GitHub] [incubator-tvm] dhruvaray opened a new pull request #5488: [TFLITE] SELECT

dhruvaray opened a new pull request #5488: URL: https://github.com/apache/incubator-tvm/pull/5488 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[incubator-tvm] branch master updated (7ea834f -> 745c8a0)

This is an automated email from the ASF dual-hosted git repository. tqchen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from 7ea834f [team] add reviewer kparzysz-quic (#5482) add 745c8a0 [RUNTIME] Improved Packed FFI for

[incubator-tvm] branch master updated (745c8a0 -> 5d75992)

This is an automated email from the ASF dual-hosted git repository. tqchen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from 745c8a0 [RUNTIME] Improved Packed FFI for optional. (#5478) add 5d75992 [VTA] Fix VTA compile issue

[incubator-tvm] branch master updated: [team] add reviewer kparzysz-quic (#5482)

This is an automated email from the ASF dual-hosted git repository. tqchen pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git The following commit(s) were added to refs/heads/master by this push: new 7ea834f [team] add reviewer

[GitHub] [incubator-tvm] FrozenGene commented on pull request #5487: [Hexagon] Change "scalar" and "stack" in IDL from "inrout" to "in"

FrozenGene commented on pull request #5487: URL: https://github.com/apache/incubator-tvm/pull/5487#issuecomment-621936093 Thanks @kparzysz-quic . As we talked in https://github.com/apache/incubator-tvm/pull/5353, you mention that if we change to `in`, maybe cause cache/memory

[GitHub] [incubator-tvm] MarisaKirisame commented on pull request #5457: [Fix] Add ConstantNode to IsAtomic

MarisaKirisame commented on pull request #5457: URL: https://github.com/apache/incubator-tvm/pull/5457#issuecomment-621884928 @zhiics the ANF pass is pretty fast, and it seems like if you sum the size of the graph of all call to ANF, the size will be smaller then the original graph. so i

[GitHub] [incubator-tvm] FrozenGene commented on a change in pull request #5486: [TFLITE]Select op support for tflite frontend

FrozenGene commented on a change in pull request #5486: URL: https://github.com/apache/incubator-tvm/pull/5486#discussion_r418114942 ## File path: python/tvm/relay/frontend/tflite.py ## @@ -2357,6 +2371,20 @@ def get_expr(self, input_tensor_idx): def has_expr(self,

[GitHub] [incubator-tvm] Hzfengsy commented on pull request #5483: [TIR][Printer] text format printer considering future parsing use

Hzfengsy commented on pull request #5483: URL: https://github.com/apache/incubator-tvm/pull/5483#issuecomment-621810131 Some comments: 1. There are more than one spaces before the left brace in the allocation line ``` allocate(B.local, float32, [64]) { ```

[GitHub] [incubator-tvm] maheshambule commented on a change in pull request #5474: [Frontend][TFLite] ADD_N operator

maheshambule commented on a change in pull request #5474: URL: https://github.com/apache/incubator-tvm/pull/5474#discussion_r418116057 ## File path: python/tvm/relay/frontend/tflite.py ## @@ -774,6 +775,21 @@ def convert_square(self, op): return out +def

[GitHub] [incubator-tvm] michalpiszczek commented on pull request #5436: [TFLite Runtime] Re-enable test for remote execution via RPC

michalpiszczek commented on pull request #5436: URL: https://github.com/apache/incubator-tvm/pull/5436#issuecomment-621939200 @tqchen I believe I've done that here: https://github.com/apache/incubator-tvm/pull/5436/commits/530ee65c96db562745561092dcec43e136acbaec . I followed the pattern

[GitHub] [incubator-tvm] u99127 commented on a change in pull request #5474: [Frontend][TFLite] ADD_N operator

u99127 commented on a change in pull request #5474: URL: https://github.com/apache/incubator-tvm/pull/5474#discussion_r418110316 ## File path: python/tvm/relay/frontend/tflite.py ## @@ -774,6 +775,21 @@ def convert_square(self, op): return out +def

[GitHub] [incubator-tvm] liangfu commented on a change in pull request #5417: [RUNTIME][uTVM] AutoTVM + uTVM for Cortex-M7

liangfu commented on a change in pull request #5417: URL: https://github.com/apache/incubator-tvm/pull/5417#discussion_r417774863 ## File path: python/tvm/micro/base.py ## @@ -133,44 +152,91 @@ def __exit__(self, exc_type, exc_value, exc_traceback): self._exit()

[GitHub] [incubator-tvm-vta] remotego commented on pull request #7: [pynq_driver] fix device early return

remotego commented on pull request #7: URL: https://github.com/apache/incubator-tvm-vta/pull/7#issuecomment-621724098 @huajsj Thank you for the reply. Let me explain more on this issue. The original name of 0x18 (24) register of Compute Module is

[GitHub] [incubator-tvm-vta] remotego edited a comment on pull request #7: [pynq_driver] fix device early return

remotego edited a comment on pull request #7: URL: https://github.com/apache/incubator-tvm-vta/pull/7#issuecomment-621724098 @huajsj Thank you for the reply. Let me explain more on this issue. The original name of 0x18 (24) register of Compute Module is

[GitHub] [incubator-tvm] siju-samuel commented on a change in pull request #5474: [Frontend][TFLite] ADD_N operator

siju-samuel commented on a change in pull request #5474: URL: https://github.com/apache/incubator-tvm/pull/5474#discussion_r417851626 ## File path: python/tvm/relay/frontend/tflite.py ## @@ -863,6 +845,21 @@ def convert_add(self, op): return

[GitHub] [incubator-tvm] mbaret commented on a change in pull request #5474: [Frontend][TFLite] ADD_N operator

mbaret commented on a change in pull request #5474: URL: https://github.com/apache/incubator-tvm/pull/5474#discussion_r417873031 ## File path: python/tvm/relay/frontend/tflite.py ## @@ -863,6 +845,21 @@ def convert_add(self, op): return

[GitHub] [incubator-tvm] maheshambule commented on a change in pull request #5474: [Frontend][TFLite] ADD_N operator

maheshambule commented on a change in pull request #5474: URL: https://github.com/apache/incubator-tvm/pull/5474#discussion_r417808625 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1896,6 +1896,41 @@ def test_forward_mediapipe_hand_landmark():

[GitHub] [incubator-tvm] FrozenGene commented on a change in pull request #5461: [MXNET]broadcast and logical op support

FrozenGene commented on a change in pull request #5461: URL: https://github.com/apache/incubator-tvm/pull/5461#discussion_r417832135 ## File path: python/tvm/relay/frontend/mxnet.py ## @@ -1712,6 +1712,33 @@ def _get_bias_requantize_scale(_inputs, _data_scale, _kernel_scale):

[GitHub] [incubator-tvm] wsl-inspur opened a new pull request #5485: [TOPI][Winograd] Optimization of Conv2d Winograd algorithm on Tensor …

wsl-inspur opened a new pull request #5485: URL: https://github.com/apache/incubator-tvm/pull/5485 - Optimization of Conv2d Winograd algorithm on Tensor Core for NHWC layout. - Winograd with tensor core outperforms original winograd algorithm for all the batchsizes. - However,

[GitHub] [incubator-tvm] FrozenGene commented on pull request #5475: [FRONTEND][TFLITE]Logical not op support

FrozenGene commented on pull request #5475: URL: https://github.com/apache/incubator-tvm/pull/5475#issuecomment-621682464 Thanks @siju-samuel This is an automated message from the Apache Git Service. To respond to the

[incubator-tvm] branch master updated (90b08f5 -> 095f565)

This is an automated email from the ASF dual-hosted git repository. zhaowu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from 90b08f5 [intrin] a few more math functions (#5468) add 095f565 [FRONTEND][TFLITE]Logical not op

[GitHub] [incubator-tvm] FrozenGene commented on a change in pull request #5485: [TOPI][Winograd] Optimization of Conv2d Winograd algorithm on Tensor …

FrozenGene commented on a change in pull request #5485: URL: https://github.com/apache/incubator-tvm/pull/5485#discussion_r417839497 ## File path: topi/python/topi/cuda/conv2d_nhwc_winograd.py ## @@ -0,0 +1,639 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [incubator-tvm] wsl-inspur commented on a change in pull request #5485: [TOPI][Winograd] Optimization of Conv2d Winograd algorithm on Tensor …

wsl-inspur commented on a change in pull request #5485: URL: https://github.com/apache/incubator-tvm/pull/5485#discussion_r417902315 ## File path: topi/python/topi/cuda/conv2d_nhwc_winograd.py ## @@ -0,0 +1,639 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [incubator-tvm] FrozenGene commented on pull request #5485: [TOPI][Winograd] Optimization of Conv2d Winograd algorithm on Tensor …

FrozenGene commented on pull request #5485: URL: https://github.com/apache/incubator-tvm/pull/5485#issuecomment-621791710 For performance, have you tried some other layouts? I have some exp on CPU. The more suitable layout on CPU of NHWC input is: ``` input_tile: alpha, alpha,

[GitHub] [incubator-tvm] FrozenGene commented on a change in pull request #5485: [TOPI][Winograd] Optimization of Conv2d Winograd algorithm on Tensor …

FrozenGene commented on a change in pull request #5485: URL: https://github.com/apache/incubator-tvm/pull/5485#discussion_r417960925 ## File path: topi/python/topi/cuda/conv2d_nhwc_winograd.py ## @@ -0,0 +1,639 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [incubator-tvm] FrozenGene edited a comment on pull request #5485: [TOPI][Winograd] Optimization of Conv2d Winograd algorithm on Tensor …

FrozenGene edited a comment on pull request #5485: URL: https://github.com/apache/incubator-tvm/pull/5485#issuecomment-621791710 For performance, have you tried some other layouts on GPU? I have some exp on CPU. The more suitable layout on CPU of NHWC input is: ``` input_tile: