Re: [PATCH] Make jump thread path carry more information

2013-09-27 Thread Jeff Law 

On 09/27/2013 08:42 AM, James Greenhalgh wrote:

On Thu, Sep 26, 2013 at 04:26:35AM +0100, Jeff Law wrote:

Bootstrapped and regression tested on x86_64-unknown-linux-gnu.
Installed on trunk.


Hi Jeff,

This patch caused a regression on Arm and AArch64 in:

PASS-FAIL: gcc.c-torture/execute/memcpy-2.c execution,  -O3 
-fomit-frame-pointer

 From what I can see, the only place the behaviour of the threader has
changed is in this hunk:
Yes.  The old code was dropping the tail off the thread path; if we're 
seeing failures on the ARM port as a result of fixing that goof we 
obviously need to address them.


Let me take a looksie :-)

If you could pass along a .i file it'd be helpful in case I want to look 
at something under the debugger.



jeff

Re: [PATCH] Make jump thread path carry more information

2013-09-27 Thread James Greenhalgh 
On Fri, Sep 27, 2013 at 04:32:10PM +0100, Jeff Law wrote:
 If you could pass along a .i file it'd be helpful in case I want to look 
 at something under the debugger.

I've opened http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58553 to save
everyone's inboxes.

Let me know if I can do anything else to help.

Cheers,
James

Re: [PATCH] Make jump thread path carry more information

2013-09-27 Thread Jeff Law 

On 09/27/2013 10:48 AM, James Greenhalgh wrote:

On Fri, Sep 27, 2013 at 04:32:10PM +0100, Jeff Law wrote:

If you could pass along a .i file it'd be helpful in case I want to look
at something under the debugger.


I've opened http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58553 to save
everyone's inboxes.

Let me know if I can do anything else to help.

THanks.  I'm investigating.

jeff