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,AIX] Nil check size threshold control for AIX

From: Ian Lance Taylor <iant at golang dot org>
To: "REIX, Tony" <tony dot reix at atos dot net>
Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, "dje dot gcc at gmail dot com" <dje dot gcc at gmail dot com>, "thanm at google dot com" <thanm at google dot com>
Date: Tue, 12 Dec 2017 09:30:16 -0800
Subject: Re: [PATCH,AIX] Nil check size threshold control for AIX
Authentication-results: sourceware.org; auth=none
References: <B37989F2852398498001550C29155BE51791C4A3@FRCRPVV9EX3MSX.ww931.my-it-solutions.net>

On Tue, Dec 12, 2017 at 12:59 AM, REIX, Tony <tony.reix@atos.net> wrote:
> Description:
>  * This patch tells the Go frontend to insert explicit guards (check for nil -> panic) for AIX since relying on a fault does not work on AIX for page 0.
>
> Tests:
>  * AIX: Build: SUCCESS
>    - build made by means of gmake.
>
> ChangeLog:
>   * go-lang.c (go_langhook_init): Handle AIX case for nil_check_size_threshold.

Thanks.  I changed it to avoid #ifdef in the middle of the function,
and committed as follows.

Ian

2017-12-12  Tony Reix  <tony.reix@atos.net>
    Ian Lance Taylor  <iant@golang.org>

* go-lang.c (TARGET_AIX): Define if not defined.
(go_langhook_init): Set nil_check_size_threshold to -1 on AIX.

Attachment: patch.txt
Description: Text document

References:
- [PATCH,AIX] Nil check size threshold control for AIX
  - From: REIX, Tony

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