[PATCH] PPC enable Altivec for setjmp/longjmp part 1 of 5

Ulrich Drepper drepper@redhat.com
Fri Feb 13 23:26:00 GMT 2004


Steve Munroe wrote:

> Thought I needed features.h to get the __THROW definitions but I can use 

You don't need __THROW in internal headers.  __THROW is only for C++.


> sys/cdefs.h. I missed the __BEGIN_DECLS in my last pass. Gone now. The 
> include protection is just habit and forgot to change the name. How about 
> I change this to:
> 
> +#ifndef                 __NOVMX_SETJMP_H
> +#define                 __NOVMX_SETJMP_H                1
> +
> +#include <sys/cdefs.h>
> +#include <bits/wordsize.h>

No need for the sys/cdefs.h include either.


> NOVMX needes its own definition as the offset to __mask_was_saved and 
> __saved_mask will differ because the size of __jmpbuf changes. Is it 
> clearer if I change the name?

Yes, please do that.

> +/* copied from setjmp/setjmp.h, powerpc/bits/setjmp.h and modified
> +   appropriately to keep backward compatible with setjmp without
> +   AltiVec/VMX support.  This file is not exported and the interfaces
> +   are private to libc.  */

That's fine.  This together with removing the other bits only used in
public headers will make it clear that this is an internal header.

-- 
➧ Ulrich Drepper ➧ Red Hat, Inc. ➧ 444 Castro St ➧ Mountain View, CA ❖



More information about the Libc-alpha mailing list