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? That should disable this optimization and others that rely on the same idea.
-- Marc Glisse