Comment #6 on issue 2868 by [email protected]: Performance regression
after bleeding edge revision 16406
http://code.google.com/p/v8/issues/detail?id=2868
Unfortunately, haven't gotten around to this in the last few days. Attached
patch by Jakob fixes part of the regression, but not completely.
diff --git a/src/hydrogen-gvn.cc b/src/hydrogen-gvn.cc
index 9a02a1d..cdac3ef 100644
--- a/src/hydrogen-gvn.cc
+++ b/src/hydrogen-gvn.cc
@@ -401,7 +401,7 @@ void
HGlobalValueNumberingPhase::ComputeBlockSideEffects() {
for (HInstructionIterator it(block); !it.Done(); it.Advance()) {
HInstruction* instr = it.Current();
side_effects.Add(instr->ChangesFlags());
- if (instr->IsDeoptimize()) {
+ if (instr->IsDeoptimize() || instr->IsThrow()) {
block_side_effects_[id].RemoveAll();
side_effects.RemoveAll();
break;
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
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.
