Revision: 14631
Author: [email protected]
Date: Mon May 13 00:51:20 2013
Log: Merged r14545 into 3.18 branch.
Don't double unlink in DCE.
[email protected]
Review URL: https://codereview.chromium.org/15123002
http://code.google.com/p/v8/source/detail?r=14631
Modified:
/branches/3.18/src/hydrogen.cc
/branches/3.18/src/version.cc
=======================================
--- /branches/3.18/src/hydrogen.cc Tue Apr 30 09:36:45 2013
+++ /branches/3.18/src/hydrogen.cc Mon May 13 00:51:20 2013
@@ -5380,6 +5380,9 @@
while (!worklist.is_empty()) {
HInstruction* instr = worklist.RemoveLast();
+ // This happens when an instruction is used multiple times as operand.
That
+ // in turn could happen through GVN.
+ if (!instr->IsLinked()) continue;
if (FLAG_trace_dead_code_elimination) {
HeapStringAllocator allocator;
StringStream stream(&allocator);
=======================================
--- /branches/3.18/src/version.cc Tue Apr 30 12:22:07 2013
+++ /branches/3.18/src/version.cc Mon May 13 00:51:20 2013
@@ -35,7 +35,7 @@
#define MAJOR_VERSION 3
#define MINOR_VERSION 18
#define BUILD_NUMBER 5
-#define PATCH_LEVEL 2
+#define PATCH_LEVEL 3
// Use 1 for candidates and 0 otherwise.
// (Boolean macro values are not supported by all preprocessors.)
#define IS_CANDIDATE_VERSION 0
--
--
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.
