Subject: Re: CVS commit: src/include
To: Jason Thorpe <thorpej@shagadelic.org>
From: Perry E. Metzger <perry@piermont.com>
List: source-changes
Date: 08/21/2006 12:47:45
Jason Thorpe <thorpej@shagadelic.org> writes:
> On Aug 21, 2006, at 7:51 AM, Perry E. Metzger wrote:
>> I'm not sure this is 100% correct. I think it would be better to do an
>> #ifdef so that __builtin_offsetof() is used only if __GNUC__, and
>> otherwise use the old one. Otherwise we can break compilation with
>> other compilers. (It might be reasonable to make that contingent on
>> gcc4...)
>
> Yes, you also need to provide the old version for lint(1).
>
> If __builtin_offsetof() is provided by GCC3, no harm in allowing it
> for that compiler.

Vigorously agreed.

Perry