Reviewers: titzer,
Description:
Enable inlining tests that pass with new scheduler.
[email protected]
TEST=test-run-inlining/InlineTwiceDependentDiamond
Please review this at https://codereview.chromium.org/704293002/
Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+1, -6 lines):
M src/compiler/schedule.h
M test/cctest/cctest.status
Index: src/compiler/schedule.h
diff --git a/src/compiler/schedule.h b/src/compiler/schedule.h
index
b65a50719b1bdb2a03956d7e8fb4dee99cc88b46..95b93b3e0d6e8400d4f565aa268e65800ffd3356
100644
--- a/src/compiler/schedule.h
+++ b/src/compiler/schedule.h
@@ -265,8 +265,6 @@ class Schedule FINAL : public ZoneObject {
private:
friend class Scheduler;
- friend class CodeGenerator;
- friend class ScheduleVisualizer;
friend class BasicBlockInstrumentor;
void AddSuccessor(BasicBlock* block, BasicBlock* succ);
Index: test/cctest/cctest.status
diff --git a/test/cctest/cctest.status b/test/cctest/cctest.status
index
0b769951e875ffce37267f5ff40680f7ea8b0d39..3239e862916c0a691ca3e273e02e1e1b9e94e99e
100644
--- a/test/cctest/cctest.status
+++ b/test/cctest/cctest.status
@@ -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(sigurds): 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.
