Summary: | Remove Broken CompareEq constant folding phase. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Keith Miller <keith_miller> | ||||||
Component: | JavaScriptCore | Assignee: | Keith Miller <keith_miller> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | buildbot, mark.lam, msaboff, saam, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Keith Miller
2017-07-25 17:09:38 PDT
Created attachment 316409 [details]
Patch
Created attachment 316411 [details]
Patch
Comment on attachment 316411 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=316411&action=review > Source/JavaScriptCore/dfg/DFGConstantFoldingPhase.cpp:138 > + // FIXME: We should add back the broken folding phase here for comparisions with thing we prove one side has type SpecOther. thing => things. Also, please add bug URL or remove FIXME. Comment on attachment 316411 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=316411&action=review >> Source/JavaScriptCore/dfg/DFGConstantFoldingPhase.cpp:138 >> + // FIXME: We should add back the broken folding phase here for comparisions with thing we prove one side has type SpecOther. > > thing => things. > > Also, please add bug URL or remove FIXME. Fixed. |