101365 2012-11-06 09:58:53 -0800 Should ARMv7Assembler::replaceWithJump cache-flush the nop on page boundary 2013-03-01 08:36:41 -0800 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) Unspecified Unspecified RESOLVED DUPLICATE 102759 P2 Normal --- 108645 1 yong.li.webkit webkit-unassigned oldest_to_newest 759794 0 yong.li.webkit 2012-11-06 09:58:53 -0800 ARMv7Assembler::replaceWithJump() contains following code: if (spansTwo4K) { ptr[-2] = OP_NOP_T1; ptr++; } After that, it calls linkJumpT4(ptr, to) and then cacheFlush(ptr - 2, sizeof(uint16_t) * 2), so it doesn't cache-flush this instruction. According to the comments, this seems a work around of a Cortex-A8 bug. I'm not sure if we should make sure the nop instruction should also be cache-flushed, because the instruction was probably nop anyway. Also, these spansTwo4K are said to workaround a Cortex-A8 issue. Does this also give perf penalty to other cores? 845203 1 yong.li.webkit 2013-03-01 08:36:41 -0800 *** This bug has been marked as a duplicate of bug 102759 ***