This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 2/5] rs6000: Fix TARGET_PROMOTE_FUNCTION_MODE
- From: David Edelsohn <dje dot gcc at gmail dot com>
- To: Segher Boessenkool <segher at kernel dot crashing dot org>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 14 Jan 2015 23:37:37 -0500
- Subject: Re: [PATCH 2/5] rs6000: Fix TARGET_PROMOTE_FUNCTION_MODE
- Authentication-results: sourceware.org; auth=none
- References: <a0366c6b956020e6762b6b78a73b20aba5ec1408 dot 1421282067 dot git dot segher at kernel dot crashing dot org> <072fdf6eba7388145b5a437f595fa24d84bb21dc dot 1421282067 dot git dot segher at kernel dot crashing dot org>
On Wed, Jan 14, 2015 at 8:14 PM, Segher Boessenkool
<segher@kernel.crashing.org> wrote:
> As the existing comment explains, we should always promote function
> arguments and return values. However, notwithstanding its name,
> default_promote_function_mode_always_promote does not always promote.
> Importantly, it does not for libcalls. This makes ftrapv-[12].c fail
> with 64-bit ABIs.
>
> This patch introduces an rs6000_promote_function_mode that _does_
> always promote, fixing this.
>
> Tested as usual (c,c++,fortran,ada; -m32,-m32/-mpowerpc64,-m64,-m64/-mlra).
> Is this okay for mainline?
>
>
> Segher
>
>
> 2015-01-14 Segher Boessenkool <segher@kernel.crashing.org>
>
> gcc/
> * config/rs6000/rs6000.c (TARGET_PROMOTE_FUNCTION_MODE): Implement
> as rs6000_promote_function_mode. Move comment to there.
> (rs6000_promote_function_mode): New function.
Okay.
Thanks, David