Reapply r269790 which was missed during rebase.
authorMartin Liska <mliska@suse.cz>
Fri, 10 May 2019 07:22:31 +0000 (09:22 +0200)
committerMartin Liska <marxin@gcc.gnu.org>
Fri, 10 May 2019 07:22:31 +0000 (07:22 +0000)
2019-05-10  Martin Liska  <mliska@suse.cz>

* config/i386/i386-expand.c (ix86_expand_floorceildf_32):
Reapply changes from r269790.

From-SVN: r271054

gcc/ChangeLog
gcc/config/i386/i386-expand.c

index 3c2c13d48301f6dc763bf7d326238b099ff9af91..5e30430cbe2a4188e8e56551d3fb467b88e34136 100644 (file)
@@ -1,3 +1,8 @@
+2019-05-10  Martin Liska  <mliska@suse.cz>
+
+       * config/i386/i386-expand.c (ix86_expand_floorceildf_32):
+       Reapply changes from r269790.
+
 2019-05-10  Martin Liska  <mliska@suse.cz>
 
        PR middle-end/90340
index b7ce5d0975bbf1915e9c80816036fcc823927eb1..a55d4923be481f2a083bee081bc2203a96a44d85 100644 (file)
@@ -15533,8 +15533,10 @@ ix86_expand_floorceildf_32 (rtx operand0, rtx operand1, bool do_floor)
           x2 -= 1;
      Compensate.  Ceil:
         if (x2 < x)
-          x2 -= -1;
-        return x2;
+          x2 += 1;
+       if (HONOR_SIGNED_ZEROS (mode))
+         x2 = copysign (x2, x);
+       return x2;
    */
   machine_mode mode = GET_MODE (operand0);
   rtx xa, TWO52, tmp, one, res, mask;
@@ -15560,17 +15562,16 @@ ix86_expand_floorceildf_32 (rtx operand0, rtx operand1, bool do_floor)
   /* xa = copysign (xa, operand1) */
   ix86_sse_copysign_to_positive (xa, xa, res, mask);
 
-  /* generate 1.0 or -1.0 */
-  one = force_reg (mode,
-                  const_double_from_real_value (do_floor
-                                                ? dconst1 : dconstm1, mode));
+  /* generate 1.0 */
+  one = force_reg (mode, const_double_from_real_value (dconst1, mode));
 
   /* Compensate: xa = xa - (xa > operand1 ? 1 : 0) */
   tmp = ix86_expand_sse_compare_mask (UNGT, xa, res, !do_floor);
   emit_insn (gen_rtx_SET (tmp, gen_rtx_AND (mode, one, tmp)));
-  /* We always need to subtract here to preserve signed zero.  */
-  tmp = expand_simple_binop (mode, MINUS,
+  tmp = expand_simple_binop (mode, do_floor ? MINUS : PLUS,
                             xa, tmp, NULL_RTX, 0, OPTAB_DIRECT);
+  if (!do_floor && HONOR_SIGNED_ZEROS (mode))
+    ix86_sse_copysign_to_positive (tmp, tmp, res, mask);
   emit_move_insn (res, tmp);
 
   emit_label (label);