This is an archive of the discontinued LLVM Phabricator instance.

libc/include/llvm-libc-types/fenv_t.h
27	I would highly recommend never using a typedef to a base type (or other existing type) like this. It's much better to define a struct with a single member, as you can see done for the ARM case above. This ensures that the API contract enforces that `fenv_t` and `fenv_t` values are passed, rather than `unsigned int` or `unsigned int`. On x86-64, ARM, and RISC-V, the calling convention for a struct like this is the same as for the bare integer type.

Revision Contents

Path

Size

libc/

include/

llvm-libc-macros/

linux/

3 lines

llvm-libc-types/

2 lines

src/

__support/

FPUtil/

4 lines

	Show First 20 Lines • Show All 76 Lines • ▼ Show 20 Lines
	#define SS_DISABLE 0x2			#define SS_DISABLE 0x2

	#ifdef __x86_64__			#ifdef __x86_64__
	#define MINSIGSTKSZ 2048			#define MINSIGSTKSZ 2048
	#define SIGSTKSZ 8192			#define SIGSTKSZ 8192
	#elif defined(__aarch64__)			#elif defined(__aarch64__)
	#define MINSIGSTKSZ 5120			#define MINSIGSTKSZ 5120
	#define SIGSTKSZ 16384			#define SIGSTKSZ 16384
				#elif defined(__riscv)
				#define MINSIGSTKSZ 2048
				#define SIGSTKSZ 8192
	#else			#else
	#error "Signal stack sizes not defined for your platform."			#error "Signal stack sizes not defined for your platform."
	#endif			#endif

	#define SIG_DFL ((__sighandler_t)0)			#define SIG_DFL ((__sighandler_t)0)
	#define SIG_IGN ((__sighandler_t)1)			#define SIG_IGN ((__sighandler_t)1)
	#define SIG_ERR ((__sighandler_t)-1)			#define SIG_ERR ((__sighandler_t)-1)

	#endif // __LLVM_LIBC_MACROS_LINUX_SIGNUM_MACROS_H			#endif // __LLVM_LIBC_MACROS_LINUX_SIGNUM_MACROS_H

	Show All 17 Lines
	typedef struct {			typedef struct {
	unsigned char __x86_status[28];			unsigned char __x86_status[28];
	unsigned char __mxcsr[4];			unsigned char __mxcsr[4];
	} fenv_t;			} fenv_t;
	#elif defined(__arm__) \|\| defined(_M_ARM)			#elif defined(__arm__) \|\| defined(_M_ARM)
	typedef struct {			typedef struct {
	unsigned int __fpscr;			unsigned int __fpscr;
	} fenv_t;			} fenv_t;
				#elif defined(__riscv)
				typedef unsigned int fenv_t;
				mcgrathrUnsubmitted Not Done Reply Inline Actions I would highly recommend never using a typedef to a base type (or other existing type) like this. It's much better to define a struct with a single member, as you can see done for the ARM case above. This ensures that the API contract enforces that `fenv_t` and `fenv_t` values are passed, rather than `unsigned int` or `unsigned int`. On x86-64, ARM, and RISC-V, the calling convention for a struct like this is the same as for the bare integer type. mcgrathr: I would highly recommend never using a typedef to a base type (or other existing type) like…
	#else			#else
	#error "fenv_t not defined for your platform"			#error "fenv_t not defined for your platform"
				abrachetUnsubmitted Done Reply Inline Actions I think the __riscv check should be above this abrachet: I think the __riscv check should be above this
	#endif			#endif

	#endif // __LLVM_LIBC_TYPES_FENV_T_H__			#endif // __LLVM_LIBC_TYPES_FENV_T_H__

	Show All 40 Lines
	LIBC_INLINE int clear_except(int) { return 0; }			LIBC_INLINE int clear_except(int) { return 0; }

	LIBC_INLINE int test_except(int) { return 0; }			LIBC_INLINE int test_except(int) { return 0; }

	LIBC_INLINE int set_except(int) { return 0; }			LIBC_INLINE int set_except(int) { return 0; }

	LIBC_INLINE int raise_except(int) { return 0; }			LIBC_INLINE int raise_except(int) { return 0; }

				LIBC_INLINE int enable_except(int) { return 0; }

				LIBC_INLINE int disable_except(int) { return 0; }

	LIBC_INLINE int get_round() { return FE_TONEAREST; }			LIBC_INLINE int get_round() { return FE_TONEAREST; }

	LIBC_INLINE int set_round(int) { return 0; }			LIBC_INLINE int set_round(int) { return 0; }

	LIBC_INLINE int get_env(fenv_t *) { return 0; }			LIBC_INLINE int get_env(fenv_t *) { return 0; }

	LIBC_INLINE int set_env(const fenv_t *) { return 0; }			LIBC_INLINE int set_env(const fenv_t *) { return 0; }

	Show All 25 Lines