Summary: | DFG should know how to speculate StringOrOther | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Filip Pizlo <fpizlo> | ||||||||||||
Component: | JavaScriptCore | Assignee: | Filip Pizlo <fpizlo> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | commit-queue, keith_miller, mark.lam, msaboff, saam | ||||||||||||
Priority: | P2 | ||||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||
Hardware: | All | ||||||||||||||
OS: | All | ||||||||||||||
Attachments: |
|
Description
Filip Pizlo
2016-03-06 12:36:20 PST
Created attachment 273140 [details]
work in progress
Created attachment 273141 [details]
it might be done
Created attachment 273143 [details]
the patch
Attachment 273143 [details] did not pass style-queue:
ERROR: Source/JavaScriptCore/dfg/DFGNode.h:1964: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Total errors found: 1 in 15 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 273143 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=273143&action=review r=me > Source/JavaScriptCore/ChangeLog:17 > + To do this, I needed to introduce code into the FTL that creates new blocks. We still had that > + awful FTL_NEW_BLOCK idiom since the only way to debug LLVM IR was to ascribe names to basic > + blocks. B3 IR is inherently more debuggable since unlike LLVM, B3 knows how to always respect > + code origin, and it knows how to print the code origin nicely in the dumps. So, rather than > + continue using FTL_NEW_BLOCK(m_out, ("things")), I replaced all of that stuff with > + m_out.newBlock(). It's much nicer that way. Nice! Comment on attachment 273143 [details] the patch Attachment 273143 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/933202 Number of test failures exceeded the failure limit. Created attachment 273146 [details]
Archive of layout-test-results from ews115 for mac-yosemite
The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews115 Port: mac-yosemite Platform: Mac OS X 10.10.5
(In reply to comment #7) > Created attachment 273146 [details] > Archive of layout-test-results from ews115 for mac-yosemite > > The attached test failures were seen while running run-webkit-tests on the > mac-debug-ews. > Bot: ews115 Port: mac-yosemite Platform: Mac OS X 10.10.5 Looks like I forgot to use ManualOperandSpeculation in some places. Created attachment 273154 [details]
patch for landing
This should resolve the debug test failures.
Attachment 273154 [details] did not pass style-queue:
ERROR: Source/JavaScriptCore/dfg/DFGNode.h:1964: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Total errors found: 1 in 15 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Landed in http://trac.webkit.org/changeset/197649 |