Revision: 19596
Author:   [email protected]
Date:     Fri Feb 28 10:31:05 2014 UTC
Log:      A64: fix cctest/test-assembler-a64

The simulator now deletes its decoder in its dtor. Therefore, we must
always allocate the decoder on the heap.

BUG=none
[email protected], [email protected]
LOG=n

Review URL: https://codereview.chromium.org/183893005
http://code.google.com/p/v8/source/detail?r=19596

Modified:
 /branches/bleeding_edge/test/cctest/test-assembler-a64.cc

=======================================
--- /branches/bleeding_edge/test/cctest/test-assembler-a64.cc Wed Feb 26 12:01:05 2014 UTC +++ /branches/bleeding_edge/test/cctest/test-assembler-a64.cc Fri Feb 28 10:31:05 2014 UTC
@@ -113,15 +113,16 @@
 #ifdef USE_SIMULATOR

 // Run tests with the simulator.
-#define SETUP_SIZE(buf_size)                   \
-  Isolate* isolate = Isolate::Current();       \
-  HandleScope scope(isolate);                  \
-  ASSERT(isolate != NULL);                     \
-  byte* buf = new byte[buf_size];              \
-  MacroAssembler masm(isolate, buf, buf_size); \
-  Decoder<DispatchingDecoderVisitor> decoder;  \
-  Simulator simulator(&decoder);               \
-  PrintDisassembler* pdis = NULL;              \
+#define SETUP_SIZE(buf_size)                    \
+  Isolate* isolate = Isolate::Current();        \
+  HandleScope scope(isolate);                   \
+  ASSERT(isolate != NULL);                      \
+  byte* buf = new byte[buf_size];               \
+  MacroAssembler masm(isolate, buf, buf_size);  \
+  Decoder<DispatchingDecoderVisitor>* decoder = \
+      new Decoder<DispatchingDecoderVisitor>(); \
+  Simulator simulator(decoder);                 \
+  PrintDisassembler* pdis = NULL;               \
   RegisterDump core;

/* if (Cctest::trace_sim()) { \

--
--
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/groups/opt_out.

Reply via email to