Re: Patch ping (Re: [PATCH PR82965/PR83991]Fix invalid profile count in vectorization peeling)

2018-03-11 Thread Paul Hua
On Fri, Mar 9, 2018 at 10:51 PM, Bin.Cheng wrote: > On Fri, Mar 9, 2018 at 10:25 AM, Paul Hua wrote: >> It's looks fixed >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82965#c12 on mips64el. > Hmm, is it fixed? or is it exposed now on mips64el? I read the latter > from the comment. It is fixe

Re: Patch ping (Re: [PATCH PR82965/PR83991]Fix invalid profile count in vectorization peeling)

2018-03-09 Thread Bin.Cheng
On Fri, Mar 9, 2018 at 10:25 AM, Paul Hua wrote: > It's looks fixed > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82965#c12 on mips64el. Hmm, is it fixed? or is it exposed now on mips64el? I read the latter from the comment. I think the issue is like explained, but haven't dug into when/why it

Re: Patch ping (Re: [PATCH PR82965/PR83991]Fix invalid profile count in vectorization peeling)

2018-03-09 Thread Paul Hua
It's looks fixed https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82965#c12 on mips64el. Thanks. On Mon, Feb 26, 2018 at 8:02 PM, Bin.Cheng wrote: > Ping^2 > > Thanks, > bin > > On Mon, Feb 19, 2018 at 5:14 PM, Jakub Jelinek wrote: >> Hi! >> >> Honza, do you think you could have a look at this P1

Re: Patch ping (Re: [PATCH PR82965/PR83991]Fix invalid profile count in vectorization peeling)

2018-02-26 Thread Bin.Cheng
Ping^2 Thanks, bin On Mon, Feb 19, 2018 at 5:14 PM, Jakub Jelinek wrote: > Hi! > > Honza, do you think you could have a look at this P1 fix? > > Thanks. > > On Wed, Jan 31, 2018 at 10:03:51AM +, Bin Cheng wrote: >> Hi, >> This patch fixes invalid profile count information in vectorization pe

Patch ping (Re: [PATCH PR82965/PR83991]Fix invalid profile count in vectorization peeling)

2018-02-19 Thread Jakub Jelinek
Hi! Honza, do you think you could have a look at this P1 fix? Thanks. On Wed, Jan 31, 2018 at 10:03:51AM +, Bin Cheng wrote: > Hi, > This patch fixes invalid profile count information in vectorization peeling. > Current implementation is a bit confusing for me since it tries to compute > an

[PATCH PR82965/PR83991]Fix invalid profile count in vectorization peeling

2018-01-31 Thread Bin Cheng
Hi, This patch fixes invalid profile count information in vectorization peeling. Current implementation is a bit confusing for me since it tries to compute an overall probability based on scaling probability and change of estimated niters. This patch does it in two steps. Firstly it does the scal