From: Alan Modra <amodra@gmail.com>
Date: Thu, 29 Nov 2018 04:56:33 +0000 (+1030)
Subject: [RS6000] Bogus easy_fp_constant call
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=abf07e52b2f16ad1234bdb89b9c8b3d704b67b11;p=gcc.git

[RS6000] Bogus easy_fp_constant call

easy_fp_constant accepts const_double, easy_vector_constant accepts
const_vector.  Calling one from the other therefore will always return
false.  I believe the zero_constant call in easy_vector_constant will
cover the 0.0 case of easy_fp_constant.

	* config/rs6000/predicates.md (easy_vector_constant): Don't call
	easy_fp_constant.

From-SVN: r266606
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 56dd30f9b3e..b1ea3999752 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2018-11-29  Alan Modra  <amodra@gmail.com>
+
+	* config/rs6000/predicates.md (easy_vector_constant): Don't call
+	easy_fp_constant.
+
 2018-11-29  Alan Modra  <amodra@gmail.com>
 
 	* config.in (HAVE_AS_PLTSEQ): Add.
diff --git a/gcc/config/rs6000/predicates.md b/gcc/config/rs6000/predicates.md
index e98b00dec5d..afe1b286833 100644
--- a/gcc/config/rs6000/predicates.md
+++ b/gcc/config/rs6000/predicates.md
@@ -644,12 +644,6 @@
 (define_predicate "easy_vector_constant"
   (match_code "const_vector")
 {
-  /* Because IEEE 128-bit floating point is considered a vector type
-     in order to pass it in VSX registers, it might use this function
-     instead of easy_fp_constant.  */
-  if (FLOAT128_VECTOR_P (mode))
-    return easy_fp_constant (op, mode);
-
   if (VECTOR_MEM_ALTIVEC_OR_VSX_P (mode))
     {
       int value = 256;