openvino Conditional Compilation for ov::OpSet factory

Conditional Compilation for ov::OpSet factory

Open vurusovs opened this issue 2 years ago • 3 comments

The changes introduce new OpSet factory inherited from CC factory to enable operations elimination. Old factory (FactoryRegistry) wasn't updated because in this case CC dependency leaks to a lot of targets, so old factory is deprecated and leaved as is for backward compatibility
Add new macroses to CC factory to work with builders via function parameter instead of template parameter - it's a limitation of ov::OpSet::insert() method
Move NodeBuilder from FactoryRegistry to ov::OpSet to remove use of deprecated API

Mar 22 '22 11:03 vurusovs

Do we have examples of usage? E.g. how many MBs do we need to read Resnet-50 or other topologies with compare to current MBs of IR FE?

Mar 28 '22 15:03 ilya-lavrenov

Do we have examples of usage? E.g. how many MBs do we need to read Resnet-50 or other topologies with compare to current MBs of IR FE?

We have conditional_compilation tests, I'll provide results from the branch a bit later

Mar 28 '22 15:03 vurusovs

Could you please check how load time is changed?

@vladislav-volkov Load time isn't changed, please, find time tests results in a ticket

Mar 30 '22 07:03 vurusovs

Current PR is replaced with https://github.com/openvinotoolkit/openvino/pull/13189 in order implementation is changed to simpler one

Sep 27 '22 12:09 vurusovs