From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 33964 invoked by alias); 21 Nov 2017 17:09:57 -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 33948 invoked by uid 89); 21 Nov 2017 17:09:57 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.7 required=5.0 tests=BAYES_00,KB_WAM_FROM_NAME_SINGLEWORD,SPF_HELO_PASS,T_RP_MATCHES_RCVD autolearn=no version=3.3.2 spammy=spring, silence, enhancements, HContent-Transfer-Encoding:8bit X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 21 Nov 2017 17:09:56 +0000 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id DD9F625C2F; Tue, 21 Nov 2017 17:09:54 +0000 (UTC) Received: from localhost.localdomain (ovpn-112-12.rdu2.redhat.com [10.10.112.12]) by smtp.corp.redhat.com (Postfix) with ESMTP id DE0BF6A04B; Tue, 21 Nov 2017 17:09:53 +0000 (UTC) Subject: Re: [RFC][PATCH] Extend DCE to remove unnecessary new/delete-pairs To: =?UTF-8?Q?Dominik_Inf=c3=bchr?= , GCC Patches References: <8305B5F4-2A96-4698-8C2E-3255658B5C12@theobroma-systems.com> From: Jeff Law Message-ID: <21ed3448-5c28-c43d-c437-1ca7d6250a03@redhat.com> Date: Tue, 21 Nov 2017 17:13:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <8305B5F4-2A96-4698-8C2E-3255658B5C12@theobroma-systems.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-IsSubscribed: yes X-SW-Source: 2017-11/txt/msg01928.txt.bz2 On 11/21/2017 04:14 AM, Dominik Inführ wrote: > Hi, > > this patch tries to extend tree-ssa-dce.c to remove unnecessary new/delete-pairs (it already does that for malloc/free). Clang does it too and it seems to be allowed by http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2013/n3664.html. I’ve bootstrapped/regtested on aarch64-linux and x86_64-linux. Just a note, we've transitioned into stage3 in preparation for the upcoming gcc-8 release in the spring. During stage3 we're addressing bugfixes, not further enhancements (with the exception of enhancements that were posted prior to stage1 close). So it's unlikely anyone will dig into this right now, unless there's an existing bugzilla around this missed optimization. Just wanted to let you know where things stood so you don't interpret silence as "we don't care". jeff