On Jun 30, 2012, at 2:18 AM, Zoltan Herczeg wrote:

> wouldn't it be better, if the watchpoint would insert the nops, detects if
> called again (and no more nops are inserted) and the label would work as
> it worked before (we could get rid of one label type in this case). The
> operations performed by inlined labels seems too much at the moment.

In practice the space between a watchpoint and the subsequent label is almost 
never smaller than maxJumpReplacementSize, which means we pretty much never 
need to emit a nop.  The mechanism largely exists as a safety net, to ensure 
correct behaviour in very unlikely cases.  Switching to inserting nops after 
watchpoints would add a large number of unnecessary instructions to the code.

cheers,
G.
_______________________________________________
squirrelfish-dev mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/squirrelfish-dev

Reply via email to