This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Fix UBSan builtin types

From: Jeff Law <law at redhat dot com>
To: Yury Gribov <y dot gribov at samsung dot com>, Marek Polacek <polacek at redhat dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Jakub Jelinek <jakub at redhat dot com>, Vyacheslav Tyrtov <v dot tyrtov at samsung dot com>
Date: Thu, 20 Aug 2015 11:28:50 -0600
Subject: Re: [PATCH] Fix UBSan builtin types
Authentication-results: sourceware.org; auth=none
References: <55D6037F dot 7030704 at samsung dot com>

On 08/20/2015 10:42 AM, Yury Gribov wrote:

Hi all,

GCC builtins BUILT_IN_UBSAN_HANDLE_NONNULL_ARG and
BUILT_IN_UBSAN_HANDLE_NONNULL_ARG_ABORT were using
BT_FN_VOID_PTR_PTRMODE whereas they are really BT_FN_VOID_PTR:
   void __ubsan::__ubsan_handle_nonnull_return(NonNullReturnData *Data)

The patch fixes it.  I only tested ubsan.exp (I doubt that bootstrap +
full testsuite will add anything to this).

Ok for trunk?

Best regards,
Yury Gribov

fix-ubsan-builtin-types-1.patch


commit d4747c9c7f78789ec7119fce07cd4526c4168ee0
Author: Yury Gribov<y.gribov@samsung.com>
Date:   Thu Aug 20 19:10:30 2015 +0300

     2015-08-20  Yury Gribov<y.gribov@samsung.com>

     gcc/
     	* sanitizer.def (BUILT_IN_UBSAN_HANDLE_NONNULL_ARG,
     	BUILT_IN_UBSAN_HANDLE_NONNULL_ARG): Fix builtin types.


OK.
jeff

References:
- [PATCH] Fix UBSan builtin types
  - From: Yury Gribov

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]