Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suppress multiple linker warnings for ngraph on Win32 #2879

Merged

Conversation

ilya-lavrenov
Copy link
Contributor

@ilya-lavrenov ilya-lavrenov commented Oct 28, 2020

Like:

2020-10-28T12:19:21.7751543Z LINK : warning LNK4286: symbol '?get_shape@?$Output@VNode@ngraph@@@ngraph@@QEBAAEBVShape@2@XZ (public: class ngraph::Shape const & __cdecl ngraph::Output<class ngraph::Node>::get_shape(void)const )' defined in 'node_output.cpp.obj' is imported by 'ngraph_builders.lib(reduce_ops.cpp.obj)'
2020-10-28T12:19:21.7753487Z LINK : warning LNK4217: symbol '?get_shape@?$Output@VNode@ngraph@@@ngraph@@QEBAAEBVShape@2@XZ (public: class ngraph::Shape const & __cdecl ngraph::Output<class ngraph::Node>::get_shape(void)const )' defined in 'node_output.cpp.obj' is imported by 'ngraph_builders.lib(autobroadcast.cpp.obj)' in function '"class std::shared_ptr<class ngraph::Node> __cdecl ngraph::builder::broadcast_value_pdpd_style(class ngraph::Output<class ngraph::Node> const &,class ngraph::Shape const &,__int64)" (?broadcast_value_pdpd_style@builder@ngraph@@YA?AV?$shared_ptr@VNode@ngraph@@@std@@AEBV?$Output@VNode@ngraph@@@2@AEBVShape@2@_J@Z)'
2020-10-28T12:19:21.7755535Z LINK : warning LNK4286: symbol '?get_shape@?$Output@VNode@ngraph@@@ngraph@@QEBAAEBVShape@2@XZ (public: class ngraph::Shape const & __cdecl ngraph::Output<class ngraph::Node>::get_shape(void)const )' defined in 'node_output.cpp.obj' is imported by 'ngraph_builders.lib(reshape.cpp.obj)'
2020-10-28T12:19:21.7756859Z LINK : warning LNK4286: symbol '?get_shape@?$Output@VNode@ngraph@@@ngraph@@QEBAAEBVShape@2@XZ (public: class ngraph::Shape const & __cdecl ngraph::Output<class ngraph::Node>::get_shape(void)const )' defined in 'node_output.cpp.obj' is imported by 'ngraph_builders.lib(split.cpp.obj)'
2020-10-28T12:19:21.7758370Z LINK : warning LNK4286: symbol '?get_shape@?$Output@VNode@ngraph@@@ngraph@@QEBAAEBVShape@2@XZ (public: class ngraph::Shape const & __cdecl ngraph::Output<class ngraph::Node>::get_shape(void)const )' defined in 'node_output.cpp.obj' is imported by 'ngraph_builders.lib(matmul_factory.cpp.obj)'
2020-10-28T12:19:21.7759655Z LINK : warning LNK4286: symbol '??1?$Output@VNode@ngraph@@@ngraph@@QEAA@XZ (public: __cdecl ngraph::Output<class ngraph::Node>::~Output<class ngraph::Node>(void))' defined in 'attribute_visitor.cpp.obj' is imported by 'ngraph_builders.lib(matmul_factory.cpp.obj)'

@ilya-lavrenov ilya-lavrenov added category: Core OpenVINO Core (aka ngraph) category: build OpenVINO cmake script / infra platform: win32 OpenVINO on Windows platform labels Oct 28, 2020
@ilya-lavrenov ilya-lavrenov added this to the 2021.2 milestone Oct 28, 2020
@ilya-lavrenov ilya-lavrenov requested review from ilyachur and a team October 28, 2020 14:56
@ilya-lavrenov ilya-lavrenov marked this pull request as draft December 9, 2020 14:37
@ilyachur ilyachur self-requested a review January 12, 2021 11:13
@ilya-lavrenov ilya-lavrenov force-pushed the ngraph-linker-warnings-win32 branch from 284331b to 9bdf02c Compare January 12, 2021 13:13
@ilya-lavrenov ilya-lavrenov force-pushed the ngraph-linker-warnings-win32 branch 3 times, most recently from c8c69dc to f1a2bd3 Compare January 12, 2021 14:54
@ilya-lavrenov ilya-lavrenov marked this pull request as ready for review January 12, 2021 15:20
@ilya-lavrenov ilya-lavrenov changed the title Fixed multiple linker warnings on Win32 Suppress multiple linker warnings on Win32 Jan 12, 2021
@ilya-lavrenov ilya-lavrenov changed the title Suppress multiple linker warnings on Win32 Suppress multiple linker warnings for ngraph on Win32 Jan 12, 2021
@ilyachur ilyachur merged commit 37e051b into openvinotoolkit:master Jan 13, 2021
@ilya-lavrenov ilya-lavrenov deleted the ngraph-linker-warnings-win32 branch January 13, 2021 09:23
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Jan 14, 2021
…t#2879)

* Fixed multiple linker warnings on Win32

* Fixed linkage on Windows

* Fixed compilation for Win32

* Last fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: build OpenVINO cmake script / infra category: Core OpenVINO Core (aka ngraph) platform: win32 OpenVINO on Windows platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants