https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90187
--- Comment #8 from Jakub Jelinek ---
Author: jakub
Date: Wed Apr 24 09:27:14 2019
New Revision: 270537
URL: https://gcc.gnu.org/viewcvs?rev=270537=gcc=rev
Log:
PR target/90187
* config/i386/i386.c (ix86_expand_sse_fp_minmax):
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90187
--- Comment #7 from Uroš Bizjak ---
(In reply to Jakub Jelinek from comment #6)
> Or would you prefer:
> --- gcc/config/i386/i386.c.jj 2019-04-16 10:40:15.077091789 +0200
> +++ gcc/config/i386/i386.c2019-04-23 11:55:59.397227347 +0200
> @@
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90187
--- Comment #6 from Jakub Jelinek ---
Or would you prefer:
--- gcc/config/i386/i386.c.jj 2019-04-16 10:40:15.077091789 +0200
+++ gcc/config/i386/i386.c 2019-04-23 11:55:59.397227347 +0200
@@ -23712,7 +23712,10 @@ ix86_expand_sse_fp_minmax
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90187
--- Comment #5 from Jakub Jelinek ---
Created attachment 46228
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46228=edit
gcc9-pr90187.patch
Untested fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90187
Jakub Jelinek changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90187
--- Comment #3 from Uroš Bizjak ---
It looks like middle-end is bypassing sminv2df3 expander and constructs RTX by
hand. This should not be done, since the expander decides whether IEEE or
non-IEEE variant should be used.
Please note that there
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90187
--- Comment #2 from Jakub Jelinek ---
Simplified testcase:
double a[64];
double *foo (void);
void
bar (int x, const double *y)
{
int i;
for (i = 0; i < x; i++)
if (y[i] < a[i])
a[i] = y[i];
}
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90187
Jakub Jelinek changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
