Re: [PATCH] Fix PR63422 to handle profile insanities

2014-10-02 Thread Jeff Law
On 10/01/14 11:03, Xinliang David Li wrote: Probably need to file a bug to track the copyrename2 problem. Yea, that would definitely be appreciated. Jeff

Re: [PATCH] Fix PR63422 to handle profile insanities

2014-10-02 Thread Jeff Law
On 10/01/14 10:59, Teresa Johnson wrote: This patch removes some asserts my jump threading patch r215739 added. An upstream pass (copyrename2) is introducing some bogus profile counts, so we can't assert that counts are 0 when there is no profile data for the function. Tested on testcase attache

Re: [PATCH] Fix PR63422 to handle profile insanities

2014-10-02 Thread Teresa Johnson
On Wed, Oct 1, 2014 at 10:03 AM, Xinliang David Li wrote: > Probably need to file a bug to track the copyrename2 problem. I have filed https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63443 to track this. > > David > > On Wed, Oct 1, 2014 at 9:59 AM, Teresa Johnson wrote: >> This patch removes some

Re: [PATCH] Fix PR63422 to handle profile insanities

2014-10-01 Thread Xinliang David Li
Probably need to file a bug to track the copyrename2 problem. David On Wed, Oct 1, 2014 at 9:59 AM, Teresa Johnson wrote: > This patch removes some asserts my jump threading patch r215739 added. > An upstream pass (copyrename2) is introducing some bogus profile > counts, so we can't assert that

[PATCH] Fix PR63422 to handle profile insanities

2014-10-01 Thread Teresa Johnson
This patch removes some asserts my jump threading patch r215739 added. An upstream pass (copyrename2) is introducing some bogus profile counts, so we can't assert that counts are 0 when there is no profile data for the function. Tested on testcase attached to PR63422. Currently running gcc regress