From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 35037 invoked by alias); 29 Jun 2017 20:12:41 -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 35022 invoked by uid 89); 29 Jun 2017 20:12:40 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-9.8 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,GIT_PATCH_2,GIT_PATCH_3,KAM_ASCII_DIVIDERS,RCVD_IN_DNSWL_NONE,RCVD_IN_SORBS_SPAM,SPF_PASS autolearn=ham version=3.3.2 spammy= X-HELO: mail-yw0-f169.google.com Received: from mail-yw0-f169.google.com (HELO mail-yw0-f169.google.com) (209.85.161.169) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 29 Jun 2017 20:12:39 +0000 Received: by mail-yw0-f169.google.com with SMTP id l21so33069363ywb.1 for ; Thu, 29 Jun 2017 13:12:39 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to; bh=t9Zt/3bKs+1PZ8HzdqrZ62SvH/fSSViR9I9f9QAIKYE=; b=E3chLiPGDAgQmtH5m8aLe3DJeTTsa3XaWWZ5P1OvC2v8F7itcKEFALvIoE2d8qQ0Fp 6clZxtpnrkbz+ZnUVKcO2YHYnaThbJ0hqseLeTRz3om+xbXJihTs2aA6yxMcT0vnLVao 8kklsJZp6gDDIQZQ9+xd4oq7JKDawJUsnzVPdzu5acD/oI/lHUNWTRuvmwFqq66rb7Yv KapPN0lnzOftld6zle9fB8HgN73XDM+Ak0ht4EiF2A8xgrpHjscJpi+b0OncWTzZ973r cG67qUAAPJSadtgXvYOUJkSTYYa8fmCZ0l1H7nzm83r327VwUviGRAYgU2lJHCqJ/Fh7 0Jnw== X-Gm-Message-State: AKS2vOzsfwo3kAX2S9XmXSB3Nsqqd3/BQ/0gU4RLN/ektVQIIab8BVPn FVmOtDMUjtR/SGMMBZUFRjmb6X4WCyPs X-Received: by 10.129.72.17 with SMTP id v17mr4411662ywa.160.1498767157277; Thu, 29 Jun 2017 13:12:37 -0700 (PDT) MIME-Version: 1.0 Received: by 10.129.47.200 with HTTP; Thu, 29 Jun 2017 13:12:36 -0700 (PDT) From: Andrew Pinski Date: Thu, 29 Jun 2017 20:12:00 -0000 Message-ID: Subject: [PATCH] [PR 81245] Fix tree-if-conv calling of update_stmt after fold_stmt To: GCC Patches Content-Type: multipart/mixed; boundary="001a114dd1a202dbdf05531eeca1" X-IsSubscribed: yes X-SW-Source: 2017-06/txt/msg02325.txt.bz2 --001a114dd1a202dbdf05531eeca1 Content-Type: text/plain; charset="UTF-8" Content-length: 546 Hi, As described in the bug, tree-if-conv is calling update_stmt on an old stmt which might have been removed from the IR already (transforming of an assignment to a call in this case). This fixes the problem by calling update_stmt on the new statement that fold_stmt might have created. OK? Bootstrapped and tested on aarch64-linux-gnu with no regressions. Thanks, Andrew Pinski ChangeLog: * tree-if-conv.c (predicate_scalar_phi): Update new_stmt if fold_stmt returned true. testsuite/ChangeLog: * gcc.dg/torture/pr81245.c: New testcase. --001a114dd1a202dbdf05531eeca1 Content-Type: text/plain; charset="US-ASCII"; name="fixifcvt.diff.txt" Content-Disposition: attachment; filename="fixifcvt.diff.txt" Content-Transfer-Encoding: base64 X-Attachment-Id: f_j4iv5gcn0 Content-length: 1647 SW5kZXg6IHRlc3RzdWl0ZS9nY2MuZGcvdG9ydHVyZS9wcjgxMjQ1LmMKPT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PQotLS0gdGVzdHN1aXRlL2djYy5kZy90b3J0 dXJlL3ByODEyNDUuYwkobm9uZXhpc3RlbnQpCisrKyB0ZXN0c3VpdGUvZ2Nj LmRnL3RvcnR1cmUvcHI4MTI0NS5jCSh3b3JraW5nIGNvcHkpCkBAIC0wLDAg KzEsMTYgQEAKKy8qIHsgZGctb3B0aW9ucyAiLWZmYXN0LW1hdGgiIH0gKi8K Ky8qIHsgZGctZG8gY29tcGlsZSB9ICovCisvKiBUaGlzIHRlc3QgdXNlZCB0 byBjcmFzaCB0aGUgdmVjdG9yaXplciBhcyB0aGUgaWZjb252ZXJ0IHBhc3MK KyAgIHVzZWQgdG8gY29udmVydCB0aGUgaWYgdG8gY29weXNpZ24gYnV0IGNh bGxlZCB1cGRhdGVfc3RtdCBvbgorICAgdGhlIG9sZCBzdGF0ZW1lbnQgYWZ0 ZXIgY2FsbGluZyBmb2xkX3N0bXQuICovCitkb3VibGUgc2dbMThdOwordm9p ZCBmKHZvaWQpCit7CisgIGZvciAoaW50IGkgPSAwIDtpIDwgMTg7aSsrKQor ICB7CisgICAgaWYgKHNnW2ldIDwgMC4wKQorICAgICAgc2dbaV0gPSAtMS4w OworICAgIGVsc2UKKyAgICAgIHNnW2ldID0gMS4wOworICB9Cit9CkluZGV4 OiB0cmVlLWlmLWNvbnYuYwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSB0 cmVlLWlmLWNvbnYuYwkocmV2aXNpb24gMjQ5NzY5KQorKysgdHJlZS1pZi1j b252LmMJKHdvcmtpbmcgY29weSkKQEAgLTE4NTMsNyArMTg1Myw4IEBACiAg ICAgICBuZXdfc3RtdCA9IGdpbXBsZV9idWlsZF9hc3NpZ24gKHJlcywgcmhz KTsKICAgICAgIGdzaV9pbnNlcnRfYmVmb3JlIChnc2ksIG5ld19zdG10LCBH U0lfU0FNRV9TVE1UKTsKICAgICAgIGdpbXBsZV9zdG10X2l0ZXJhdG9yIG5l d19nc2kgPSBnc2lfZm9yX3N0bXQgKG5ld19zdG10KTsKLSAgICAgIGZvbGRf c3RtdCAoJm5ld19nc2ksIGlmY3Z0X2ZvbGxvd19zc2FfdXNlX2VkZ2VzKTsK KyAgICAgIGlmIChmb2xkX3N0bXQgKCZuZXdfZ3NpLCBpZmN2dF9mb2xsb3df c3NhX3VzZV9lZGdlcykpCisJbmV3X3N0bXQgPSBnc2lfc3RtdCAobmV3X2dz aSk7CiAgICAgICB1cGRhdGVfc3RtdCAobmV3X3N0bXQpOwogCiAgICAgICBp ZiAoZHVtcF9maWxlICYmIChkdW1wX2ZsYWdzICYgVERGX0RFVEFJTFMpKQo= --001a114dd1a202dbdf05531eeca1--