[webkit-reviews] review granted: [Bug 121064] DFG should support Int48 for local variables : [Attachment 211866] the patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Sep 17 09:47:32 PDT 2013
Oliver Hunt <oliver at apple.com> has granted Filip Pizlo <fpizlo at apple.com>'s
request for review:
Bug 121064: DFG should support Int48 for local variables
https://bugs.webkit.org/show_bug.cgi?id=121064
Attachment 211866: the patch
https://bugs.webkit.org/attachment.cgi?id=211866&action=review
------- Additional Comments from Oliver Hunt <oliver at apple.com>
Looks good -- the only thing i'm really uncomfortable with is the Int48s type -
why do we need this? I can't see anywhere that you're not explicitly using the
values 1 or 2 and then directly loading indices 0 or 1 - I'm assuming that
there's a templatised function somewhere that you pass these to where it's nice
to just iterate the list but i can't find it :(
Also Int48s seems like an odd name, i'd prefer Int48Tuple or something. Then
if you wanted you could typedef Int48Pair, etc into existence.
More information about the webkit-reviews
mailing list