Revision: 14413
Author: [email protected]
Date: Wed Apr 24 05:39:30 2013
Log: Properly consider Double inputs for representation inference when
they are not marked as kFlexibleRepresentation
Review URL: https://codereview.chromium.org/14408006
http://code.google.com/p/v8/source/detail?r=14413
Modified:
/branches/bleeding_edge/src/hydrogen-instructions.cc
=======================================
--- /branches/bleeding_edge/src/hydrogen-instructions.cc Tue Apr 23
02:23:07 2013
+++ /branches/bleeding_edge/src/hydrogen-instructions.cc Wed Apr 24
05:39:30 2013
@@ -2248,12 +2248,10 @@
Representation left_rep = left()->representation();
Representation right_rep = right()->representation();
- if (left_rep.is_more_general_than(rep) &&
- left()->CheckFlag(kFlexibleRepresentation)) {
+ if (left_rep.is_more_general_than(rep) && !left_rep.IsTagged()) {
rep = left_rep;
}
- if (right_rep.is_more_general_than(rep) &&
- right()->CheckFlag(kFlexibleRepresentation)) {
+ if (right_rep.is_more_general_than(rep) && !right_rep.IsTagged()) {
rep = right_rep;
}
// Consider observed output representation, but ignore it if it's Double,
--
--
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.
