Boost C++ Libraries: Ticket #11815: boost::config defines BOOST_NO_CXX11_RVALUE_REFERENCES under icc 15

attachment set

nmusolino@… — Sun, 22 Nov 2015 00:10:46 GMT

attachment → intrusive_ptr_test.cpp

Demonstration of boost::config problem

John Maddock — Sun, 22 Nov 2015 10:00:15 GMT

I don't see this (in develop anyway), some comments:

At Intel's request we changed our configuration to match whatever version of GCC Intel is emulating, so the underlying gcc version is critical.
Your test output above shows BOOST_NO_CXX11_RVALUE_REFERENCES as *not* being set for Intel and GCC alike.
The best way to see what is and isn't being set is to cd into libs/config/test and build and run the program config_info.cpp. I don't see BOOST_NO_CXX11_RVALUE_REFERENCES set in either C++11 or C++ 14 mode.

Best, John.

anonymous — Sun, 22 Nov 2015 13:40:07 GMT

I'm sorry, I made a mistake when examining different versions of boost. BOOST_NO_CXX11_RVALUE_REFERENCES is defined under boost 1.54:

$ icc -I/path/to/boost/1.54.0/include/  -std=c++11 ./intrusive_ptr_test.cpp -o intrusive_ptr_test_icc
$ $ ./intrusive_ptr_test_icc
__cplusplus: 201103
BOOST_VERSION: 105400
std::is_nothrow_move_constructible<S_ptr>: false
BOOST_NO_CXX11_RVALUE_REFERENCES: <defined>
__INTEL_COMPILER: 1500
BOOST_INTEL_CXX_VERSION: 1500

$ g++ -I/path/to/boost/1.54.0/include/  -std=c++11 ./intrusive_ptr_test.cpp -o intrusive_ptr_test_gcc
$ ./intrusive_ptr_test_gcc
__cplusplus: 201103
BOOST_VERSION: 105400
std::is_nothrow_move_constructible<S_ptr>: true
BOOST_NO_CXX11_RVALUE_REFERENCES: <undefined>

I can build the config_info program if it would be helpful.

version changed

anonymous — Sun, 22 Nov 2015 13:40:51 GMT

version Boost 1.56.0 → Boost 1.54.0

nmusolino@… — Sun, 22 Nov 2015 14:28:32 GMT

Using config_info, I do see some BOOST_NO_CXX11_* macros, in particular BOOST_NO_CXX11_NOEXCEPT and BOOST_NO_CXX11_CONSTEXPR, that appear to be defined incorrectly under boost 1.56.0 and icc. The _NOEXCEPT issue causes the intrusive_ptr class to not be nothrow-movable.

Would you prefer that I create a new ticket, or place more information here? My inclination is to create a new ticket, to refer to a different boost version [1.56.0].

nmusolino@… — Sun, 22 Nov 2015 14:28:35 GMT

Would you prefer that I create a new ticket, or place more information here? My inclination is to create a new ticket, to refer to a different boost version [1.56.0].

John Maddock — Sun, 22 Nov 2015 17:46:08 GMT

Post the info here, but I still don't see it, note that for Intel-15 and later the configuration defaults to just

#include <boost/config/compiler/gcc.hpp>

Plus some logic to correctly set BOOST_COMPILER.

So you should see exactly the same configuration as the GCC version being emulated.

What does your config_info.cpp output look like (note - all of it please! ;) )

anonymous — Sun, 22 Nov 2015 19:20:29 GMT

Thanks, by the way, for your responsiveness on this. I appreciate your efforts.

$ tar -xzf boost_1_56_0.tar.gz
$ icc --version
icc (ICC) 15.0.4 20150805
Copyright (C) 1985-2015 Intel Corporation.  All rights reserved.
$ icc -I./boost_1_56_0/ -std=c++11 ./boost_1_56_0/libs/config/test/config_info.cpp -o ./config_info_1_56_0_icc
$ ./config_info_1_56_0_icc > config_info_output_icc15.txt

I noticed that icc was emulating g++ 5.1 (see \_\_VERSION\_\_ in config_info_output_icc51.txt), so I built config_info using g++ 5.1 for comparison.

$ g++ --version
g++ (TPM built 20150527) 5.1.0
Copyright (C) 2015 Free Software Foundation, Inc.
$ g++ -I./boost_1_56_0 -std=c++11 ./boost_1_56_0/libs/config/test/config_info.cpp -o config_info_1_56_0_gcc51
$ ./config_info_1_56_0_gcc51 > config_info_output_gcc51.txt

The following boost config definitions seem incorrect to me, because icc 15 supports "constexpr" and "noexcept". I realize that the definition of BOOST_NOEXCEPT is probably a consequence of BOOST_NO_CXX11_NOEXCEPT being defined. Excerpt from attached file config_info_output_icc15.txt:

    BOOST_COMPILER_CONFIG                   ="boost/config/compiler/intel.hpp"
    [...]
    BOOST_NO_CXX11_CONSTEXPR                 [no value]
    [...]
    BOOST_NO_CXX11_NOEXCEPT                  [no value]
    [...]
    BOOST_CONSTEXPR                          [no value]
    BOOST_CONSTEXPR_OR_CONST                =const
    BOOST_STATIC_CONSTEXPR                  =static const
    BOOST_NOEXCEPT                           [no value]
    [...]

In contrast, these defintions appear correct under g++ 5.1. Excerpt from attached file config_info_output_gcc51.txt:

    BOOST_COMPILER_CONFIG                   ="boost/config/compiler/gcc.hpp"
    [...]
    BOOST_CONSTEXPR                         =constexpr
    BOOST_CONSTEXPR_OR_CONST                =constexpr
    BOOST_STATIC_CONSTEXPR                  =static constexpr
    BOOST_NOEXCEPT                          =noexcept

attachment set

nmusolino@… — Sun, 22 Nov 2015 19:20:52 GMT

attachment → config_info_output_icc15.txt

attachment set

nmusolino@… — Sun, 22 Nov 2015 19:21:06 GMT

attachment → config_info_output_gcc51.txt

nmusolino@… — Sun, 22 Nov 2015 19:28:43 GMT

Just to clarify, all the information in comment 6 pertains to boost 1.56.0. This is probably clear from the shell commands and the output, but I just want to highlight that here.

status changed; resolution set

John Maddock — Mon, 23 Nov 2015 08:48:14 GMT

status new → closed
resolution → worksforme

OK I missed the boost-1.56 part - we didn't get around to configuring for Intel-15 until this autumn - if you can upgrade to 1.59 this should be fixed.