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: [patch] alias.c: Replace "return NULL_RTX" with "return 0".

From: "Kaveh R. Ghazi" <ghazi at caip dot rutgers dot edu>
To: jason at redhat dot com
Cc: fjh at cs dot mu dot oz dot au, gcc-patches at gcc dot gnu dot org, kazu at cs dot umass dot edu, zack at codesourcery dot com
Date: Thu, 12 Jun 2003 17:32:52 -0400 (EDT)
Subject: Re: [patch] alias.c: Replace "return NULL_RTX" with "return 0".
References: <20030612.141353.55502198.kazu@cs.umass.edu><20030612184112.GD8506@mars.cs.mu.oz.au><87of133zoz.fsf@egil.codesourcery.com><20030612185541.GE8506@mars.cs.mu.oz.au><wvlel1zcb5t.fsf@prospero.boston.redhat.com><87k7br3vem.fsf@egil.codesourcery.com> <wvl65nbcabr.fsf@prospero.boston.redhat.com>

 > > I won't argue readability (though I disagree) but NULL is *NOT* more
 > > reliable in the presence of varargs; there is no guarantee that NULL
 > > doesn't expand to "0".  In the presence of varargs one has no choice
 > > but to write an explicit cast.
 > 
 > A system that defines NULL to 0 where that is a different size from
 > (void*)0 is broken.  Does such brokenness actually exist?
 > Jason

I doubt it.  FWIW we've been using plain NULL as a terminator for the
variadic `concat' function for years and I'm unaware of any
complaints.

--
Kaveh R. Ghazi			ghazi@caip.rutgers.edu

References:
- [patch] alias.c: Replace "return NULL_RTX" with "return 0".
  - From: Kazu Hirata
- Re: [patch] alias.c: Replace "return NULL_RTX" with "return 0".
  - From: Fergus Henderson
- Re: [patch] alias.c: Replace "return NULL_RTX" with "return 0".
  - From: Zack Weinberg
- Re: [patch] alias.c: Replace "return NULL_RTX" with "return 0".
  - From: Fergus Henderson
- Re: [patch] alias.c: Replace "return NULL_RTX" with "return 0".
  - From: Jason Merrill
- Re: [patch] alias.c: Replace "return NULL_RTX" with "return 0".
  - From: Zack Weinberg
- Re: [patch] alias.c: Replace "return NULL_RTX" with "return 0".
  - From: Jason Merrill

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