https://codereview.chromium.org/13950013/diff/3001/src/hydrogen.cc
File src/hydrogen.cc (right):
https://codereview.chromium.org/13950013/diff/3001/src/hydrogen.cc#newcode3745
src/hydrogen.cc:3745: worklist.Add(phi, zone());
I think you may want to prevent duplicate entries in the worklist by
guarding this with a BitVector (look at other worklist algorithms in
this file for examples). The computed result is correct either way, but
the list could grow excessively for pathological inputs, as there can
easily be cycles in Phi def-use chains.
https://codereview.chromium.org/13950013/
--
--
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.
