Boost C++ Libraries: Ticket #6359: Intel Composer XE 2011 (12.0) fails to build Boost.Regex with ICU support

attachment set

Wed, 04 Jan 2012 23:23:49 GMT

attachment → icubld.diff

Patch to have intel-win toolset use the correct ICU library names

John Maddock — Thu, 05 Jan 2012 13:32:48 GMT

Confirmed.

Reduced test case is just:

void test(...);
void test(const int*&);
int main()
{
   int * p = 0;
   test(p);
}

Reported to Intel as support issue #656654.

A workaround is:

Index: regex_format.hpp
===================================================================
--- regex_format.hpp	(revision 76120)
+++ regex_format.hpp	(working copy)
@@ -180,8 +180,14 @@
    }
    inline int toi(ForwardIter& i, ForwardIter j, int base)
    {
+#if defined(_MSC_VER) && defined(__INTEL_COMPILER) && (__INTEL_COMPILER == 9999)
+      // Workaround for Intel support issue #656654.
+      // See also https://svn.boost.org/trac/boost/ticket/6359
+      return toi(i, j, base, mpl::false_());
+#else
       typedef typename boost::is_convertible<ForwardIter, const char_type*&>::type tag_type;
       return toi(i, j, base, tag_type());
+#endif
    }
    const traits&    m_traits;       // the traits class for localised formatting operations

HTH, John.

status changed; resolution set

John Maddock — Thu, 05 Jan 2012 13:33:50 GMT

status new → closed
resolution → fixed

(In [76319]) Add workaround for Intel-12.1 on Windows. Fixes #6359.

John Maddock — Thu, 05 Jan 2012 13:35:49 GMT

PS Forgot to add that your patch will break ICU build on Intel-Linux. I haven't been able to get a version to work that selects "intel-win" as the toolset :-(

Thu, 05 Jan 2012 17:02:02 GMT

That's OK...the patch was designed to facilitate demonstrating the issue, and not actually intended to be used in production. (We're actually using Boost.CMake to build our copies of Boost libraries, so we have other ways to get the right library names linked in.)

However, I had to make a tweak to your workaround. The ifdef had to be:

#if defined(_MSC_VER) && defined(__INTEL_COMPILER) && ((__INTEL_COMPILER == 9999) || (__INTEL_COMPILER == 1210))

since Composer XE 2011 Update 8 has gotten the correct value of __INTEL_COMPILER.

John Maddock — Thu, 05 Jan 2012 17:31:37 GMT

(In [76324]) Fix pp-logic. Refs #6359.

John Maddock — Thu, 05 Jan 2012 17:32:45 GMT

Dang.... your updated patch has been applied, also found a way to fix Boost.Build to look for the correct ICU libs...

John.