Reviewers: Toon Verwaest,

Description:
Removed useless Canonicalize implementations subsumed by dead code elimination.


Please review this at http://codereview.chromium.org/11362085/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/hydrogen-instructions.h
  M src/hydrogen-instructions.cc


Index: src/hydrogen-instructions.cc
diff --git a/src/hydrogen-instructions.cc b/src/hydrogen-instructions.cc
index 72d3aab4944596cd1caf7d09a38f8829ca9d5528..feac1be0bcfcdf45f6c435f7899acd645d24e4e8 100644
--- a/src/hydrogen-instructions.cc
+++ b/src/hydrogen-instructions.cc
@@ -859,16 +859,6 @@ void HLoadFieldByIndex::PrintDataTo(StringStream* stream) {
 }


-HValue* HConstant::Canonicalize() {
-  return HasNoUses() ? NULL : this;
-}
-
-
-HValue* HTypeof::Canonicalize() {
-  return HasNoUses() ? NULL : this;
-}
-
-
 HValue* HBitwise::Canonicalize() {
   if (!representation().IsInteger32()) return this;
   // If x is an int32, then x & -1 == x, x | 0 == x and x ^ 0 == x.
Index: src/hydrogen-instructions.h
diff --git a/src/hydrogen-instructions.h b/src/hydrogen-instructions.h
index 4ad98b83c2751a3ff2322e8df1f1a28caad18b59..d11835452c72d7024f2a38bb49a854eba7ce45f1 100644
--- a/src/hydrogen-instructions.h
+++ b/src/hydrogen-instructions.h
@@ -2610,7 +2610,6 @@ class HConstant: public HTemplateInstruction<0> {
   }

   virtual bool EmitAtUses() { return !representation().IsDouble(); }
-  virtual HValue* Canonicalize();
   virtual void PrintDataTo(StringStream* stream);
   virtual HType CalculateInferredType();
   bool IsInteger() { return handle()->IsSmi(); }
@@ -5034,7 +5033,6 @@ class HTypeof: public HTemplateInstruction<2> {
   HValue* context() { return OperandAt(0); }
   HValue* value() { return OperandAt(1); }

-  virtual HValue* Canonicalize();
   virtual void PrintDataTo(StringStream* stream);

   virtual Representation RequiredInputRepresentation(int index) {


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to