...
Created attachment 359701 [details] Patch
Comment on attachment 359701 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=359701&action=review r=me > Source/JavaScriptCore/dfg/DFGDoesGC.cpp:388 > + return node->binaryUseKind() == BigIntUse; Might as well return true here. Because the clobberize check above, we won’t reach here. However, it’s nicer just to say true IMO
Created attachment 359704 [details] Benchmarks Patch seems to be perf neutral on macOS x86_64.
Comment on attachment 359701 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=359701&action=review Thank you for the review! >> Source/JavaScriptCore/dfg/DFGDoesGC.cpp:388 >> + return node->binaryUseKind() == BigIntUse; > > Might as well return true here. Because the clobberize check above, we won’t reach here. However, it’s nicer just to say true IMO Makes sense. Changed.
Created attachment 359705 [details] Patch
Comment on attachment 359705 [details] Patch Clearing flags on attachment: 359705 Committed r240244: <https://trac.webkit.org/changeset/240244>
All reviewed patches have been landed. Closing bug.
<rdar://problem/47433654>