https://gcc.gnu.org/bugzilla/show_bug.cgi?id=29253
Andrew Pinski changed:
What|Removed |Added
Attachment #55235|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=29253
--- Comment #13 from Andrew Pinski ---
Created attachment 55235
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55235=edit
Slightly different patch
This is a slightly different patch but it basically does the same except it
uses
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=29253
Andrew Pinski changed:
What|Removed |Added
Last reconfirmed|2008-02-04 14:27:46 |2021-8-8
--- Comment #12 from Andrew
--- Comment #9 from rguenth at gcc dot gnu dot org 2008-02-05 10:54 ---
Created an attachment (id=15098)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15098action=view)
patch
Try this instead.
--
rguenth at gcc dot gnu dot org changed:
What|Removed
--- Comment #10 from bergner at gcc dot gnu dot org 2008-02-05 16:38
---
Bootstrap and regtesting is in progress on the new patch.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29253
--- Comment #11 from bergner at gcc dot gnu dot org 2008-02-05 19:34
---
Now we get a different ICE with the following test case:
[EMAIL PROTECTED]:~/gcc/PR29253/bugs cat _abs_r16.f95
elemental function _gfortran_specific__abs_r16 (parm)
real (kind=16), intent (in) :: parm
real
--- Comment #5 from rguenth at gcc dot gnu dot org 2008-02-04 14:57 ---
Executable testcase:
extern double fabs(double);
extern void abort(void);
void __attribute__((noinline)) foo(double x)
{
if (fabs (x) != 0.0)
abort ();
}
int main ()
{
foo(-0.0);
return 0;
}
--
--- Comment #4 from rguenth at gcc dot gnu dot org 2008-02-04 14:51 ---
Created an attachment (id=15093)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15093action=view)
patch
This should fix it. Can someone test it on ppc please?
--
--- Comment #3 from dje at gcc dot gnu dot org 2008-02-04 14:27 ---
main ()
{
if (fabs (-0.0) != 0.0)
abort ();
}
compile with -msoft-float and optimization on powerpc-linux.
--
dje at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #8 from bergner at gcc dot gnu dot org 2008-02-04 17:04 ---
Ok, I bootstrapped with revision 132091. The patched compiler ICE's on the
following test case:
typedef float TFtype __attribute__ ((mode (TF)));
TFtype
divtc3 (TFtype a, TFtype b)
{
if (__builtin_fabsl (a)
--- Comment #6 from rguenth at gcc dot gnu dot org 2008-02-04 15:07 ---
Uh, and we fold ABS x != 0.0 to x != 0.0, which makes the testcase
uninteresting as well. Also it happens that for double we succeed with
generating the nojump variant. And I don't have a cross with long-double
--- Comment #7 from bergner at gcc dot gnu dot org 2008-02-04 16:30 ---
I'm testing the patch from Comment #4 on powerpc64-linux.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29253
--- Comment #2 from steven at gcc dot gnu dot org 2008-02-03 14:37 ---
Test case and HOWTO reproduce would be welcome.
--
steven at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from rguenth at gcc dot gnu dot org 2006-09-28 09:59 ---
Confirmed. As expand_copysign, expand_fabs should defer to the library in this
case.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
14 matches
Mail list logo
