Fix PR64876, regressions in powerpc64 Go testsuite
Alan Modra
amodra@gmail.com
Tue Feb 3 22:55:00 GMT 2015
On Tue, Feb 03, 2015 at 03:08:01PM +0100, Jakub Jelinek wrote:
> On Wed, Feb 04, 2015 at 12:27:35AM +1030, Alan Modra wrote:
> > @@ -33002,7 +33092,9 @@ rs6000_call_aix (rtx value, rtx func_desc, rtx fla
> > originally direct, the 3rd word has not been written since no
> > trampoline has been built, so we ought not to load it, lest we
> > override a static chain value. */
> > - if (!direct_call_p && TARGET_POINTERS_TO_NESTED_FUNCTIONS)
> > + if (!direct_call_p
> > + && TARGET_POINTERS_TO_NESTED_FUNCTIONS
> > + && !chain_already_loaded (crtl->emit.sequence_stack->last))
>
> Shouldn't that be !chain_already_loaded (get_last_insn_anywhere ()) ?
I considered that interface but it's not best option. We may have
already emitted insns for the topmost sequence, ie. the GEN_CALL. So
you'd need to loop over insns in chain_already_loaded. But not too
far, as that might run into a previous call.
--
Alan Modra
Australia Development Lab, IBM
More information about the Gcc-patches
mailing list