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, rfc] Semantics of attribute (aligned), misscompilation of crtstuff


Daniel Jacobowitz wrote:
> On Thu, Oct 25, 2007 at 07:29:43PM -0700, Mark Mitchell wrote:
>> I think this is a documentation bug.  In particular, the manual does not
>> list "packed" under "Variable Attributes" -- but it does say, in the
>> description of the "aligned" attribute that "packed" can decrease
>> alignment.  Is that what suggested to you that the combination was
>> valid?  I think what's happened is that the "aligned" documentation is
>> shared between "Type Attributes" and "Variable Attributes"; we probably
>> need to change the "Variable Attributes" copy not to mention "packed"
>> since it doesn't make sense in this context.
> 
> I disagree that it doesn't make sense in this context.  If
> attribute((aligned)) does not specify an exact alignment, I think it
> behooves us to add some other way to do so.

I should have been clearer: by "doesn't make sense" I meant "isn't
presently supported by the compiler".  Adding a mechanism to specify to
an alignment exactly seems like a fine thing to me, but until we do, we
should fix the documentation to document the compiler we actually have,
not the one we wish we had. :-)

-- 
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713


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