From: Janne Blomqvist <jb@gcc.gnu.org>
Date: Mon, 25 Sep 2017 06:44:18 +0000 (+0300)
Subject: Remove unnecessary fold_convert in gfc_(un)likely
X-Git-Tag: upstream/12.2.0~36770
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b35659407b92cc26f48a9a8d5acad63d9c8b212b;p=platform%2Fupstream%2Fgcc.git

Remove unnecessary fold_convert in gfc_(un)likely

This patch removes an unnecessary fold_convert to boolean_type_node at
the end of gfc_likely and gfc_unlikely.  It makes no difference to the
generated code, but makes tree dumps a little bit cleaner.

2017-09-25  Janne Blomqvist  <jb@gcc.gnu.org>
        * trans.c (gfc_unlikely): Remove unnecessary fold_convert.
        (gfc_likely): Likewise.

Regtested on x86_64-pc-linux-gnu.

From-SVN: r253132
---

diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog
index 29504c8..4d64a95 100644
--- a/gcc/fortran/ChangeLog
+++ b/gcc/fortran/ChangeLog
@@ -1,3 +1,8 @@
+2017-09-25  Janne Blomqvist  <jb@gcc.gnu.org>
+
+	* trans.c (gfc_unlikely): Remove unnecessary fold_convert.
+	(gfc_likely): Likewise.
+
 2017-09-24  Thomas Koenig  <tkoenig@gcc.gnu.org>
 	    Steven G. Kargl  <kargl@gcc.gnu.org>
 
diff --git a/gcc/fortran/trans.c b/gcc/fortran/trans.c
index 149f482..53bc428 100644
--- a/gcc/fortran/trans.c
+++ b/gcc/fortran/trans.c
@@ -2275,7 +2275,6 @@ gfc_unlikely (tree cond, enum br_predictor predictor)
 				  build_int_cst (integer_type_node,
 						 predictor));
     }
-  cond = fold_convert (boolean_type_node, cond);
   return cond;
 }
 
@@ -2297,7 +2296,6 @@ gfc_likely (tree cond, enum br_predictor predictor)
 				  build_int_cst (integer_type_node,
 						 predictor));
     }
-  cond = fold_convert (boolean_type_node, cond);
   return cond;
 }