https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
Bill Schmidt changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #15 from Bill Schmidt ---
Matthias, the code is now fixed everywhere upstream. Do you need a merge into
ibm/gcc-5-branch?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #14 from Bill Schmidt ---
Author: wschmidt
Date: Mon Apr 4 15:47:51 2016
New Revision: 234718
URL: https://gcc.gnu.org/viewcvs?rev=234718=gcc=rev
Log:
[gcc]
2016-04-04 Bill Schmidt
Jakub
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #13 from Bill Schmidt ---
Author: wschmidt
Date: Mon Apr 4 15:45:59 2016
New Revision: 234717
URL: https://gcc.gnu.org/viewcvs?rev=234717=gcc=rev
Log:
[gcc]
2016-04-04 Bill Schmidt
Jakub
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #12 from Bill Schmidt ---
Author: wschmidt
Date: Mon Apr 4 15:42:19 2016
New Revision: 234716
URL: https://gcc.gnu.org/viewcvs?rev=234716=gcc=rev
Log:
[gcc]
2016-04-04 Bill Schmidt
Jakub
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #11 from Bill Schmidt ---
Jakub, thanks, I've verified that works and makes for a much better patch.
Will post shortly on gcc-patches.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #10 from Bill Schmidt ---
Ok, sounds good. I have vacation this afternoon, but will revisit this over
the weekend or Monday.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #9 from Jakub Jelinek ---
I've missed the pass_optimize_widening_mul::execute in your patch, that is also
another spot where you'd want to call it. But the sincos hunks should be safe
as is.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #8 from Bill Schmidt ---
The tree-inline part only shows up after fixing the part in
tree-ssa-math-opts.c, where the initial failure occurs. The DECL is already
encoded as a BUILT_IN_POW by the time we get that far.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #7 from Jakub Jelinek ---
Ah, but gimple_call_combined_fn already performs this.
So perhaps all you need is the tree-inline.c part?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #6
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
Bill Schmidt changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |wschmidt at gcc dot
gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #5 from Bill Schmidt ---
Created attachment 38156
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38156=edit
Patch that permits this to compile
The attached patch allows the compilation to succeed in spite of the incorrect
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #4 from Bill Schmidt ---
(I should say, presumably unreachable. This source code looks pretty dicey in
the first place, but nonetheless we should probably tolerate it at this stage
of optimization.)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #3 from Bill Schmidt ---
So we have an unreachable call to pow with the wrong number of arguments. I
suppose the expansion logic for builtin_pow should tolerate this situation and
just do nothing with it.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
--- Comment #2 from ktkachov at gcc dot gnu.org ---
Using gdb, the gimple stmt causing the ICE is:
# .MEM_10 = VDEF <.MEM_1(D)>
ret_5 = pow (1.0e+0);
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70457
ktkachov at gcc dot gnu.org changed:
What|Removed |Added
Target|powerpc64le-linux-gnu |powerpc64le-linux-gnu,
17 matches
Mail list logo