From abf07e52b2f16ad1234bdb89b9c8b3d704b67b11 Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Thu, 29 Nov 2018 15:26:33 +1030 Subject: [PATCH] [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 --- gcc/ChangeLog | 5 +++++ gcc/config/rs6000/predicates.md | 6 ------ 2 files changed, 5 insertions(+), 6 deletions(-) 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 + + * config/rs6000/predicates.md (easy_vector_constant): Don't call + easy_fp_constant. + 2018-11-29 Alan Modra * 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; -- 2.30.2