This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, CHKP, i386] Fix retval generated for instrumented calls returning values on multiple registers
- From: Jan Hubicka <hubicka at ucw dot cz>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: Jan Hubicka <hubicka at ucw dot cz>, Ilya Enkovich <enkovich dot gnu at gmail dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 2 Apr 2015 23:14:51 +0200
- Subject: Re: [PATCH, CHKP, i386] Fix retval generated for instrumented calls returning values on multiple registers
- Authentication-results: sourceware.org; auth=none
- References: <20150402150759 dot GA6244 at msticlxl57 dot ims dot intel dot com> <20150402201706 dot GG21276 at atrey dot karlin dot mff dot cuni dot cz> <CAFULd4Z-hYWBG59-yXwPcrXUnc50CZru_SVfroHsK2mBJVStpw at mail dot gmail dot com>
> On Thu, Apr 2, 2015 at 10:17 PM, Jan Hubicka <hubicka@ucw.cz> wrote:
>
> >> rtx b0 = gen_rtx_REG (BND64mode, FIRST_BND_REG);
> >> rtx b1 = gen_rtx_REG (BND64mode, FIRST_BND_REG + 1);
> >> - retval = gen_rtx_PARALLEL (VOIDmode, gen_rtvec (3, retval, b0, b1));
> >> - chkp_put_regs_to_expr_list (retval);
> >> + if (GET_CODE (retval) == PARALLEL)
> >> + {
> >> + b0 = gen_rtx_EXPR_LIST (VOIDmode, b0, const0_rtx);
> >> + b1 = gen_rtx_EXPR_LIST (VOIDmode, b1, const0_rtx);
> >> + rtx par = gen_rtx_PARALLEL (VOIDmode, gen_rtvec (2, b0, b1));
> >> + retval = chkp_join_splitted_slot (retval, par);
> >
> > I do not understand this code. Isn't it just droping the original return value
> > constructing prallelcontaining only the BND_REGs?
>
> Please see chkp_join_splitted_slot that merges two parallels.
I see, OK then :)
Honza
>
> Uros.