This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH GCC]Correct cand_chain and stmt_cand_map for copy/cast
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: "bin.cheng" <bin dot cheng at linux dot alibaba dot com>, Bill Schmidt <wschmidt at linux dot ibm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 15 May 2019 13:09:44 +0200
- Subject: Re: [PATCH GCC]Correct cand_chain and stmt_cand_map for copy/cast
- References: <b1d78d6d-1008-4005-a172-16deac45b12c.bin.cheng@linux.alibaba.com>
On Wed, May 15, 2019 at 7:54 AM bin.cheng <bin.cheng@linux.alibaba.com> wrote:
>
> Hi,
> I noticed that cand_chain (first_interp/next_interp) is not maintained correctly
> in slsr_process_copy/slsr_process_cast (now slsr_process_copycast). This one
> fixes the issue, as well as records the "first" cand in stmt_cand_map.
>
> Hi Bill, is this correct or I misunderstood the code? Bootstrap and test on x86_64.
Looks good to me, still waiting for Bills feedback (now CCed).
Richard.
> Thanks,
> bin
>
> 2019-05-15 Bin Cheng <bin.cheng@linux.alibaba.com>
>
> * gimple-ssa-strength-reduction.c (slsr_process_copycast): Record
> information about next_interp and the first cand.