onnxruntime
Drop QDQ around more nodes
#21376
Merged

Drop QDQ around more nodes #21376

skottmckay merged 45 commits into microsoft:main from mcollinswisc:qdq_optim
mcollinswisc
mcollinswisc Remove QDQ nodes around Flatten
7d38ea47
mcollinswisc Add more operators for which QDQ can be removed
b3b506b7
mcollinswisc Keep QDQ nodes w/ nonpositive scale around MaxPool
2160e44a
mcollinswisc Unit test on QDQ w/ nonpositive scale around MaxPool
61c5d844
mcollinswisc Merge branch 'main' into qdq_optim
e3176e63
mcollinswisc Merge branch 'qdq_optim_nonpositive_scale' into qdq_optim
e84b3e6b
mcollinswisc Unit test on removing QDQ around Expand
ce1ee8aa
mcollinswisc Add selector to remove QDQ nodes around Min, Max, and Abs
73cc2122
mcollinswisc Change formatting according to clangformat
3b94f983
mcollinswisc Unit test that QDQ nodes are remove around Tile
daf808eb
mcollinswisc Merge branch 'qdq_optim_nonpositive_scale' into qdq_optim
641747f0
mcollinswisc Merge branch 'main' into qdq_optim
7e5db770
mcollinswisc Merge branch 'main' into qdq_optim_nonpositive_scale
58e525ca
mcollinswisc Switch to std::fileystem::path in IsQOrDQScalePositiveConstantScalar
68def1e7
mcollinswisc Merge branch 'qdq_optim_nonpositive_scale' into qdq_optim
342034d6
mcollinswisc Remove SpaceToDepth and SpaceToDepth from Drop QDQ optimization
cf10b50c
mcollinswisc Fix grammar in comment
939f2406
mcollinswisc Unit test that QDQ is dropped around Slice
452bdd12
mcollinswisc Unit test on removing QDQ around GatherElements
a67d3fb9
mcollinswisc Apply lintrunner
c7a50da3
mcollinswisc Drop QDQ around ReduceMin & ReduceMax, not Min & Max
82fdcc47
mcollinswisc Disallow 16bit for ReduceMin & ReduceMax
9b3bf090
mcollinswisc Unit test on dropping QDQ from around ReduceMin/Max
872f983b
mcollinswisc Fix comment in ReduceExtremumDropQDQ test case
5bf7d846
mcollinswisc Remove selector to drop QDQ around Abs
28824d68
mcollinswisc Merge branch 'main' into qdq_optim
83d85f3f
mcollinswisc Reformatting from lintrunner
71253a8a
mcollinswisc
jywu-msft jywu-msft requested a review from adrianlizarraga adrianlizarraga 1 year ago
skottmckay
skottmckay
skottmckay
azure-pipelines
azure-pipelines
azure-pipelines
skottmckay
skottmckay commented on 2024-07-25
mcollinswisc Fix comment grammar according to review suggestion
191dc150
mcollinswisc Register drop_action_no_int16_nor_nonpositive_scale in minimal build
023bbafa
mcollinswisc Continue line to keep it under 120 chars
c287525f
mcollinswisc Change name "no nonpositive scale" to "and positive scale"
e34258d6
mcollinswisc Merge branch 'main' into qdq_optim_nonpositive_scale
a880a21d
mcollinswisc More reformatting to keep lines under 120 chars
b0753e43
mcollinswisc Merge branch 'qdq_optim_nonpositive_scale' into qdq_optim
102e9526
skottmckay
skottmckay
skottmckay
azure-pipelines
azure-pipelines
azure-pipelines
mcollinswisc Merge branch 'main' into qdq_optim
8144f660
mcollinswisc Undo spacing change in Gemm test
610a6a93
mcollinswisc Delete drop_action_no_nonpositive_scale_name
d63854b7
mcollinswisc Alphabetize operator names included in base selector
848f7014
mcollinswisc Alphabetize operators in no 16-bit & positive selector
b1de950e
mcollinswisc [Merge branch 'main' into qdq_optim
c193e0eb
mcollinswisc Move some comments to line before
fb91e416
mcollinswisc Merge branch 'main' into qdq_optim
49edace3
mcollinswisc mcollinswisc marked this pull request as ready for review 1 year ago
mcollinswisc mcollinswisc requested a review from skottmckay skottmckay 1 year ago
skottmckay
skottmckay
skottmckay
azure-pipelines
azure-pipelines
azure-pipelines
mcollinswisc Change spacing around comments according to clang-format/lintrunner
85010fe2
skottmckay
skottmckay
skottmckay
azure-pipelines
azure-pipelines
azure-pipelines
mcollinswisc
mcollinswisc Merge branch 'main' into qdq_optim
b7fd7b7d
mcollinswisc Set /bigobj for qdq_transformer_test.cc
e5638e5e
skottmckay
skottmckay
skottmckay
azure-pipelines
azure-pipelines
azure-pipelines
skottmckay
azure-pipelines
skottmckay
skottmckay approved these changes on 2024-08-27
skottmckay skottmckay merged 5d54dc14 into main 1 year ago

Login to write a write a comment.

Login via GitHub

Assignees
No one assigned
Labels
Milestone