https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49959
Andrew Pinski changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49959
--- Comment #6 from CVS Commits ---
The trunk branch has been updated by Andrew Pinski :
https://gcc.gnu.org/g:69f1a8af45d8a42003c21489019ddfb01d88d30d
commit r14-573-g69f1a8af45d8a42003c21489019ddfb01d88d30d
Author: Andrew Pinski
Date: Wed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49959
--- Comment #5 from Andrew Pinski ---
Created attachment 55016
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55016&action=edit
Patch
For some reason it didn't make it to the mailing list yet. Will figure out why
in a few. patches 2 and 3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49959
--- Comment #4 from Andrew Pinski ---
Note Pre is able to remove the casts since GCC 8.
So the original testcase is fixed but if you change it to:
#define ABS(X)(((X)>0)?(X):-(X))
unsigned long
test_abs(int *cur)
{
unsigned long sad = 0;
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49959
Andrew Pinski changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |pinskia at gcc dot
gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49959
--- Comment #2 from Andrew Pinski 2012-01-21
08:03:40 UTC ---
If we had a pass which was able to convert:
:
iftmp.0_6 = (long unsigned int) D.1708_4;
goto ;
:
D.1712_8 = -D.1708_4;
iftmp.0_9 = (long unsigned int) D.1712_8;
:
# iftmp.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49959
Richard Guenther changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
