On Wed, Dec 19, 2018 at 10:20:50PM +0100, Tobias Hansen wrote:
> +#ifdef __GNUC__
> +#define MarkStackBottomBags() \
> + _MarkStackBottomBags(__builtin_frame_address(0));
> +/*
> +#else
> + * TODO: Detect the best stack frame detection technique at configure time
> + *
> +#define MarkStackBottomBags() \
> + register void* rbp asm("rbp"); \
> + _MarkStackBottomBags(rbp);
> +*/
> +#endif
This do not seem particularly portable...
Why should rbp be a valid asm instruction ?
Nowadays about any compiler define __GNUC__, whether or not they implement
__builtin_frame_address.
And __builtin_frame_address by itself is not particularly reliable.
Cheers,
--
Bill. <ballo...@debian.org>
Imagine a large red swirl here.
