On Wed, May 15, 2019 at 7:54 AM bin.cheng <bin.ch...@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.ch...@linux.alibaba.com> > > * gimple-ssa-strength-reduction.c (slsr_process_copycast): Record > information about next_interp and the first cand.
