public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
From: "manu at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug tree-optimization/59124] [4.9/5/6 Regression] Wrong warnings "array subscript is above array bounds" Date: Fri, 18 Sep 2015 18:32:00 -0000 [thread overview] Message-ID: <bug-59124-4-0LO76xs9CH@http.gcc.gnu.org/bugzilla/> (raw) In-Reply-To: <bug-59124-4@http.gcc.gnu.org/bugzilla/> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59124 --- Comment #28 from Manuel López-Ibáñez <manu at gcc dot gnu.org> --- (In reply to baoshan from comment #27) > > It seems GCC at some moment unrolls the loop and creates such block with > > those ranges. Probably, the block is unreachable, but it would be better to > > not create it in the first place. Finding out where and why it is created > > would help to figure out a fix. > > > At pass "cunrolli", it would unroll the loops according the estimated > iterate times. The problem is at this time it use array > ref(infer_loop_bounds_from_ref) to infer the iterate times which is not > accurate. The inaccurate iterate times result the extra blocks. The question is why is it inaccurate? Note that cunrolli says: Statement _14 = baz[_9]; is executed at most 5 (bounded by 5) + 1 times in loop 2. Loop 2 iterates at most 6 times. Analyzing # of iterations of loop 2 exit condition [i_2 + 4294967295, + , 4294967295] != 0 bounds on difference of bases: -4294967295 ... 0 Applying pattern match.pd:71, generic-match.c:9500 result: # of iterations i_2 + 4294967295, bounded by 4294967295 Estimating sizes for loop 2 BB: 7, after_exit: 0 size: 2 if (j_3 != 0) Exit condition will be eliminated in last copy. BB: 6, after_exit: 1 size: 1 _9 = j_3 + 4294967295; size: 0 _10 = (long unsigned int) _9; Applying pattern match.pd:71, generic-match.c:9500 Applying pattern match.pd:136, generic-match.c:6019 size: 1 _11 = _10 * 4; Applying pattern match.pd:71, generic-match.c:9500 Applying pattern match.pd:136, generic-match.c:6019 size: 1 _13 = bar_12(D) + _11; size: 1 _14 = baz[_9]; size: 1 *_13 = _14; size: 7-0, last_iteration: 2-2 Loop size: 7 Estimated size after unrolling: 28 pr59124.c:8:5: note: loop turned into non-loop; it never loops. pr59124.c:8:5: note: loop with 7 iterations completely unrolled Last iteration exit edge was proved true. Forced statement unreachable: _14 = baz[_9]; However, it is clear that _14 = baz[_9] is executed only 5 times (not 5 times + 1). Why is this estimate wrong? >From gcc-bugs-return-497534-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Fri Sep 18 18:53:24 2015 Return-Path: <gcc-bugs-return-497534-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org> Delivered-To: listarch-gcc-bugs@gcc.gnu.org Received: (qmail 11657 invoked by alias); 18 Sep 2015 18:53:24 -0000 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: <gcc-bugs.gcc.gnu.org> List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/> List-Post: <mailto:gcc-bugs@gcc.gnu.org> List-Help: <mailto:gcc-bugs-help@gcc.gnu.org> Sender: gcc-bugs-owner@gcc.gnu.org Delivered-To: mailing list gcc-bugs@gcc.gnu.org Received: (qmail 9437 invoked by uid 48); 18 Sep 2015 18:53:20 -0000 From: "pangbw at gmail dot com" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug tree-optimization/59124] [4.9/5/6 Regression] Wrong warnings "array subscript is above array bounds" Date: Fri, 18 Sep 2015 18:53:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: tree-optimization X-Bugzilla-Version: 4.8.3 X-Bugzilla-Keywords: diagnostic X-Bugzilla-Severity: normal X-Bugzilla-Who: pangbw at gmail dot com X-Bugzilla-Status: NEW X-Bugzilla-Resolution: X-Bugzilla-Priority: P2 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: 4.9.4 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: <bug-59124-4-AF85ZgoiXK@http.gcc.gnu.org/bugzilla/> In-Reply-To: <bug-59124-4@http.gcc.gnu.org/bugzilla/> References: <bug-59124-4@http.gcc.gnu.org/bugzilla/> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-SW-Source: 2015-09/txt/msg01512.txt.bz2 Content-length: 378 https://gcc.gnu.org/bugzilla/show_bug.cgi?idY124 --- Comment #29 from baoshan <pangbw at gmail dot com> --- > However, it is clear that _14 = baz[_9] is executed only 5 times (not 5 > times + 1). Why is this estimate wrong? The max value of n is 6, so the max value of i is 5 as "i < n", then the max value of j is 4 as "j = i - 1" which means the max iterate times is 4.
next prev parent reply other threads:[~2015-09-18 18:32 UTC|newest] Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-11-14 0:44 [Bug tree-optimization/59124] New: [4.8 " d.g.gorbachev at gmail dot com 2013-11-14 9:51 ` [Bug tree-optimization/59124] [4.8/4.9 " rguenth at gcc dot gnu.org 2013-11-14 17:56 ` d.g.gorbachev at gmail dot com 2013-11-21 14:39 ` rguenth at gcc dot gnu.org 2014-03-12 14:33 ` jakub at gcc dot gnu.org 2014-05-22 9:03 ` [Bug tree-optimization/59124] [4.8/4.9/4.10 " rguenth at gcc dot gnu.org 2014-12-19 13:25 ` [Bug tree-optimization/59124] [4.8/4.9/5 " jakub at gcc dot gnu.org 2015-01-27 9:50 ` rguenth at gcc dot gnu.org 2015-01-27 10:59 ` rguenth at gcc dot gnu.org 2015-02-18 2:22 ` solar-gcc at openwall dot com 2015-02-18 4:37 ` solar-gcc at openwall dot com 2015-02-19 14:14 ` rguenth at gcc dot gnu.org 2015-02-24 13:09 ` rguenth at gcc dot gnu.org 2015-04-16 12:14 ` [Bug tree-optimization/59124] [4.8/4.9/5/6 " georgmueller at gmx dot net 2015-05-26 15:34 ` georgmueller at gmx dot net 2015-06-01 23:49 ` daniel at imperfectcode dot com 2015-06-23 8:16 ` rguenth at gcc dot gnu.org 2015-06-26 19:53 ` [Bug tree-optimization/59124] [4.9/5/6 " jakub at gcc dot gnu.org 2015-06-26 20:26 ` jakub at gcc dot gnu.org 2015-09-10 21:04 ` pangbw at gmail dot com 2015-09-11 0:29 ` manu at gcc dot gnu.org 2015-09-11 16:13 ` pangbw at gmail dot com 2015-09-11 16:51 ` manu at gcc dot gnu.org 2015-09-17 18:18 ` pangbw at gmail dot com 2015-09-17 19:02 ` pangbw at gmail dot com 2015-09-18 17:59 ` pangbw at gmail dot com 2015-09-18 18:32 ` manu at gcc dot gnu.org [this message] 2015-09-18 19:17 ` manu at gcc dot gnu.org 2015-09-18 21:11 ` pangbw at gmail dot com 2015-09-22 20:06 ` pangbw at gmail dot com 2021-01-05 9:14 ` [Bug tree-optimization/59124] [6 " szotsaki at gmail dot com
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=bug-59124-4-0LO76xs9CH@http.gcc.gnu.org/bugzilla/ \ --to=gcc-bugzilla@gcc.gnu.org \ --cc=gcc-bugs@gcc.gnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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).