From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 80865 invoked by alias); 31 Jul 2017 11:16:28 -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 80824 invoked by uid 89); 31 Jul 2017 11:16:26 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.9 required=5.0 tests=AWL,BAYES_00,KAM_MANYTO,RCVD_IN_DNSWL_LOW,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 spammy=HTo:U*ro, reworks, H*M:a5e8, realignment X-HELO: sasl.smtp.pobox.com Received: from pb-smtp2.pobox.com (HELO sasl.smtp.pobox.com) (64.147.108.71) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 31 Jul 2017 11:16:24 +0000 Received: from sasl.smtp.pobox.com (unknown [127.0.0.1]) by pb-smtp2.pobox.com (Postfix) with ESMTP id 2F70E9AC3A; Mon, 31 Jul 2017 07:16:20 -0400 (EDT) Received: from pb-smtp2.nyi.icgroup.com (unknown [127.0.0.1]) by pb-smtp2.pobox.com (Postfix) with ESMTP id 259819AC39; Mon, 31 Jul 2017 07:16:20 -0400 (EDT) Received: from [192.168.1.4] (unknown [76.215.41.237]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by pb-smtp2.pobox.com (Postfix) with ESMTPSA id 1B04D9AC35; Mon, 31 Jul 2017 07:16:19 -0400 (EDT) To: gcc-patches , Uros Bizjak , Jan Hubicka , "H.J. Lu" , Martin Liska , Rainer Orth , Mike Stump From: Daniel Santos Subject: [PATCH 0/6] [i386] PR80969 Fix ICE with -mabi=ms -mavx512f Message-ID: Date: Mon, 31 Jul 2017 11:16:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="------------57BEFE06787A9A992EBF0363" X-Pobox-Relay-ID: AD6E22A4-75E1-11E7-8CD1-9D2B0D78B957-06139138!pb-smtp2.pobox.com X-IsSubscribed: yes X-SW-Source: 2017-07/txt/msg02005.txt.bz2 This is a multi-part message in MIME format. --------------57BEFE06787A9A992EBF0363 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-length: 1114 When working on the Wine64 project to use aligned SSE MOVs after SP realignment and adding -mcall-ms2sysv-xlogues, I overlooked the fact that the function body may require a stack alignment greater than 16-bytes. This can result in an ICE with -mabi=ms -mavx512f and some other cases. This patch set reworks the strategy for calculating the frame layout following normal (inline) integral register saves (at frame.reg_save_offset) to the start of the frame for the local function (frame.frame_pointer_offset). I've completed a bootstrap and full regression test with no additional failures, but I don't have access to a machine with avx512 extensions. I have manually run the tests that need it using the Intel SDE, but I haven't been able to validate that my check_effective_target_avx512f_runtime code in gcc/testsuite/lib/target-supports.exp is correctly enabling the tests for pr80969-4*.c. As an aside note, I still have some rework of the ms-sysv.exp tests that I haven't yet to submitted and in which I'm adding more tests for cases with uncommon stacks, as in PR 81563. Thanks, Daniel --------------57BEFE06787A9A992EBF0363 Content-Type: text/plain; charset=UTF-8; name="pr80969.gcc.ChangeLog" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="pr80969.gcc.ChangeLog" Content-length: 968 MjAxNy0wNy0yMyAgRGFuaWVsIFNhbnRvcyAgPGRhbmllbC5zYW50b3NAcG9i b3guY29tPgoKCSogY29uZmlnL2kzODYvaTM4Ni5oIChpeDg2X2ZyYW1lOjpv dXRsaW5lZF9zYXZlX29mZnNldCk6IFJlbW92ZSBmaWVsZC4KCShpeDg2X2Zy YW1lOjpzdGFja19yZWFsaWduX2FsbG9jYXRlX29mZnNldCk6IExpa2V3aXNl LgoJKGl4ODZfZnJhbWU6OnN0YWNrX3JlYWxpZ25fYWxsb2NhdGUpOiBOZXcg ZmllbGQuCgkoc3RydWN0IG1hY2hpbmVfZnJhbWVfc3RhdGUpOiBNb2RpZnkg Y29tbWVudHMuCgkobWFjaGluZV9mcmFtZV9zdGF0ZTo6c3BfcmVhbGlnbmVk X2ZwX2VuZCk6IE5ldyBmaWVsZC4KCShtYWNoaW5lX2Z1bmN0aW9uOjpjYWxs X21zMnN5c3ZfcGFkX291dCk6IFJlbW92ZSBmaWVsZC4KCSogY29uZmlnL2kz ODYvaTM4Ni5jICh4bG9ndWVfbGF5b3V0OjpnZXRfc3RhY2tfc3BhY2VfdXNl ZCk6IE1vZGlmeS4KCShpeDg2X2NvbXB1dGVfZnJhbWVfbGF5b3V0KTogTGlr ZXdpc2UuCgkoc3BfdmFsaWRfYXQpOiBMaWtld2lzZS4KCShmcF92YWxpZF9h dCk6IExpa2V3aXNlLgoJKGNob29zZV9iYXNlYWRkcik6IE1vZGlmeSBjb21t ZW50cy4KCShpeDg2X2VtaXRfb3V0bGluZWRfbXMyc3lzdl9zYXZlKTogTW9k aWZ5LgoJKGl4ODZfZXhwYW5kX3Byb2xvZ3VlKTogTGlrZXdpc2UuCgkoaXg4 Nl9leHBhbmRfZXBpbG9ndWUpOiBNb2RpZnkgY29tbWVudHMuCg== --------------57BEFE06787A9A992EBF0363 Content-Type: text/plain; charset=UTF-8; name="pr80969.gcc.testsuite.ChangeLog" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="pr80969.gcc.testsuite.ChangeLog" Content-length: 480 MjAxNy0wNy0yMyAgRGFuaWVsIFNhbnRvcyAgPGRhbmllbC5zYW50b3NAcG9i b3guY29tPgoJKiBnY2MudGFyZ2V0L2kzODYvcHI4MDk2OS0xLmM6IE5ldyB0 ZXN0Y2FzZS4KCSogZ2NjLnRhcmdldC9pMzg2L3ByODA5NjktMmEuYzogTGlr ZXdpc2UuCgkqIGdjYy50YXJnZXQvaTM4Ni9wcjgwOTY5LTIuYzogTGlrZXdp c2UuCgkqIGdjYy50YXJnZXQvaTM4Ni9wcjgwOTY5LTMuYzogTGlrZXdpc2Uu CgkqIGdjYy50YXJnZXQvaTM4Ni9wcjgwOTY5LTRhLmM6IExpa2V3aXNlLgoJ KiBnY2MudGFyZ2V0L2kzODYvcHI4MDk2OS00Yi5jOiBMaWtld2lzZS4KCSog Z2NjLnRhcmdldC9pMzg2L3ByODA5NjktNC5jOiBMaWtld2lzZS4K --------------57BEFE06787A9A992EBF0363--