boostorg/fiber

Problem building with latest MSVC

mclow opened this issue · 29 comments

mclow commented

Tom Kent wrote on the mailing list (27-Oct):

I tried building on windows/visual studio from the nightly snapshot. There
are errors for msvc-14.0, 14.1, and 14.2. See example below.

Tom

compile-c-c++
D:\RB\bin.v2\boost\bin.v2\libs\fiber\build\msvc-14.1\release\address-model-64\threading-multi\algo\algorithm.obj
algorithm.cpp
.\boost/fiber/algo/algorithm.hpp(33): warning C4251:
'boost::fibers::algo::algorithm::use_count_': struct 'std::atomic' needs to have dll-interface to be used by clients of class
'boost::fibers::algo::algorithm'
C:\Program Files (x86)\Microsoft Visual
Studio\2017\BuildTools\VC\Tools\MSVC\14.16.27023\include\xxatomic(162):
note: see declaration of 'std::atomic'
.\boost/fiber/waker.hpp(58): warning C4251:
'boost::fibers::waker_with_hook::waker_queue_hook_': class
'boost::intrusive::slist_member_hook<>' needs to have dll-interface to be
used by clients of class 'boost::fibers::waker_with_hook'
.\boost/fiber/waker.hpp(18): note: see declaration of
'boost::intrusive::slist_member_hook<>'
.\boost/fiber/waker.hpp(72): warning C4251:
'boost::intrusive::slist_impl<boost::intrusive::mhtraits<Parent,MemberHook,pointer-to-member(0x10)>,boost::intrusive::slist_defaults::size_type,4,boost::intrusive::slist_defaults::header_holder_type>::data_':
struct
'boost::intrusive::slist_impl<boost::intrusive::mhtraits<Parent,MemberHook,pointer-to-member(0x10)>,boost::intrusive::slist_defaults::size_type,4,boost::intrusive::slist_defaults::header_holder_type>::data_t'
needs to have dll-interface to be used by clients of class
'boost::intrusive::slist_impl<boost::intrusive::mhtraits<Parent,MemberHook,pointer-to-member(0x10)>,boost::intrusive::slist_defaults::size_type,4,boost::intrusive::slist_defaults::header_holder_type>'
with
[
Parent=boost::fibers::waker_with_hook,
MemberHook=boost::fibers::detail::waker_queue_hook
]
.\boost/intrusive/slist.hpp(236): note: see declaration of
'boost::intrusive::slist_impl<boost::intrusive::mhtraits<Parent,MemberHook,pointer-to-member(0x10)>,boost::intrusive::slist_defaults::size_type,4,boost::intrusive::slist_defaults::header_holder_type>::data_t'
with
[
Parent=boost::fibers::waker_with_hook,
MemberHook=boost::fibers::detail::waker_queue_hook
]
.\boost/intrusive/slist.hpp(2113): error C2039: 'swap_trailing_nodes': is
not a member of 'boost::intrusive::circular_slist_algorithms'
with
[

NodeTraits=boost::intrusive::slist_node_traitsboost::intrusive::hook_defaults::void_pointer
]
.\boost/intrusive/circular_slist_algorithms.hpp(396): note: see declaration
of 'boost::intrusive::circular_slist_algorithms'
with
[

NodeTraits=boost::intrusive::slist_node_traitsboost::intrusive::hook_defaults::void_pointer
]
.\boost/intrusive/slist.hpp(2113): note: while compiling class template
member function 'void
boost::intrusive::slist_impl<boost::intrusive::mhtraits<Parent,MemberHook,pointer-to-member(0x10)>,boost::intrusive::slist_defaults::size_type,4,boost::intrusive::slist_defaults::header_holder_type>::priv_swap_lists(boost::intrusive::slist_node
*,boost::intrusive::slist_node
*,boost::move_detail::bool_)'
with
[
Parent=boost::fibers::waker_with_hook,
MemberHook=boost::fibers::detail::waker_queue_hook,
VoidPointer=boost::intrusive::hook_defaults::void_pointer
]
.\boost/intrusive/slist.hpp(2182): note: see reference to class template
instantiation
'boost::intrusive::slist_impl<boost::intrusive::mhtraits<Parent,MemberHook,pointer-to-member(0x10)>,boost::intrusive::slist_defaults::size_type,4,boost::intrusive::slist_defaults::header_holder_type>'
being compiled
with
[
Parent=boost::fibers::waker_with_hook,
MemberHook=boost::fibers::detail::waker_queue_hook
]
.\boost/fiber/waker.hpp(72): note: see reference to class template
instantiation
'boost::intrusive::slist<boost::fibers::waker_with_hook,boost::intrusive::member_hookboost::fibers::waker_with_hook,boost::fibers::detail::waker_queue_hook,pointer-to-member(0x10),boost::intrusive::constant_time_size,boost::intrusive::cache_last>'
being compiled
.\boost/intrusive/slist.hpp(2113): error C3861: 'swap_trailing_nodes':
identifier not found

call
"D:\RB\bin.v2\boost\bin.v2\standalone\msvc\msvc-14.1\address-model-64\architecture-x86\msvc-setup.bat"
nul
cl /Zm800 -nologo
@"D:\RB\bin.v2\boost\bin.v2\libs\fiber\build\msvc-14.1\release\address-model-64\threading-multi\algo\algorithm.obj.rsp"

...failed compile-c-c++
D:\RB\bin.v2\boost\bin.v2\libs\fiber\build\msvc-14.1\release\address-model-64\threading-multi\algo\algorithm.obj...

olk commented

\boost/intrusive/slist.hpp(2113): error C3861: 'swap_trailing_nodes':
identifier not found

seams to me a problem of boost.intrusive - builds successfully with gcc-10 and clang-10

mclow commented

Is this solved? or just not a Boost.Fiber problem? If the latter, how about we transfer the issue to Boost.Intrusive?

olk commented

As I wrote at the dev mailing list I believe it is an issue of boost.intrusive (or MSVC):

.\boost/intrusive/slist.hpp(2113): error C3861: 'swap_trailing_nodes':
identifier not found

swap_trailing_nodes is defined in linear_slist_algorithms<>.
It seams that in slist.hpp static const bool linear = 0 != (BoolFlags & slist_bool_flags::linear_pos) evaluates to false using MSVC (it evaluates to true for gcc-10/clang-10).
If static bool linear is evaluated to false typedef node_algorithms is of type circular_slist_algorithms<> (that doesn't provide swap_trailing_nodes) instead of linear_slist_algorithms<>.

olk commented

I don't know how to transfer issues between projects - sorry

igaztanaga commented

Thank for the ping. Looks like a MSVC instantiation bug, I'll have some time to work on this in a few hours to find a workaround.

igaztanaga commented

I am missing something because I can't reproduce the bug with msvc-14.1 and 14.2. Command prompt/cl.exe versions are:

msvc-14.1
Visual Studio 2017 Developer Command Prompt v15.9.28
cl.exe -> versión 19.16.27043

msvc-14.2
Visual Studio 2019 Developer Command Prompt v16.7.7
cl.exe -> versión 19.27.29112

b2 toolset=msvc-14.2 variant=release address-model=64 threading=multi --with-fiber stage

Performing configuration checks

- default address-model    : 32-bit
- default architecture     : x86
- C++11 mutex              : yes
- has BCrypt API           : yes
- has stat::st_mtim        : no
- has stat::st_mtimensec   : no
- has stat::st_mtimespec   : no
- has stat::st_birthtim    : no
- has stat::st_birthtimensec : no
- has stat::st_birthtimespec : no
- has statx                : no
- has statx syscall        : no
- cxx11_auto_declarations  : yes
- cxx11_constexpr          : yes
- cxx11_defaulted_functions : yes
- cxx11_final              : yes
- cxx11_hdr_mutex          : yes
- cxx11_hdr_tuple          : yes
- cxx11_lambdas            : yes
- cxx11_noexcept           : yes
- cxx11_nullptr            : yes
- cxx11_rvalue_references  : yes
- cxx11_template_aliases   : yes
- cxx11_thread_local       : yes
- cxx11_variadic_templates : yes

Component configuration:

- atomic                   : not building
- chrono                   : not building
- container                : not building
- context                  : not building
- contract                 : not building
- coroutine                : not building
- date_time                : not building
- exception                : not building
- fiber                    : building
- filesystem               : not building
- graph                    : not building
- graph_parallel           : not building
- headers                  : not building
- iostreams                : not building
- json                     : not building
- locale                   : not building
- log                      : not building
- math                     : not building
- mpi                      : not building
- nowide                   : not building
- program_options          : not building
- python                   : not building
- random                   : not building
- regex                    : not building
- serialization            : not building
- stacktrace               : not building
- system                   : not building
- test                     : not building
- thread                   : not building
- timer                    : not building
- type_erasure             : not building
- wave                     : not building

...patience...
...found 2174 targets...
...updating 93 targets...
msvc.compile.asm bin.v2\libs\context\build\msvc-14.2\release\address-model-64\link-static\threading-multi\asm\make_x86_64_ms_pe_masm.obj
Assembling: libs\context\src\asm\make_x86_64_ms_pe_masm.asm
boost-install.generate-cmake-config-version- bin.v2\tools\boost_install\BoostConfigVersion.cmake
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\Boost-1.75.0\BoostConfigVersion.cmake
bin.v2\tools\boost_install\BoostConfigVersion.cmake
1 archivo(s) copiado(s).
boost-install.generate-cmake-config- bin.v2\libs\context\build\stage\boost_context-config.cmake
boost-install.generate-cmake-config-version- bin.v2\libs\context\build\stage\boost_context-config-version.cmake
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_context-1.75.0\boost_context-config.cmake
bin.v2\libs\context\build\stage\boost_context-config.cmake
1 archivo(s) copiado(s).
boost-install.generate-cmake-config- bin.v2\libs\headers\build\stage\boost_headers-config.cmake
boost-install.generate-cmake-config- bin.v2\libs\filesystem\build\stage\boost_filesystem-config.cmake
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_context-1.75.0\boost_context-config-version.cmake
bin.v2\libs\context\build\stage\boost_context-config-version.cmake
1 archivo(s) copiado(s).
boost-install.generate-cmake-config-version- bin.v2\libs\filesystem\build\stage\boost_filesystem-config-version.cmake
boost-install.generate-cmake-config- bin.v2\libs\fiber\build\stage\boost_fiber-config.cmake
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_headers-1.75.0\boost_headers-config.cmake
bin.v2\libs\headers\build\stage\boost_headers-config.cmake
1 archivo(s) copiado(s).
boost-install.generate-cmake-config-version- bin.v2\libs\fiber\build\stage\boost_fiber-config-version.cmake
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_filesystem-1.75.0\boost_filesystem-config.cmake
bin.v2\libs\filesystem\build\stage\boost_filesystem-config.cmake
1 archivo(s) copiado(s).
boost-install.generate-cmake-config- bin.v2\libs\fiber\build\stage\boost_fiber_numa-config.cmake
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_filesystem-1.75.0\boost_filesystem-config-version.cmake
bin.v2\libs\filesystem\build\stage\boost_filesystem-config-version.cmake
1 archivo(s) copiado(s).
boost-install.generate-cmake-config-version- bin.v2\libs\fiber\build\stage\boost_fiber_numa-config-version.cmake
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_fiber-1.75.0\boost_fiber-config-version.cmake
bin.v2\libs\fiber\build\stage\boost_fiber-config-version.cmake
1 archivo(s) copiado(s).
boost-install.generate-cmake-config-version- bin.v2\libs\headers\build\stage\boost_headers-config-version.cmake
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_fiber_numa-1.75.0\boost_fiber_numa-config.cmake
bin.v2\libs\fiber\build\stage\boost_fiber_numa-config.cmake
1 archivo(s) copiado(s).
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_fiber-1.75.0\boost_fiber-config.cmake
bin.v2\libs\fiber\build\stage\boost_fiber-config.cmake
1 archivo(s) copiado(s).
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_fiber_numa-1.75.0\boost_fiber_numa-config-version.cmake
bin.v2\libs\fiber\build\stage\boost_fiber_numa-config-version.cmake
1 archivo(s) copiado(s).
compile-c-c++ bin.v2\libs\context\build\msvc-14.2\release\address-model-64\link-static\threading-multi\dummy.obj
dummy.cpp
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_headers-1.75.0\boost_headers-config-version.cmake
bin.v2\libs\headers\build\stage\boost_headers-config-version.cmake
1 archivo(s) copiado(s).
msvc.compile.asm bin.v2\libs\context\build\msvc-14.2\release\address-model-64\link-static\threading-multi\asm\jump_x86_64_ms_pe_masm.obj
Assembling: libs\context\src\asm\jump_x86_64_ms_pe_masm.asm
msvc.compile.asm bin.v2\libs\context\build\msvc-14.2\release\address-model-64\link-static\threading-multi\asm\ontop_x86_64_ms_pe_masm.obj
Assembling: libs\context\src\asm\ontop_x86_64_ms_pe_masm.asm
compile-c-c++ bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\utf8_codecvt_facet.obj
utf8_codecvt_facet.cpp
compile-c-c++ bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\codecvt_error_category.obj
codecvt_error_category.cpp
compile-c-c++ bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\path_traits.obj
path_traits.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\future.obj
future.cpp
compile-c-c++ bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\exception.obj
exception.cpp
compile-c-c++ bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\portability.obj
portability.cpp
compile-c-c++ bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\unique_path.obj
unique_path.cpp
compile-c-c++ bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\windows_file_codecvt.obj
windows_file_codecvt.cpp
compile-c-c++ bin.v2\libs\context\build\msvc-14.2\release\address-model-64\link-static\threading-multi\windows\stack_traits.obj
stack_traits.cpp
msvc.archive bin.v2\libs\context\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_context-vc142-mt-x64-1_75.lib
boost-install.generate-cmake-variant- bin.v2\libs\context\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_context-variant-vc142-mt-x64-1_75-static.cmake
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\libboost_context-vc142-mt-x64-1_75.lib
bin.v2\libs\context\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_context-vc142-mt-x64-1_75.lib
1 archivo(s) copiado(s).
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_context-1.75.0\libboost_context-variant-vc142-mt-x64-1_75-static.cmake
bin.v2\libs\context\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_context-variant-vc142-mt-x64-1_75-static.cmake
1 archivo(s) copiado(s).
compile-c-c++ bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\path.obj
path.cpp
compile-c-c++ bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\directory.obj
directory.cpp
compile-c-c++ bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\operations.obj
operations.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\condition_variable.obj
condition_variable.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\barrier.obj
barrier.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\algo\algorithm.obj
algorithm.cpp
msvc.archive bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_filesystem-vc142-mt-x64-1_75.lib
boost-install.generate-cmake-variant- bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_filesystem-variant-vc142-mt-x64-1_75-static.cmake
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\libboost_filesystem-vc142-mt-x64-1_75.lib
bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_filesystem-vc142-mt-x64-1_75.lib
1 archivo(s) copiado(s).
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\properties.obj
properties.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\timed_mutex.obj
timed_mutex.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\fiber.obj
fiber.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\waker.obj
waker.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\mutex.obj
mutex.cpp
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_filesystem-1.75.0\libboost_filesystem-variant-vc142-mt-x64-1_75-static.cmake
bin.v2\libs\filesystem\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_filesystem-variant-vc142-mt-x64-1_75-static.cmake
1 archivo(s) copiado(s).
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\recursive_mutex.obj
recursive_mutex.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\recursive_timed_mutex.obj
recursive_timed_mutex.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\algo\round_robin.obj
round_robin.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\scheduler.obj
scheduler.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\context.obj
context.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\algo\shared_work.obj
shared_work.cpp
compile-c-c++ bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\algo\work_stealing.obj
work_stealing.cpp
msvc.archive bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_fiber-vc142-mt-x64-1_75.lib
boost-install.generate-cmake-variant- bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_fiber-variant-vc142-mt-x64-1_75-static.cmake
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\libboost_fiber-vc142-mt-x64-1_75.lib
bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_fiber-vc142-mt-x64-1_75.lib
1 archivo(s) copiado(s).
common.copy C:\Data\Libs\boost_1_75_0_b1_rc1\stage\lib\cmake\boost_fiber-1.75.0\libboost_fiber-variant-vc142-mt-x64-1_75-static.cmake
bin.v2\libs\fiber\build\msvc-14.2\release\address-model-64\link-static\threading-multi\libboost_fiber-variant-vc142-mt-x64-1_75-static.cmake
1 archivo(s) copiado(s).
...updated 93 targets...

pdimov commented

Confirmed; b2 --with-fiber address-model=64 variant=release works for me with both msvc-14.1 and msvc-14.2. With msvc-14.0, I have an error, but it's not the same one.

compile-c-c++ bin.v2\libs\fiber\build\msvc-14.0\release\address-model-64\link-static\threading-multi\waker.obj
waker.cpp
.\boost/fiber/detail/spinlock_ttas_adaptive.hpp(48): error C2039: 'min': is not a member of 'std'
pdimov commented

b2 --with-fiber address-model=64 variant=release link=shared reproduces the errors.

igaztanaga commented

I started fixing the compilation errors as fiber in MSVC requires to instantiate things that in theory are not supported to be instantiated for circular lists, but after "swap_trailing_nodes" error any other unsupported instantiation (including static asserts) are triggered. boost::intrusive::slist supports some member only if the right options are used (this is documented and checked with some static asserts).

So it seems that boost::fiber::wait_queue (new in Boost 1.75), derives publicly from a boost::intrusive list. This provokes C4275 as intrusive classes are templates and not supported by be dllexported. Reviewing similar issues in the net, it seems that exported classes that inherit from a template instantiation that is not explicitly exported then the compiler implicitly applies dllexport to this template instantiation, which provokes the compilation errors (I think). Since wait_queue is marked as BOOST_FIBERS_DECL, MSVC instantiates all functions of the class template boost::intrusive::slist, including the unsupported ones for circular lists. Changing the slist from being a base class to a member seems to avoid the instantiation problem:

class BOOST_FIBERS_DECL wait_queue
{
    detail::waker_slist_t slist_{};
public:
    void suspend_and_wait( detail::spinlock_lock &, context *);
    bool suspend_and_wait_until( detail::spinlock_lock &,
                                 context *,
                                 std::chrono::steady_clock::time_point const&);
    void notify_one();
    void notify_all();
};

instead of

class BOOST_FIBERS_DECL wait_queue : public detail::waker_slist_t
{
    using base = detail::waker_slist_t;
public:
    using base::base;

    void suspend_and_wait( detail::spinlock_lock &, context *);
    bool suspend_and_wait_until( detail::spinlock_lock &,
                                 context *,
                                 std::chrono::steady_clock::time_point const&);
    void notify_one();
    void notify_all();
};

(the implementation or waker.cpp must be changed accordingly, replacing base::xxx() calls with slist_.xxx()).

So it seems that this is a Fiber issue when exporting the wait_queue class, but with an easy solution.

olk commented

Compiling the code with gcc or clang on Linux I don't get the "swap_trailing_nodes" error which lets me conclude that linear list has been chosen in slist. But MSVC on Windows uses the circular list which raises the "swap_trailing_nodes" error.
I don't understand the reason of this difference.

igaztanaga commented

The difference is that according to MSVC documentation any template class used as a base class for an exported object is also exported, which means it is fully instantiated by the compiler. Boost.Intrusive classes don't support instantiation of all members because some member functions have sense only with some options. This is similar to a std::vector, you can't instantiate all members that require copy constructible types for std::vector, but if you put such type as the base class of an dll-exported class in MSVC you'll get a compilation error in those functions. That's why putting the type as a member solves the problem: the compiler does not fully instantiate the type. More information:

General Rules and Limitations

"Because this is common pattern with templates, the compiler changed the semantics of dllexport when it is applied to a class that has one or more base-classes and when one or more of the base classes is a specialization of a class template. In this case, the compiler implicitly applies dllexport to the specializations of class templates."

DLL-export is out of the standard so rules between GCC/Clang and MSVC might differ.

olk commented

I've pushed a fixed but I don't use Windows. Could someone check if the issue is fixed, please?

pdimov commented

DLL-export is out of the standard so rules between GCC/Clang and MSVC might differ.

clang-cl (toolset=clang-win) gives the same error.

pdimov commented

Latest develop fails with a bunch of

libs\fiber\src\scheduler.cpp(40): error C2039: 'empty': is not a member of 'boost::fibers::wait_queue'
C:\boost-git\develop\boost/fiber/waker.hpp(71): note: see declaration of 'boost::fibers::wait_queue'
libs\fiber\src\scheduler.cpp(231): error C2039: 'empty': is not a member of 'boost::fibers::wait_queue'
C:\boost-git\develop\boost/fiber/waker.hpp(71): note: see declaration of 'boost::fibers::wait_queue'
olk commented

next try

pdimov commented

Compiles now, with a number of 4251 ("needs a DLL interface") warnings.

pdimov commented

The tests pass, but the Jamfile hardcodes <link>static. I'm running them now with link=shared and this removed.

pdimov commented

link=shared tests fail with

msvc.link ..\..\bin.v2\libs\fiber\test\test_condition_mt_dispatch_native.test\msvc-14.2\release\context-impl-winfib\threadapi-win32\threading-multi\test_condition_mt_dispatch_native.exe
   Creating library ..\..\bin.v2\libs\fiber\test\test_condition_mt_dispatch_native.test\msvc-14.2\release\context-impl-winfib\threadapi-win32\threading-multi\test_condition_mt_dispatch_native.lib and object ..\..\bin.v2\libs\fiber\test\test_condition_mt_dispatch_native.test\msvc-14.2\release\context-impl-winfib\threadapi-win32\threading-multi\test_condition_mt_dispatch_native.exp
MSVCRT.lib(exe_main.obj) : error LNK2019: unresolved external symbol main referenced in function "int __cdecl __scrt_common_main_seh(void)" (?__scrt_common_main_seh@@YAHXZ)
..\..\bin.v2\libs\fiber\test\test_condition_mt_dispatch_native.test\msvc-14.2\release\context-impl-winfib\threadapi-win32\threading-multi\test_condition_mt_dispatch_native.exe : fatal error LNK1120: 1 unresolved externals

which seems unrelated.

olk commented

Maybe msvcrt.lib is missing...

mclow commented

@olk please merge this to master. thanks.

olk commented

done

pdimov commented

No, a main function is missing. My guess is that when using the DLL version of Boost.Test you need to supply a main function somehow, as it can't come from a DLL (the static version of Boost.Test probably supplies main.)

igaztanaga commented

Can we transfer this back to fiber?

pdimov commented

Done.

pdimov commented

You're not going to fix the tests to work with link=shared?

olk commented

I've no idea because I don't develop/use MS Windows - but you are welcome to provide a fix.

pdimov commented

It doesn't work under Linux either.