Summary: | B3::reduceStrength() should do DCE | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Filip Pizlo <fpizlo> | ||||
Component: | JavaScriptCore | Assignee: | Filip Pizlo <fpizlo> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | barraclough, benjamin, commit-queue, ggaren, mark.lam, mhahnenb, msaboff, nrotem, oliver, saam, sam | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 150507 | ||||||
Attachments: |
|
Description
Filip Pizlo
2015-10-28 18:45:54 PDT
Created attachment 264284 [details]
the patch
Attachment 264284 [details] did not pass style-queue:
ERROR: Source/JavaScriptCore/b3/B3ProcedureInlines.h:39: More than one command on the same line [whitespace/newline] [4]
Total errors found: 1 in 7 files
If any of these errors are false positives, please file a bug against check-webkit-style.
(In reply to comment #2) > Attachment 264284 [details] did not pass style-queue: > > > ERROR: Source/JavaScriptCore/b3/B3ProcedureInlines.h:39: More than one > command on the same line [whitespace/newline] [4] > Total errors found: 1 in 7 files > > > If any of these errors are false positives, please file a bug against > check-webkit-style. Fixed. Comment on attachment 264284 [details]
the patch
LGTM
Landed in http://trac.webkit.org/changeset/191718 |