Hi,
This is an obvious patch removing redundant check on component distance in
tree-predcom.c  Bootstrap and test along with next patch.  Is it OK?

Thanks,
bin
2017-11-15  Bin Cheng  <bin.ch...@arm.com>

        * tree-predcom.c (add_ref_to_chain): Remove check on distance.
From 8b62802309b2d14a2fca4446b9f6f8f8670a450b Mon Sep 17 00:00:00 2001
From: Bin Cheng <binch...@e108451-lin.cambridge.arm.com>
Date: Fri, 20 Oct 2017 15:56:03 +0100
Subject: [PATCH 1/2] redundant-dist-check-20171102.txt

---
 gcc/tree-predcom.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/gcc/tree-predcom.c b/gcc/tree-predcom.c
index 747c1b8..499cedb 100644
--- a/gcc/tree-predcom.c
+++ b/gcc/tree-predcom.c
@@ -1063,11 +1063,6 @@ add_ref_to_chain (chain_p chain, dref ref)
 
   gcc_assert (wi::les_p (root->offset, ref->offset));
   widest_int dist = ref->offset - root->offset;
-  if (wi::leu_p (MAX_DISTANCE, dist))
-    {
-      free (ref);
-      return;
-    }
   gcc_assert (wi::fits_uhwi_p (dist));
 
   chain->refs.safe_push (ref);
-- 
1.9.1

Reply via email to