[libc] Incorrect architecture guarding macros in x86 sigsetjmp.cpp

[bojle]

This has the following structure of code inclusion guards:

#if __i386__
// i386 implementation
#endif 
// x86-64 implementation

Whereas it should really be:

#if __i386__
// i386 implementation
#else 
// x86-64 implementation
#endif 

setjmp.cpp does this correctly.

Is this intended? Or is it an issue? I can make a quick fix if this is a bug.

@SchrodingerZhu @michaelrj-google

[llvmbot]

@llvm/issue-subscribers-libc

Author: Shreeyash Pandey (bojle)

[This](https://github.com/llvm/llvm-project/blob/main/libc/src/setjmp/x86_64/sigsetjmp.cpp) has the following structure of code inclusion guards:

#if __i386__
// i386 implementation
#endif 
// x86-64 implementation

Whereas it should really be:

#if __i386__
// i386 implementation
#else 
// x86-64 implementation
#endif 

setjmp.cpp does this correctly.

Is this intended? Or is it an issue? I can make a quick fix if this is a bug.

@SchrodingerZhu @michaelrj-google

[michaelrj-google]

Looks unintentional, probably hasn't been an issue since we don't have proper 32 bit testing. A quick fix would be great