Reviewers: fschneider,
Message:
I override only in concrete classes, even if kUseGVN is set in a
non-concrete
instruction base class (setting kUseGVN in such a way is error prone too,
so we
should consider getting rid of it).
We could also consider when verifying instructions:
if (instr->CheckFlag(kUseGVN)) { instr->DataEquals(instr); }
to verify that we don't have the UNREACHABLE() implementation.
Description:
Change the default implementation of DataEquals for Hydrogen instructions.
The former default was true. The new default is false and the default
implementation is UNREACHABLE so it asserts in debug builds. The function
is overridden in all concrete instruction classes that might have the flag
kUseGVN set.
Please review this at http://codereview.chromium.org/6255013/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/hydrogen-instructions.h
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
