This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [flag-types.h] don't assume 32-bit ints
- From: Jakub Jelinek <jakub at redhat dot com>
- To: DJ Delorie <dj at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 16 Oct 2014 08:56:18 +0200
- Subject: Re: [flag-types.h] don't assume 32-bit ints
- Authentication-results: sourceware.org; auth=none
- References: <201410152216 dot s9FMGjq3012196 at greed dot delorie dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Wed, Oct 15, 2014 at 06:16:45PM -0400, DJ Delorie wrote:
>
> Since flag-types.h is used in target code, it needs to be -Wall-safe
> for targets with 16-bit "int". OK?
With a ChangeLog entry yes.
> --- gcc/flag-types.h (revision 216287)
> +++ gcc/flag-types.h (working copy)
> @@ -229,17 +229,17 @@ enum sanitize_code {
> SANITIZE_RETURN = 1 << 10,
> SANITIZE_SI_OVERFLOW = 1 << 11,
> SANITIZE_BOOL = 1 << 12,
> SANITIZE_ENUM = 1 << 13,
> SANITIZE_FLOAT_DIVIDE = 1 << 14,
> SANITIZE_FLOAT_CAST = 1 << 15,
> - SANITIZE_BOUNDS = 1 << 16,
> - SANITIZE_ALIGNMENT = 1 << 17,
> - SANITIZE_NONNULL_ATTRIBUTE = 1 << 18,
> - SANITIZE_RETURNS_NONNULL_ATTRIBUTE = 1 << 19,
> - SANITIZE_OBJECT_SIZE = 1 << 20,
> + SANITIZE_BOUNDS = 1UL << 16,
> + SANITIZE_ALIGNMENT = 1UL << 17,
> + SANITIZE_NONNULL_ATTRIBUTE = 1UL << 18,
> + SANITIZE_RETURNS_NONNULL_ATTRIBUTE = 1UL << 19,
> + SANITIZE_OBJECT_SIZE = 1UL << 20,
> SANITIZE_UNDEFINED = SANITIZE_SHIFT | SANITIZE_DIVIDE | SANITIZE_UNREACHABLE
> | SANITIZE_VLA | SANITIZE_NULL | SANITIZE_RETURN
> | SANITIZE_SI_OVERFLOW | SANITIZE_BOOL | SANITIZE_ENUM
> | SANITIZE_BOUNDS | SANITIZE_ALIGNMENT
> | SANITIZE_NONNULL_ATTRIBUTE
> | SANITIZE_RETURNS_NONNULL_ATTRIBUTE
Jakub