[PATCH] Fix for ExecutionEngine buglet

Kaylor, Andrew andrew.kaylor at intel.com
Fri Nov 15 09:57:15 PST 2013


Looks good.

Committed as r194831.


-----Original Message-----
From: Dale E. Martin [mailto:dale at the-martins.org] 
Sent: Friday, November 15, 2013 7:05 AM
To: llvm-commits at cs.uiuc.edu
Cc: Kaylor, Andrew
Subject: Re: [PATCH] Fix for ExecutionEngine buglet

On Wed, Nov 13, 2013 at 07:44:32PM +0000, Kaylor, Andrew wrote:
> I think the early return is preferable.  It doesn't look like the 
> variables that get updated without the early return are used for 
> anything other than statistic tracking, but they would be inaccurate in that case.

I made the change to do an early return and confirmed that it still fixes my failing test.

Patch attached.

Thanks,
  Dale
--
Dale E. Martin - dale at the-martins.org
http://the-martins.org/~dmartin




More information about the llvm-commits mailing list