[Webkit-unassigned] [Bug 34424] Soft modulo routine for ARM_TRADITIONAL

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 4 11:47:19 PST 2010


Gavin Barraclough <barraclough at apple.com> changed:

           What    |Removed                     |Added
  Attachment #48140|review?                     |review+
               Flag|                            |

--- Comment #10 from Gavin Barraclough <barraclough at apple.com>  2010-02-04 11:47:18 PST ---
(From update of attachment 48140)
Okay, this basically all looks great.  A couple of small things:

(1) One of the implementations of 'emitSlow_op_mod' contains an
'ASSERT_NOT_REACHED();', one doesn't (before and after your patch).  I think
this should be there in both cases, unless I'm missing something please add
this to the case without the assert.
(2) For configurable JIT optimizations we've been using switches of the format
"ENABLE_JIT_OPTIMIZE_???", and I think it would make sense to move the switch
to bring it in line with these.  (part of the reason is that it may be nice at
some point to move these out of Platform so that changing these switches only
means recompiling the JIT rather than all of WebCore).  I'd suggest
"ENABLE_JIT_OPTIMIZE_MOD", alongside the other JIT_OPTIMIZE switches, and I'd
suggest defining it as something like:


With these changes, r+.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list