[PATCH, AARCH64] Enable fuse-caller-save for AARCH64
Tom de Vries
Tom_deVries@mentor.com
Thu Jun 19 12:26:00 GMT 2014
On 19-06-14 05:21, Richard Henderson wrote:
> On 06/01/2014 03:00 AM, Tom de Vries wrote:
>> +/* Emit call insn with PAT and do aarch64-specific handling. */
>> +
>> +bool
>> +aarch64_emit_call_insn (rtx pat)
>> +{
>> + rtx insn = emit_call_insn (pat);
>> +
>> + rtx *fusage = &CALL_INSN_FUNCTION_USAGE (insn);
>> + clobber_reg (fusage, gen_rtx_REG (word_mode, IP0_REGNUM));
>> + clobber_reg (fusage, gen_rtx_REG (word_mode, IP1_REGNUM));
>> +}
>> +
>
> Which can't have been bootstrapped, since this has no return stmt.
> Why the bool return type anyway? Nothing appears to use it.
>
Richard,
Indeed, the return type should be void, this patch fixes that.
I have no setup to bootstrap this on aarch64. I've build an aarch64 compiler and
ran the gcc.target/aarch64/fuse-caller-save.c testcase.
Committed as obvious.
Thanks,
- Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0005-Fix-aarch64_emit_call_insn-return-type.patch
Type: text/x-patch
Size: 1226 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20140619/13ac6820/attachment.bin>
More information about the Gcc-patches
mailing list