[Bug tree-optimization/49170] [4.7 regression] Several libstdc++ tests fail to link on Solaris 8/9: cexp missing

2011-07-18 Thread ro at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49170

Rainer Orth ro at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution||FIXED
   Target Milestone|--- |4.7.0

--- Comment #7 from Rainer Orth ro at gcc dot gnu.org 2011-07-18 15:16:07 UTC 
---
Fixed.


[Bug tree-optimization/49170] [4.7 regression] Several libstdc++ tests fail to link on Solaris 8/9: cexp missing

2011-05-27 Thread ro at CeBiTec dot Uni-Bielefeld.DE
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49170

--- Comment #4 from ro at CeBiTec dot Uni-Bielefeld.DE ro at CeBiTec dot 
Uni-Bielefeld.DE 2011-05-27 10:05:28 UTC ---
 --- Comment #3 from William J. Schmidt wschmidt at gcc dot gnu.org 
 2011-05-26 13:24:31 UTC ---
 Rainer, please try:
[...]
 and let me know if it solves the problem.

It does: I've bootstrapped i386-pc-solaris2.8 with the patch and both
the libstdc++ failures went away, as well as several gfortran ones.

Thanks.
Rainer


[Bug tree-optimization/49170] [4.7 regression] Several libstdc++ tests fail to link on Solaris 8/9: cexp missing

2011-05-27 Thread dominiq at lps dot ens.fr
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49170

--- Comment #5 from Dominique d'Humieres dominiq at lps dot ens.fr 2011-05-27 
10:15:37 UTC ---
 +  /* Make sure we have either sincos or cexp.  */
 +  if (!TARGET_HAS_SINCOS  !TARGET_C99_FUNCTIONS)
 +break;
 +

Could you please have a look to pr 31249?


[Bug tree-optimization/49170] [4.7 regression] Several libstdc++ tests fail to link on Solaris 8/9: cexp missing

2011-05-27 Thread wschmidt at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49170

--- Comment #6 from William J. Schmidt wschmidt at gcc dot gnu.org 2011-05-27 
13:30:01 UTC ---
Author: wschmidt
Date: Fri May 27 13:29:57 2011
New Revision: 174331

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=174331
Log:
Index: gcc/ChangeLog
===
--- gcc/ChangeLog(revision 174330)
+++ gcc/ChangeLog(working copy)
@@ -1,3 +1,9 @@
+2011-05-27  Bill Schmidt  wschm...@linux.vnet.ibm.com
+
+PR tree-optimization/49170
+* tree-ssa-math-opts.c (execute_cse_sincos):  Add checks for
+sincos or cexp.
+
 2011-05-27  Richard Guenther  rguent...@suse.de

 PR middle-end/49189
Index: gcc/tree-ssa-math-opts.c
===
--- gcc/tree-ssa-math-opts.c(revision 174330)
+++ gcc/tree-ssa-math-opts.c(working copy)
@@ -1093,6 +1093,10 @@ execute_cse_sincos (void)
 CASE_FLT_FN (BUILT_IN_COS):
 CASE_FLT_FN (BUILT_IN_SIN):
 CASE_FLT_FN (BUILT_IN_CEXPI):
+  /* Make sure we have either sincos or cexp.  */
+  if (!TARGET_HAS_SINCOS  !TARGET_C99_FUNCTIONS)
+break;
+
   arg = gimple_call_arg (stmt, 0);
   if (TREE_CODE (arg) == SSA_NAME)
 cfg_changed |= execute_cse_sincos_1 (arg);

Modified:
trunk/gcc/ChangeLog
trunk/gcc/tree-ssa-math-opts.c


[Bug tree-optimization/49170] [4.7 regression] Several libstdc++ tests fail to link on Solaris 8/9: cexp missing

2011-05-26 Thread wschmidt at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49170

--- Comment #1 from William J. Schmidt wschmidt at gcc dot gnu.org 2011-05-26 
12:12:00 UTC ---
I'll take a look.


[Bug tree-optimization/49170] [4.7 regression] Several libstdc++ tests fail to link on Solaris 8/9: cexp missing

2011-05-26 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49170

--- Comment #2 from Richard Guenther rguenth at gcc dot gnu.org 2011-05-26 
12:13:35 UTC ---
I think the TARGET_HAS_SINCOS || TARGET_C99_FUNCTIONS has to be moved somewhere
else.


[Bug tree-optimization/49170] [4.7 regression] Several libstdc++ tests fail to link on Solaris 8/9: cexp missing

2011-05-26 Thread wschmidt at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49170

--- Comment #3 from William J. Schmidt wschmidt at gcc dot gnu.org 2011-05-26 
13:24:31 UTC ---
Rainer, please try:

Index: tree-ssa-math-opts.c
===
--- tree-ssa-math-opts.c(revision 174277)
+++ tree-ssa-math-opts.c(working copy)
@@ -1093,6 +1093,10 @@ execute_cse_sincos (void)
 CASE_FLT_FN (BUILT_IN_COS):
 CASE_FLT_FN (BUILT_IN_SIN):
 CASE_FLT_FN (BUILT_IN_CEXPI):
+  /* Make sure we have either sincos or cexp.  */
+  if (!TARGET_HAS_SINCOS  !TARGET_C99_FUNCTIONS)
+break;
+
   arg = gimple_call_arg (stmt, 0);
   if (TREE_CODE (arg) == SSA_NAME)
 cfg_changed |= execute_cse_sincos_1 (arg);

and let me know if it solves the problem.

Thanks,
Bill