Re: Confusing problem involving uint64_t bitops

  • From: Mike Pall <mike-1502@xxxxxxxxxx>
  • To: luajit@xxxxxxxxxxxxx
  • Date: Mon, 9 Feb 2015 20:46:40 +0100

Luke Gorrie wrote:
> I was able to bisect. (It is easy to reproduce in my test environment.)
> This is the commit that makes the problem disappear:
> 
> commit 6d0654d3eca7654c9c4f8a9923907d06b177a8a1
> Author: Mike Pall <mike>
> Date:   Wed Oct 8 22:04:51 2014 +0200
> 
>     Fix fused constant loads under high register pressure.
> 
> If I revert this change from v2.1 HEAD then the problem reappears.
> 
> Case closed? (what do you think?)

You could add the first assert from the commit, but without the
actual fix. Then rebuild LuaJIT with assertions turned on. If it
asserts for your test case, then we can be sure this issue has
been fixed.

--Mike

Other related posts: