From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 124414 invoked by alias); 2 Dec 2015 21:36:40 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 124404 invoked by uid 89); 2 Dec 2015 21:36:39 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=0.6 required=5.0 tests=AWL,BAYES_20,KAM_LAZY_DOMAIN_SECURITY,T_RP_MATCHES_RCVD autolearn=no version=3.3.2 X-HELO: usmailout2.samsung.com Received: from mailout2.w2.samsung.com (HELO usmailout2.samsung.com) (211.189.100.12) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Wed, 02 Dec 2015 21:36:39 +0000 Received: from uscpsbgm1.samsung.com (u114.gpu85.samsung.co.kr [203.254.195.114]) by mailout2.w2.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTP id <0NYR005G42P07O30@mailout2.w2.samsung.com> for gcc-patches@gcc.gnu.org; Wed, 02 Dec 2015 16:36:36 -0500 (EST) Received: from ussync4.samsung.com ( [203.254.195.84]) by uscpsbgm1.samsung.com (USCPMTA) with SMTP id 98.99.23844.4646F565; Wed, 2 Dec 2015 16:36:36 -0500 (EST) Received: from WSPOP ([105.140.33.249]) by ussync4.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTPA id <0NYR003DS2OZSP80@ussync4.samsung.com>; Wed, 02 Dec 2015 16:36:36 -0500 (EST) From: Sebastian Paul Pop To: 'Richard Biener' , sebpop@gmail.com Cc: gcc-patches@gcc.gnu.org, hiraditya@msn.com References: <1449074094-15117-1-git-send-email-s.pop@samsung.com> <1449074094-15117-2-git-send-email-s.pop@samsung.com> <26A4BFA1-EF5B-4DA1-A203-4EA3B41AC996@gmail.com> In-reply-to: <26A4BFA1-EF5B-4DA1-A203-4EA3B41AC996@gmail.com> Subject: RE: [PATCH 2/2] [graphite] fix invalid bounds on array refs Date: Wed, 02 Dec 2015 21:36:00 -0000 Message-id: <005501d12d49$856b2870$90417950$@samsung.com> MIME-version: 1.0 Content-type: text/plain; charset=UTF-8 Content-transfer-encoding: quoted-printable X-SW-Source: 2015-12/txt/msg00369.txt.bz2 Do you recommend that we add a gcc_assert that min is always lower than max? The change in Graphite code can be reverted then: >+ /* Fortran has some arrays where high bound is -1 and low is 0. */ >+ if (integer_onep (fold_build2 (LT_EXPR, boolean_type_node, high, >low))) >+ return false; -----Original Message----- But either that is the case or the frontend has a bug and should be fixed. = So your patch doesn't make any sense. Richard.