Hi, Thanks. I added all fixes to the patch, bootstrapped/regtested it on Linux/x86_64. Linux/i686 in progress. Ok for trunk if testing passes successfully?
Julia
On Wed, Sep 30, 2015 at 5:50 AM, H.J. Lu <hjl.to...@gmail.com> wrote:
> On Tue, Sep 29, 2015 at 5:02 PM, H.J. Lu <hjl.to...@gmail.com> wrote:
>> On Tue, Sep 29, 2015 at 4:53 PM, Mike Stump <mikest...@comcast.net> wrote:
>>> On Sep 29, 2015, at 3:10 PM, H.J. Lu <hjl.to...@gmail.com> wrote:
>>>> On Tue, Sep 29, 2015 at 2:23 PM, Mike Stump <mikest...@comcast.net> wrote:
>>>>> On Sep 29, 2015, at 1:59 PM, H.J. Lu <hjl.to...@gmail.com> wrote:
>>>>>> commit f3a6675a8d69d810d2cad0c090a762094a0a8622
>>>>>> Author: H.J. Lu <hjl.to...@gmail.com>
>>>>>> Date: Tue Sep 29 13:47:18 2015 -0700
>>>>>>
>>>>>> Define EPILOGUE_USES in i386 so that all preserved registers are used
>>>>>> by the epilogue of interrupt handler. Don't explicitly mark BP and SP
>>>>>> registers as used since they are always used in epilogue.
>>>>>>
>>>>>> Please take a look.
>>>>>
>>>>> Oh, too bad you didn’t copy it here. The easiest thing to blow is the
>>>>> addition of reload_completed && on the condition
>>>
>>>
>>>> static bool
>>>> ix86_save_reg (unsigned int regno, bool maybe_eh_return)
>>>> {
>>>> /* In interrupt handler, we don't preserve MMX and x87 registers
>>>> which aren't supported when saving and restoring registers. No
>>>> need to preserve callee-saved registers unless they are modified.
>>>> We also preserve all caller-saved registers if a function call
>>>> is made in interrupt handler since the called function may change
>>>> them. Don't explicitly save BP and SP registers since they are
>>>> always preserved. */
>>>> if (cfun->machine->is_interrupt)
>>>> return ((df_regs_ever_live_p (regno)
>>>> || (call_used_regs[regno] && cfun->machine->make_calls))
>>>> && !fixed_regs[regno]
>>>> && !STACK_REGNO_P (regno)
>>>> && !MMX_REGNO_P (regno)
>>>> && regno != BP_REG
>>>> && regno != SP_REG
>>>> && (regno <= ST7_REG || regno >= XMM0_REG));
>>>>
>>>> Is this sufficient?
>>>
>>> I see no string "reload_completed &&”. Either, you need it here, or, you
>>> need it in the caller.
>>
>> Do you have a testcase to show its impact?
>
> I checked this patch into hjl/interrupt/master branch.
>
> Thanks.
>
>
> --
> H.J.
> --
> commit 8c75718a0f590cb02e7cb88e36fe12e90db62bc1
> Author: H.J. Lu <hjl.to...@gmail.com>
> Date: Tue Sep 29 19:42:16 2015 -0700
>
> Preserve registers in interrupt handler after reload
>
> * config/i386/i386.c (ix86_save_reg): Preserve callee-saved and
> caller-saved registers in interrupt handler only after reload.
>
> diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
> index 6b14471..d5c7e07 100644
> --- a/gcc/config/i386/i386.c
> +++ b/gcc/config/i386/i386.c
> @@ -11119,7 +11119,7 @@ ix86_save_reg (unsigned int regno, bool
> maybe_eh_return)
> is made in interrupt handler since the called function may change
> them. Don't explicitly save BP and SP registers since they are
> always preserved. */
> - if (cfun->machine->is_interrupt)
> + if (cfun->machine->is_interrupt && reload_completed)
> return ((df_regs_ever_live_p (regno)
> || (call_used_regs[regno]
> && cfun->machine->call_with_caller_saved_registers))
interrupt_patch
Description: Binary data
