On Wed, May 31, 2017 at 2:43 PM, Bin.Cheng <amker.ch...@gmail.com> wrote: > On Fri, May 26, 2017 at 12:49 PM, Richard Biener > <richard.guent...@gmail.com> wrote: >> On Thu, May 25, 2017 at 8:00 PM, Bin Cheng <bin.ch...@arm.com> wrote: >>> Hi, >>> I believe this tests has been wrongly modified previously. It is to test >>> that the exit check on >>> pointer shouldn't be replaced by integer IV. Somehow GCC starts replacing >>> the check on >>> integer IV with pointer IV. It's valid, though inefficient. And somehow >>> we starting checking >>> this iv replacement. This patch rectifies it by specifically checking the >>> check on pointer >>> shouldn't be replaced. >> >> So maybe it should then test that the pointer test prevails? Or >> rather that it doesn't replace >> any exit test? If 'p' changes for '_2' for unrelated reasons the >> pattern will be not testing what >> it is supposed to test... > Thanks for reviewing, I updated patch testing if condition on p_limit2 > still exists before expanding. Is it OK?
Ok. Richard. > Thanks, > bin >> >> Richard. >> >>> Bootstrap and test in series on x86_64. Is it OK? >>> Thanks, >>> bin >>> gcc/testsuite/ChangeLog >>> 2017-05-11 Bin Cheng <bin.ch...@arm.com> >>> >>> * gcc.dg/tree-ssa/ivopt_mult_4.c: Explicitly check comparison >>> on pointer should not be replaced.