Finally fix ARMv7 NEON emulation.
authorDavid 'Digit' Turner <digit@google.com>
Tue, 6 Oct 2009 18:18:29 +0000 (11:18 -0700)
committerDavid 'Digit' Turner <digit@google.com>
Tue, 6 Oct 2009 18:18:29 +0000 (11:18 -0700)
commitbcc6ae14820ddb24e2403d84b420ce61f371ae94
tree0a66fe380b727cf9fce01569c07a4a08f81c40b9
parentddf49e53df97a349f42c733059165dc73c9907dc
Finally fix ARMv7 NEON emulation.

The real problem was the size of the static intermediate TCG opcode buffer.

Due to its SIMD nature, a single Neon instruction can generate a very large number
of corresponding TCG opcodes. Using lots of Neon instructions in a big looop like
the one we have in our ARMv7-optimized memcpy did generate enough opcodes to
overwrite the static gen_opc_buf buffer, resulting in overwrites into the following
global buffer (gen_opparam_buf) corresponding to opcode parameters.

The end result was generation of really broken host machine code, and completely
unreliable emulation, including potential assertion failure in the liveness analysis
pass.

This patch does the following:

- bumps the buffer size from 512 to 2048
- adds sanity checks that will abort the emulator if another similar overwrite
  is detected before machine code is generated.
- remove the previous hack where we disabled the liveness analysis pass for ARMv7

Note that fixing the code generator to not use a static buffer is not trivial at
this point, and that we much prefer to stay true to the upstream sources at the
moment.

Keep in mind that a previous patch also fixed a bug in the ARM->TCG translator
(typo required changing a 0 into a 1) which affected Neon instructions too.

I can't believe I just lost 2 weeks of my life on that bug :-(
android/main.c
exec-all.h
tcg/tcg.c
tcg/tcg.h