Re: [PATCH GCC]Correct cand_chain and stmt_cand_map for copy/cast

2019-05-21 Thread Bill Schmidt
On 5/15/19 6:09 AM, Richard Biener wrote: > On Wed, May 15, 2019 at 7:54 AM bin.cheng 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 a

Re: [PATCH GCC]Correct cand_chain and stmt_cand_map for copy/cast

2019-05-15 Thread Richard Biener
On Wed, May 15, 2019 at 7:54 AM bin.cheng 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. >

[PATCH GCC]Correct cand_chain and stmt_cand_map for copy/cast

2019-05-14 Thread bin.cheng
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