[webkit-reviews] review granted: [Bug 194252] B3ReduceStrength: missing peephole optimizations for binary operations : [Attachment 361312] Patch

[bugzilla-daemon at webkit.org]

Saam Barati <sbarati at apple.com> has granted Robin Morisset
<rmorisset at apple.com>'s request for review:
Bug 194252: B3ReduceStrength: missing peephole optimizations for binary
operations
https://bugs.webkit.org/show_bug.cgi?id=194252

Attachment 361312: Patch

https://bugs.webkit.org/attachment.cgi?id=361312&action=review




--- Comment #6 from Saam Barati <sbarati at apple.com> ---
Comment on attachment 361312
  --> https://bugs.webkit.org/attachment.cgi?id=361312
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=361312&action=review

r=me

> Source/JavaScriptCore/ChangeLog:47
> +	   I did not list every single reordering of the arguments as the test
part of this patch is already significantly longer than the useful code, and
this combination should sample all the relevant paths.

That’s not our criteria for adding tests. Why don’t you just make it so your
test doesn’t hard code lhs/rhs to generate all combinations?

> Source/JavaScriptCore/b3/B3ReduceStrength.cpp:1002
> +			   &&
m_value->child(0)->child(1)->isInt(0xffffffffffffffff)

maybe use numeric_limits here?

> Source/JavaScriptCore/b3/B3ReduceStrength.cpp:2242
> +    // Turn any of these:

You should say for op Or/Xor