public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
From: "spop at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug tree-optimization/61929] [Graphite]: Banerjee and Omega differ on distance vectors (internal compiler error in compute_affine_dependence) Date: Fri, 17 Jul 2015 16:34:00 -0000 [thread overview] Message-ID: <bug-61929-4-ZJAK0Wjb5m@http.gcc.gnu.org/bugzilla/> (raw) In-Reply-To: <bug-61929-4@http.gcc.gnu.org/bugzilla/> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61929 --- Comment #10 from Sebastian Pop <spop at gcc dot gnu.org> --- Author: spop Date: Fri Jul 17 16:34:21 2015 New Revision: 225942 URL: https://gcc.gnu.org/viewcvs?rev=225942&root=gcc&view=rev Log: [graphite] fix pr61929 This fixes bootstrap of GCC with BOOT_CFLAGS="-g -O2 -fgraphite-identity -floop-nest-optimize -floop-block -floop-interchange -floop-strip-mine". It passes regstrap on amd64-linux. A previous change (https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=213816), replaced isl_int with isl_val because isl_int would be deprecated. Since isl_val has stricter checks, it exposed the bug. In the test case (isl_set_max_val) would return infinity which would remain unchecked. We now check if the value returned is an integer or not, and bail out if it isn't. The other problem was that we were allowing all kinds of data-refs in a scop. Now we discard a scop if it has any date-ref other than (ARRAY_REF, MEM_REF, COMPONENT_REF). PR middle-end/61929 * graphite-dependences.c (add_pdr_constraints): Renamed pdr->extent to pdr->subscript_sizes. * graphite-interchange.c (build_linearized_memory_access): Add back all gcc_assert's that the "isl_int to isl_val conversion" patch has removed. Refactored. (pdr_stride_in_loop): Renamed pdr->extent to pdr->subscript_sizes. * graphite-poly.c (new_poly_dr): Same. (free_poly_dr): Same. * graphite-poly.h (struct poly_dr): Same. * graphite-scop-detection.c (stmt_has_simple_data_refs_p): Ignore all data references other than ARRAY_REF, MEM_REF, and COMPONENT_REF. * graphite-scop-detection.h: Fix space. * graphite-sese-to-poly.c (build_pbb_scattering_polyhedrons): Add back all gcc_assert's removed by a previous patch. (wrap): Remove the_isl_ctx global variable that the same patch has added. (build_loop_iteration_domains): Same. (add_param_constraints): Same. (pdr_add_data_dimensions): Same. Refactored. (build_poly_dr): Renamed extent to subscript_sizes. testsuite/ PR middle-end/61929 * gcc.dg/graphite/pr61929.c: New. Added: trunk/gcc/testsuite/gcc.dg/graphite/pr61929.c Modified: trunk/gcc/ChangeLog trunk/gcc/graphite-dependences.c trunk/gcc/graphite-interchange.c trunk/gcc/graphite-poly.c trunk/gcc/graphite-poly.h trunk/gcc/graphite-scop-detection.c trunk/gcc/graphite-scop-detection.h trunk/gcc/graphite-sese-to-poly.c trunk/gcc/testsuite/ChangeLog
next prev parent reply other threads:[~2015-07-17 16:34 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-07-27 19:39 [Bug tree-optimization/61929] New: Graphite: " gcc at crg4 dot com 2014-07-29 3:24 ` [Bug tree-optimization/61929] " gcc at crg4 dot com 2014-07-29 4:10 ` gcc at crg4 dot com 2014-07-29 4:56 ` gcc at crg4 dot com 2014-07-29 15:10 ` gcc at crg4 dot com 2015-06-22 13:11 ` [Bug tree-optimization/61929] [Graphite]: " gcc at crg4 dot com 2015-06-22 13:50 ` gcc at crg4 dot com 2015-06-22 14:30 ` gcc at crg4 dot com 2015-06-22 14:32 ` gcc at crg4 dot com 2015-07-16 16:37 ` spop at gcc dot gnu.org 2015-07-17 16:34 ` spop at gcc dot gnu.org [this message] 2015-07-17 16:35 ` spop at gcc dot gnu.org 2015-07-18 0:47 ` spop at gcc dot gnu.org 2015-07-23 21:10 ` nheghathivhistha 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-61929-4-ZJAK0Wjb5m@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).