Reviewers: Yang,

Message:
Hi Yang, PTAL

Description:
Test FeedbackVectorPreservedAcrossRecompiles needs crankshaft

The new test didn't recognize that non-sse2 builds on ia32 would
disable crankshaft.

Please review this at https://codereview.chromium.org/189263007/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+1, -0 lines):
  M test/cctest/test-compiler.cc


Index: test/cctest/test-compiler.cc
diff --git a/test/cctest/test-compiler.cc b/test/cctest/test-compiler.cc
index 065a58dad8a89eb639fdbfef2d3e55508beae240..2947658b59f5bd8d480654699f5ca80835d2d286 100644
--- a/test/cctest/test-compiler.cc
+++ b/test/cctest/test-compiler.cc
@@ -316,6 +316,7 @@ TEST(FeedbackVectorPreservedAcrossRecompiles) {
   if (i::FLAG_always_opt || !i::FLAG_crankshaft) return;
   i::FLAG_allow_natives_syntax = true;
   CcTest::InitializeVM();
+  if (!CcTest::i_isolate()->use_crankshaft()) return;
   v8::HandleScope scope(CcTest::isolate());

   // Make sure function f has a call that uses a type feedback slot.


--
--
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.

Reply via email to