This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: calloc = malloc + memset

From: Marc Glisse <marc dot glisse at inria dot fr>
To: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Tue, 3 Jun 2014 16:00:17 +0200 (CEST)
Subject: Re: calloc = malloc + memset
Authentication-results: sourceware.org; auth=none
References: <alpine dot DEB dot 2 dot 02 dot 1402282337290 dot 27023 at stedding dot saclay dot inria dot fr> <CAFiYyc1Qhceic9pTM_nOL3yMC+tZ3t-jWetgNL-1E8KD4QSpCA at mail dot gmail dot com> <alpine dot DEB dot 2 dot 02 dot 1403231649440 dot 14825 at stedding dot saclay dot inria dot fr> <20140418112447 dot GB1817 at tucnak dot redhat dot com> <alpine dot DEB dot 2 dot 10 dot 1404181410410 dot 3692 at laptop-mg dot saclay dot inria dot fr> <CAFiYyc1J0-eEVTbL0uWqVdtVBSUcvRQzKWwf-=E1JBb8H+XTUQ at mail dot gmail dot com> <alpine dot DEB dot 2 dot 10 dot 1405171607030 dot 3642 at laptop-mg dot saclay dot inria dot fr>

Ping?

On Sat, 17 May 2014, Marc Glisse wrote:

Ping Jakub?
https://gcc.gnu.org/ml/gcc-patches/2014-04/msg01104.html

On Wed, 23 Apr 2014, Richard Biener wrote:

On Fri, Apr 18, 2014 at 8:27 PM, Marc Glisse <marc.glisse@inria.fr> wrote:

Thanks for the comments!


On Fri, 18 Apr 2014, Jakub Jelinek wrote:

The passes.def change makes me a little bit nervous, but if it works,
perhaps.



Would you prefer running the pass twice? I thought there would be less
resistance to moving the pass than duplicating it.


Indeed.  I think placing it after loops and CSE (thus what you have done)
makes sense.  strlenopt itself shouldn't enable much additional
optimizations.  But well, pass ordering is always tricky.

Didn't look at the rest of the changes, but Jakub is certainly able to
approve the patch so I leave it to him.


--
Marc Glisse

Follow-Ups:
- Re: calloc = malloc + memset
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]