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 0/3] Simplify the simple_return handling

From: Bernd Schmidt <bschmidt at redhat dot com>
To: Segher Boessenkool <segher at kernel dot crashing dot org>
Cc: gcc-patches at gcc dot gnu dot org
Date: Tue, 3 May 2016 15:46:16 +0200
Subject: Re: [PATCH 0/3] Simplify the simple_return handling
Authentication-results: sourceware.org; auth=none
References: <cover dot 1462256244 dot git dot segher at kernel dot crashing dot org> <57289F56 dot 70506 at redhat dot com> <20160503133128 dot GB20260 at gate dot crashing dot org>

On 05/03/2016 03:31 PM, Segher Boessenkool wrote:

If not, are you set up to test arm in
any way? Ideally you'd want to run that as well.


Good plan.  There is arm64 in the cfarm; I'll see if I can build 32-bit
as well.

Simulator testing should work, but if ppc exercises this code there'sprobably no need to do it.

I think all three patches look ok, except all your fprintf calls are
misindented.


In patch 2, bah I thought I fixed that.  I want to factor that huge function
anyway, reduce the indent a lot, okay to fix it then?  :-)


Let's keep anything that's committed clean.


Bernd

References:
- [PATCH 0/3] Simplify the simple_return handling
  - From: Segher Boessenkool
- Re: [PATCH 0/3] Simplify the simple_return handling
  - From: Bernd Schmidt
- Re: [PATCH 0/3] Simplify the simple_return handling
  - From: Segher Boessenkool

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