From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 47669 invoked by alias); 16 May 2017 14:17:13 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 47652 invoked by uid 89); 16 May 2017 14:17:12 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.4 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,RCVD_IN_SORBS_SPAM,SPF_PASS autolearn=no version=3.3.2 spammy=U*tony.reix, Hx-languages-length:975, D*atos.net, tony.reix@atos.net X-HELO: mail-qt0-f171.google.com Received: from mail-qt0-f171.google.com (HELO mail-qt0-f171.google.com) (209.85.216.171) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 16 May 2017 14:17:11 +0000 Received: by mail-qt0-f171.google.com with SMTP id f55so79533784qta.3 for ; Tue, 16 May 2017 07:17:14 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=kquq8ylN3t4A0uSllBGfeVcsdVFaqP5ReN2dDU9dmHQ=; b=JQkxzoohl1DJPArfcccvqXQSvdnqM0a0bkfWpP6zHiUzYUeQ6t6/XJoKCD9fIaer3F SS6J44o36JAbHfKeOWuTyg3M1DJaJrGw4/wT7V3pkd5gx+fLoN79BUwIJTtzRLEe6Ys3 n48LI0Q33dimG0+8Vb9eCYZ6O/wu3XbiL1TV2yHcg0ebKj3HJNv2JrW4RfhFwpJ1bG0Y N+w/kLeo5jYy/nAFaBTK33X6YBS5oXIlRW0OPSHQOsyfd91wX1HEpoLjD/ZNl41730kP gjgkhqXHInQEYxHtZoCoM7DIXhFjVKlS+5DwMJziz8h927mssnwbh78Y7PqBbY8nqUtN 6dng== X-Gm-Message-State: AODbwcB8qTx7ytE5f5gd6L258Lw1V8Gh0n18fuzXVZ/EBNTSEW+L9vkt jMDCk4EVyTqXpJhPYrnOrsN4zzKeig== X-Received: by 10.200.34.98 with SMTP id p31mr10301408qtp.2.1494944232843; Tue, 16 May 2017 07:17:12 -0700 (PDT) MIME-Version: 1.0 Received: by 10.12.185.172 with HTTP; Tue, 16 May 2017 07:17:12 -0700 (PDT) In-Reply-To: References: From: David Edelsohn Date: Tue, 16 May 2017 14:27:00 -0000 Message-ID: Subject: Re: [PATCH,AIX] Enable Stack Unwinding on AIX To: "REIX, Tony" Cc: GCC Patches , "SARTER, MATTHIEU (ext)" Content-Type: text/plain; charset="UTF-8" X-SW-Source: 2017-05/txt/msg01285.txt.bz2 On Tue, May 16, 2017 at 9:50 AM, REIX, Tony wrote: > Tests: > > The change has been tested in 32bit and 64bit on AIX 6.1, 7.1 & 7.2 by using the libbacktrace (with XCOFF support) within a signal handler context (we used a specific back.c program). > And it has been tested with Go tests on AIX for sure (recover tests on SIGSEGV do not work without this change). > > ChangeLog: > > * config/rs6000/aix-unwind.h (ppc_aix_fallback_frame_state): Add 64 bit support for AIX 6.1 and 7.X and 32 bit support for AIX 7.2. If ppc_aix_fallback_frame_state is used for both 32 bit and 64 bit, there is no need to define it twice. Just move the MD_FALLBACK_FRAME_STATE_FOR macro definition outside the #ifdef. This also needs a separate ChangeLog comment. The function that you modified is "ucontext_for". Please use that name in the ChangeLog entry. Will you need someone to apply the patch for you? Thanks, David