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: [RFA][PATCH] Isolate erroneous paths optimization

From: Marc Glisse <marc dot glisse at inria dot fr>
To: Ian Lance Taylor <iant at google dot com>
Cc: Jeff Law <law at redhat dot com>, Richard Biener <richard dot guenther at gmail dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>
Date: Wed, 6 Nov 2013 07:50:25 +0100 (CET)
Subject: Re: [RFA][PATCH] Isolate erroneous paths optimization
Authentication-results: sourceware.org; auth=none
References: <5271F493 dot 4020308 at redhat dot com> <CAFiYyc2tFbaTq4xr8xZA5wb_m9pB4+VA-9MkmNDXTVkP2gQdBA at mail dot gmail dot com> <52785087 dot 3060908 at redhat dot com> <CAKOQZ8x5FAnYzkepikoJSpX9467H0Xx_28SO3abGu6J+PSpNEQ at mail dot gmail dot com>
Reply-to: gcc-patches at gcc dot gnu dot org

On Tue, 5 Nov 2013, Ian Lance Taylor wrote:

This patch actually breaks the Go testsuite.  In Go dereferencing a
nil pointer is well-defined: it causes panic that can be caught.  This
breaks a test for that functionality by changing the panic to a
builtin_trap.

That's not a big deal; I'll just disable this optimization in the Go
frontend.

Shouldn't go use -fno-delete-null-pointer-checks by default then? Thatshould disable this optimization and others that rely on the same idea.


--
Marc Glisse

Follow-Ups:
- Re: [RFA][PATCH] Isolate erroneous paths optimization
  - From: Ian Lance Taylor

References:
- Re: [RFA][PATCH] Isolate erroneous paths optimization
  - From: Richard Biener
- Re: [RFA][PATCH] Isolate erroneous paths optimization
  - From: Jeff Law
- Re: [RFA][PATCH] Isolate erroneous paths optimization
  - From: Ian Lance Taylor

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