Reviewers: danno,
Description:
Merged r14545 into 3.18 branch.
Don't double unlink in DCE.
[email protected]
Please review this at https://codereview.chromium.org/15123002/
SVN Base: https://v8.googlecode.com/svn/branches/3.18
Affected files:
M src/hydrogen.cc
M src/version.cc
Index: src/hydrogen.cc
diff --git a/src/hydrogen.cc b/src/hydrogen.cc
index
d1312202bcf6b038b34bcaec75ebf3359091f0aa..ec98dc64a2d72495ba0a979ed8bd7b8322b2b953
100644
--- a/src/hydrogen.cc
+++ b/src/hydrogen.cc
@@ -5380,6 +5380,9 @@ void HGraph::DeadCodeElimination() {
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);
Index: src/version.cc
diff --git a/src/version.cc b/src/version.cc
index
4b5347c74dc25436da321018d3f3155953b2fb0d..733d2343b209d261b0f1e5e211b8d8758aa47dc3
100644
--- a/src/version.cc
+++ b/src/version.cc
@@ -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.
