public inbox for fortran@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH, fortran, committed] Remove unnecessary fold_convert in gfc_(un)likely
@ 2017-09-25  6:47 Janne Blomqvist
  0 siblings, 0 replies; only message in thread
From: Janne Blomqvist @ 2017-09-25  6:47 UTC (permalink / raw)
  To: fortran, gcc-patches; +Cc: Janne Blomqvist

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. Committed as obvious.
---
 gcc/fortran/trans.c | 2 --
 1 file changed, 2 deletions(-)

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;
 }
 
-- 
2.7.4

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-09-25  6:47 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-25  6:47 [PATCH, fortran, committed] Remove unnecessary fold_convert in gfc_(un)likely Janne Blomqvist

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).