openvino icon indicating copy to clipboard operation
openvino copied to clipboard

[CPU][SNIPPETS] Update the skip markup for Mamul / FC operations

Open EgorDuplensky opened this issue 3 years ago • 1 comments

taking into account recent changes:

  • Matmul also supports bias fusion
  • There is no reshape after FC node since corresponding transformation was removed

Also bias fusing should be checked first, since in general bias can be considered as a simple fusing as well.

Main loop also refactored a bit.

EgorDuplensky avatar Aug 04 '22 16:08 EgorDuplensky

Could you please add tests to check that biases aren't fused into Subgraph? We have almost the same test for Convolution

a-sidorova avatar Aug 10 '22 10:08 a-sidorova

@EgorDuplensky @a-sidorova @IvanNovoselov This one seems to be waiting for a long time. What are the next steps?

dmitry-gorokhov avatar Sep 06 '22 06:09 dmitry-gorokhov

@dmitry-gorokhov This change was postponed as not really urgent one. TODO Add unit test TODO Run benchmarking

EgorDuplensky avatar Sep 07 '22 10:09 EgorDuplensky

This PR will be closed in 2 weeks in case of no activity.

akladiev avatar May 11 '23 16:05 akladiev

This PR was closed because it has been stalled for 2 week with no activity.

akladiev avatar May 23 '23 14:05 akladiev