Revision: 25205
Author: [email protected]
Date: Thu Nov 6 18:56:33 2014 UTC
Log: Enable inlining tests that pass with new scheduler.
[email protected]
TEST=test-run-inlining/InlineTwiceDependentDiamond
Review URL: https://codereview.chromium.org/704293002
https://code.google.com/p/v8/source/detail?r=25205
Modified:
/branches/bleeding_edge/src/compiler/schedule.h
/branches/bleeding_edge/test/cctest/cctest.status
=======================================
--- /branches/bleeding_edge/src/compiler/schedule.h Wed Nov 5 10:10:28
2014 UTC
+++ /branches/bleeding_edge/src/compiler/schedule.h Thu Nov 6 18:56:33
2014 UTC
@@ -265,8 +265,6 @@
private:
friend class Scheduler;
- friend class CodeGenerator;
- friend class ScheduleVisualizer;
friend class BasicBlockInstrumentor;
void AddSuccessor(BasicBlock* block, BasicBlock* succ);
=======================================
--- /branches/bleeding_edge/test/cctest/cctest.status Fri Oct 31 13:36:10
2014 UTC
+++ /branches/bleeding_edge/test/cctest/cctest.status Thu Nov 6 18:56:33
2014 UTC
@@ -81,10 +81,7 @@
##############################################################################
# TurboFan compiler failures.
- # TODO(sigurds): The schedule is borked with multiple inlinees,
- # and cannot handle free-floating loops yet
- 'test-run-inlining/InlineTwiceDependentDiamond': [SKIP],
- 'test-run-inlining/InlineTwiceDependentDiamondDifferent': [SKIP],
+ # TODO(mstarzinger): The scheduler cannot handle free-floating loops yet.
'test-run-inlining/InlineLoop': [SKIP],
# Some tests are just too slow to run for now.
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
