Re: [PATCH] Fix PR70457 (ICE on incompatible call to built-in pow)

2016-04-04 Thread Jakub Jelinek
On Mon, Apr 04, 2016 at 10:02:44AM -0500, Bill Schmidt wrote: > Sorry again for the misunderstanding. Here are revised patches and > change logs for trunk, GCC 5, and GCC 4.9. Note that GCC 5 and GCC 4.9 > have additional exposures in tree-ssa-math-opts.c that I've repaired > similarly. The only

Re: [PATCH] Fix PR70457 (ICE on incompatible call to built-in pow)

2016-04-04 Thread Bill Schmidt
Hi Jakub, Sorry again for the misunderstanding. Here are revised patches and change logs for trunk, GCC 5, and GCC 4.9. Note that GCC 5 and GCC 4.9 have additional exposures in tree-ssa-math-opts.c that I've repaired similarly. The only differences between 5 and 4.9 are the names of the affecte

Re: [PATCH] Fix PR70457 (ICE on incompatible call to built-in pow)

2016-04-04 Thread Bill Schmidt
OK, sorry for misreading the note. This is exactly what I've done for the GCC 5 and GCC 4.9 versions, so I'll update the GCC 6 version to do the same. Bill On Mon, 2016-04-04 at 08:57 +0200, Jakub Jelinek wrote: > On Sun, Apr 03, 2016 at 06:43:47PM -0500, Bill Schmidt wrote: > > --- tree-inline.

Re: [PATCH] Fix PR70457 (ICE on incompatible call to built-in pow)

2016-04-03 Thread Jakub Jelinek
On Sun, Apr 03, 2016 at 06:43:47PM -0500, Bill Schmidt wrote: > --- tree-inline.c (revision 234702) > +++ tree-inline.c (working copy) > @@ -57,8 +57,8 @@ along with GCC; see the file COPYING3. If not see > #include "cfgloop.h" > #include "builtins.h" > #include "tree-chkp.h" > +#includ

Re: [PATCH] Fix PR70457 (ICE on incompatible call to built-in pow)

2016-04-03 Thread Bill Schmidt
On Sun, 2016-04-03 at 18:43 -0500, Bill Schmidt wrote: > Hi, > > PR70457 shows that we ICE if a call to pow() has only one argument, or > more generally doesn't match its expected signature. This can happen > both during the widen-mult phase and the tree-inline phase. As > suggested by Jakub, th

[PATCH] Fix PR70457 (ICE on incompatible call to built-in pow)

2016-04-03 Thread Bill Schmidt
Hi, PR70457 shows that we ICE if a call to pow() has only one argument, or more generally doesn't match its expected signature. This can happen both during the widen-mult phase and the tree-inline phase. As suggested by Jakub, this patch uses gimple_call_combined_fn to test for this, similarly t