On Fri, Jul 24, 2015 at 10:53 AM, Alexandre Oliva <aol...@redhat.com> wrote:
> On Jul 24, 2015, "H.J. Lu" <hongjiu...@intel.com> wrote:
>
>> Static chain returned from get_rtl_for_parm_ssa_default_def may not
>> have Pmode. This patch converts static chain to Pmode if needed.
>> Tested on Linux/x86-64. OK for trunk?
>
> Yeah, this looks good to me, thanks. Even the static chain parm_decl
> undergoes promotion through promote_decl_mode. I'd have placed the test
> and convert_to_mode call as an else to the if (!local), though. Ok with
> that change too.
>
I tried it and it doesn't work due to
mark_reg_pointer (local, TYPE_ALIGN (TREE_TYPE (TREE_TYPE (parm))));
mark_reg_pointer won't take a SUBREG. That is why I placed it after
mark_reg_pointer . Is my orginal patch OK?
--
H.J.
