diff options
author | Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> | 2023-02-17 13:48:45 +0100 |
---|---|---|
committer | Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> | 2023-02-17 13:48:45 +0100 |
commit | 593c8b73fb15683fdd1a9d85fe7e7e99540de3c8 (patch) | |
tree | 6cefce0a858e525001822e3a8fcc26e4ab03b3e2 | |
parent | ae2c1d0a9dc33f4c309a589d9f68fcd31206500e (diff) | |
download | gcc-593c8b73fb15683fdd1a9d85fe7e7e99540de3c8.zip gcc-593c8b73fb15683fdd1a9d85fe7e7e99540de3c8.tar.gz gcc-593c8b73fb15683fdd1a9d85fe7e7e99540de3c8.tar.bz2 |
fixincludes: Bypass solaris_math_12 on newer Solaris 11.4
Solaris 11 <math.h> long had this snippet
which badly broke libstdc++. This has long been undone using
fixincludes in
[fixincludes, v3] Don't define libstdc++-internal macros in Solaris 10+ <math.h>
https://gcc.gnu.org/ml/gcc-patches/2016-11/msg00330.html
However, the issue came up again recently when that code broke the LLVM
build, too, which unfortunately doesn't know about GCC's include-fixed
directory. The issue was reinvestigated and it turned out that the
workaround/hack is only needed for specific old versions of the Sun/Oracle
Studio compilers. So <math.h> now looks like
/* Accommodate historical C++11 -std=c++03 behavior of Studio 12.4 and 12.5 */
((__SUNPRO_CC == 0x5130) || (__SUNPRO_CC == 0x5140) || \
defined(__MATH_PREEMPTS_GLIBCXX_C99_MATH))
If this change is in place, there's no longer a need for the fixincludes
fix, so this patch bypasses it as appropriate.
Tested on Solaris 11.3 (without the fixed header) and recent 11.4 (with
the fixed header).
2022-11-01 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
fixincludes:
* inclhack.def (solaris_math_12): Add bypass.
* fixincl.x: Regenerate.
-rw-r--r-- | fixincludes/fixincl.x | 15 | ||||
-rw-r--r-- | fixincludes/inclhack.def | 1 |
2 files changed, 12 insertions, 4 deletions
diff --git a/fixincludes/fixincl.x b/fixincludes/fixincl.x index acb906e..416d2c2 100644 --- a/fixincludes/fixincl.x +++ b/fixincludes/fixincl.x @@ -2,11 +2,11 @@ * * DO NOT EDIT THIS FILE (fixincl.x) * - * It has been AutoGen-ed January 18, 2023 at 09:06:47 PM by AutoGen 5.18.7 + * It has been AutoGen-ed January 22, 2023 at 09:03:29 PM by AutoGen 5.18.12 * From the definitions inclhack.def * and the template file fixincl */ -/* DO NOT SVN-MERGE THIS FILE, EITHER Wed Jan 18 21:06:47 GMT 2023 +/* DO NOT SVN-MERGE THIS FILE, EITHER Sun Jan 22 21:03:29 CET 2023 * * You must regenerate it. Use the ./genfixes script. * @@ -8549,8 +8549,15 @@ tSCC* apzSolaris_Math_12Machs[] = { tSCC zSolaris_Math_12Select0[] = "#undef.*_GLIBCXX_USE_C99_MATH"; -#define SOLARIS_MATH_12_TEST_CT 1 +/* + * content bypass pattern - skip fix if pattern found + */ +tSCC zSolaris_Math_12Bypass0[] = + "__MATH_PREEMPTS_GLIBCXX_C99_MATH"; + +#define SOLARIS_MATH_12_TEST_CT 2 static tTestDesc aSolaris_Math_12Tests[] = { + { TT_NEGREP, zSolaris_Math_12Bypass0, (regex_t*)NULL }, { TT_EGREP, zSolaris_Math_12Select0, (regex_t*)NULL }, }; /* @@ -11162,7 +11169,7 @@ static const char* apzX11_SprintfPatch[] = { * * List of all fixes */ -#define REGEX_COUNT 312 +#define REGEX_COUNT 313 #define MACH_LIST_SIZE_LIMIT 187 #define FIX_COUNT 274 diff --git a/fixincludes/inclhack.def b/fixincludes/inclhack.def index b02dd8e..45e0cbc 100644 --- a/fixincludes/inclhack.def +++ b/fixincludes/inclhack.def @@ -4306,6 +4306,7 @@ fix = { files = math.h; mach = '*-*-solaris2*'; select = '#undef.*_GLIBCXX_USE_C99_MATH'; + bypass = '__MATH_PREEMPTS_GLIBCXX_C99_MATH'; sed = "/#undef[ \t]*_GLIBCXX_USE_C99_MATH/d"; test_text = << _EOText_ #if __cplusplus >= 201103L |