From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1264 invoked by alias); 2 May 2017 10:22:16 -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 1250 invoked by uid 89); 2 May 2017 10:22:15 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.4 required=5.0 tests=BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,RCVD_IN_SORBS_SPAM,SPF_PASS autolearn=no version=3.3.2 spammy=H*i:sk:PM7g@ma, H*MI:sk:PM7g@ma, H*f:sk:PM7g@ma, H*i:sk:WpsVuMR X-HELO: mail-pf0-f196.google.com Received: from mail-pf0-f196.google.com (HELO mail-pf0-f196.google.com) (209.85.192.196) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 02 May 2017 10:22:14 +0000 Received: by mail-pf0-f196.google.com with SMTP id g23so32187935pfj.1 for ; Tue, 02 May 2017 03:22:16 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:references:cc:from:message-id:date :user-agent:mime-version:in-reply-to; bh=3E8t+klGO/DeM6y4aGiYU5vCFTmYEBz4GsyKKy6j0pM=; b=BdjX+64c6HFovahFLJE7VjnUwv7ZN0F8xUAxpsepPXMXULnJd+fFJb7OQzJKuwvVnv QJpnoz+uvK+WxrE4VufwhZyxlCPsPf37hutmAfdqqqxu0RqfnHzkUIQAQX4UkK3hp6AS 8ORW5fAkoG100wh1fEXNzlZJDAFT2N0kBImySyBDKmt1vICKRUX/9Jxv/xFc9XsjMpf4 h6CWdHwCX3Vi6YBiXHA6nwQ9fu3pYVph6uHOML8uFRJk+/eQLrZ0VK57LQ8jToK8KcUt SWembgQ4PkY2dhn7byR14nhQBBXq5m+KedM+ZeyoppSwDWgAWMQxJ9UgDeDFW/98By6M V38g== X-Gm-Message-State: AN3rC/5jMrxUJEhse4g91wrb421y0UdCSeFlhkGEVqUlINN++xpseVhG uEodVVjLylLWSw== X-Received: by 10.99.44.140 with SMTP id s134mr31713839pgs.178.1493720534913; Tue, 02 May 2017 03:22:14 -0700 (PDT) Received: from ?IPv6:2001:e68:4075:98b3:f64d:30ff:fe63:5a5a? ([2001:e68:4075:98b3:f64d:30ff:fe63:5a5a]) by smtp.gmail.com with ESMTPSA id y6sm40952719pgc.40.2017.05.02.03.22.12 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 02 May 2017 03:22:14 -0700 (PDT) Subject: Re: [PATCH v4 0/12] [i386] Improve 64-bit Microsoft to System V ABI pro/epilogues To: Uros Bizjak , Daniel Santos References: <49e81c0b-07a4-22df-d7c3-2439177ac7cf@pobox.com> Cc: gcc-patches , Jan Hubicka From: JonY <10walls@gmail.com> Message-ID: Date: Tue, 02 May 2017 10:25:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="812w1xjUi3mBxjafPa4WjEcICxlnctEEw" X-IsSubscribed: yes X-SW-Source: 2017-05/txt/msg00078.txt.bz2 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --812w1xjUi3mBxjafPa4WjEcICxlnctEEw Content-Type: multipart/mixed; boundary="TbcrUXvs6OTiNQhLKVqvGPe0H0Qb1ARIG"; protected-headers="v1" From: JonY <10walls@gmail.com> To: Uros Bizjak , Daniel Santos Cc: gcc-patches , Jan Hubicka Message-ID: Subject: Re: [PATCH v4 0/12] [i386] Improve 64-bit Microsoft to System V ABI pro/epilogues References: <49e81c0b-07a4-22df-d7c3-2439177ac7cf@pobox.com> In-Reply-To: --TbcrUXvs6OTiNQhLKVqvGPe0H0Qb1ARIG Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Content-length: 1637 On 05/01/2017 11:31 AM, Uros Bizjak wrote: > On Thu, Apr 27, 2017 at 10:04 AM, Daniel Santos = wrote: >> All of patches are concerned with 64-bit Microsoft ABI functions that ca= ll >> System V ABI function which clobbers RSI, RDI and XMM6-15 and are aimed = at >> improving performance and .text size of Wine 64. I had previously submit= ted >> these as separate patch sets, but have combined them for simplicity. (Do= es >> this make the ChangeLogs too big? Please let me know if you want me to b= reak >> these back apart.) Below are the included patchsets and a summary of cha= nges >> since the previous post(s): >=20 > Well, the ChangeLog is acceptable. >=20 > I have comments on how new RTX patterns are generated and checked > (patches 9/12 and 11/12). Other patches look good to me, so after > issues with 9/12 and 11/12 are resolved, I think the patch set is > ready to go. >=20 > After the above issue is addressed, I propose to move forward by > committing the patchset, and resolve any possible issues later. There > are just too many code paths in the stack frame construction and > teardown to notice all possible interactions between new and old code. > It looks that existing code won't be affected without activating new > option, so we can be a bit less cautious with the patchset. An > important part is thus a comprehensive added test suite, which seems > to pass. >=20 > I also assume that Cygwin and MinGW people agree with the patch and > the functionality itself. >=20 > Uros. >=20 Cygwin and MinGW does not use SysV/MS transitions directly in their own code, changes should be OK. --TbcrUXvs6OTiNQhLKVqvGPe0H0Qb1ARIG-- --812w1xjUi3mBxjafPa4WjEcICxlnctEEw Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" Content-length: 858 -----BEGIN PGP SIGNATURE----- iQJGBAEBCAAwFiEE5QrdnbBX9Ppk4bbPcTtf4pwUXUUFAlkIXa8SHDEwd2FsbHNA Z21haWwuY29tAAoJEHE7X+KcFF1FV3QP/inWQGRkBD4yyDrzDnJvZV6qDSrIxLB6 HNwTnhQGMrsVAbPC/jPWkshjmbdbiMYw0iZFZqvzCEXVp/4tkRXektKhiBOg1WrW hVRSCAxTVx8gJEFMGSRebMLIbqfHuldBBjksBNenUl+dbzn5q4ghRn5vcs5kRXPr MIzRELjnspSJstlYb7T3xdJJYddzor2QG4FKb/k0xqgf+tocluwL6Z7bX8Yf+uYK nsEEnWdYPuhbMnnEiJa1FCXvoZ15EXJXK0+7v5DREJwifhbqQzwpgnUoGANVIU2X PuuCpFanYXbmbZA0wffQ0cTAJvsNjMcXjRPaoxqWKp+cYGvKjrJb0HIAX1gnXnf2 ciUSu52lZszIkRyXy2IithS0qycjx96djhT7eO2MzYUXprj+A6Dvt9T8M10szsns bzr5hQWDdA8y55wqU7jehbX+WO6cvQwyJDrOfjYOsKffQEgXc6uxyzJc17DXHUWd AdsUUK4lR168Tc/Z6UpGzXpMwwpE/0iQcr8MyDy0sYhFGgexOkp1S0G61f+nffyG RvryDHKdFw6XoMBPDCMI1oMFOlK2up6Q4FCoIQ5cPz5hKKoRH6LRuJnCxCBJbaiE ZtmArGEs820/2IiII6UPpIkKP85Rf1ArJQVOu1lg9P3zrChYxJh3CdtW2NArJqUb Cl1NW1PaQF4C =KA5s -----END PGP SIGNATURE----- --812w1xjUi3mBxjafPa4WjEcICxlnctEEw--