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: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)


On Wed, Mar 23, 2011 at 11:20:24AM -0700, Richard Henderson wrote:
> On 03/23/2011 11:04 AM, Jakub Jelinek wrote:
> > For several targets that would grow the context significantly, do you think
> > we really have to do that for all targets rather than just for the
> > sizeof (_Unwind_Word) > sizeof (void *) ones?  The accessors are wrapped
> > in inline functions anyway.
> 
> Given that ia64 doesn't use this file, we're talking about 512 bytes for
> most 64-bit targets.  I suppose that's 512 bytes we needn't allocate for
> others, but...
> 
> The problem is that we suddenly have a version mismatch.  Suppose we 
> conditionally add this, then next decade we need to add some other field
> unconditionally.  Now determining what version X means is non-trivial.
> 
> What targets are you worried most about?

E.g. for ppc64 that's 1160 bytes, mmix 2104 bytes, ...
But if you think the 6 or so conditionals would be too much burden, just
ignore me.

	Jakub


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