Enable inlining tests that pass with new scheduler.
authormstarzinger@chromium.org <mstarzinger@chromium.org>
Thu, 6 Nov 2014 18:56:33 +0000 (18:56 +0000)
committermstarzinger@chromium.org <mstarzinger@chromium.org>
Thu, 6 Nov 2014 18:56:43 +0000 (18:56 +0000)
R=titzer@chromium.org
TEST=test-run-inlining/InlineTwiceDependentDiamond

Review URL: https://codereview.chromium.org/704293002

Cr-Commit-Position: refs/heads/master@{#25205}
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@25205 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

src/compiler/schedule.h
test/cctest/cctest.status

index b65a507..95b93b3 100644 (file)
@@ -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 0b76995..9e85a0c 100644 (file)
   ##############################################################################
   # 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.