Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(405)

Unified Diff: runtime/vm/intermediate_language.h

Issue 11280230: Optimize checked mode asserts with uninstantiated types and known constant type-arguments. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: runtime/vm/intermediate_language.h
===================================================================
--- runtime/vm/intermediate_language.h (revision 15455)
+++ runtime/vm/intermediate_language.h (working copy)
@@ -26,6 +26,7 @@
class Instruction;
class LocalVariable;
class Range;
+class FlowGraphOptimizer;
// TODO(srdjan): Add _ByteArrayBase, get:length.
@@ -469,7 +470,7 @@
// Returns a replacement for the instruction or NULL if the instruction can
// be eliminated. By default returns the this instruction which means no
// change.
- virtual Instruction* Canonicalize();
+ virtual Instruction* Canonicalize(FlowGraphOptimizer* optimizer);
// Insert this instruction before 'next'.
void InsertBefore(Instruction* next);
@@ -1136,7 +1137,7 @@
// Definitions can be canonicalized only into definitions to ensure
// this check statically we override base Canonicalize with a Canonicalize
// returning Definition (return type is covariant).
- virtual Definition* Canonicalize();
+ virtual Definition* Canonicalize(FlowGraphOptimizer* optimizer);
protected:
friend class RangeAnalysis;
@@ -1849,7 +1850,7 @@
const AbstractType& dst_type,
const String& dst_name)
: token_pos_(token_pos),
- dst_type_(dst_type),
+ dst_type_(AbstractType::ZoneHandle(dst_type.raw())),
dst_name_(dst_name),
is_eliminated_(false) {
ASSERT(value != NULL);
@@ -1871,6 +1872,9 @@
intptr_t token_pos() const { return token_pos_; }
const AbstractType& dst_type() const { return dst_type_; }
+ void set_dst_type(const AbstractType& dst_type) {
+ dst_type_ = dst_type.raw();
+ }
const String& dst_name() const { return dst_name_; }
bool is_eliminated() const {
@@ -1893,11 +1897,11 @@
virtual intptr_t ResultCid() const { return value()->ResultCid(); }
virtual intptr_t GetPropagatedCid();
- virtual Definition* Canonicalize();
+ virtual Definition* Canonicalize(FlowGraphOptimizer* optimizer);
private:
const intptr_t token_pos_;
- const AbstractType& dst_type_;
+ AbstractType& dst_type_;
const String& dst_name_;
bool is_eliminated_;
@@ -1939,7 +1943,7 @@
virtual intptr_t ResultCid() const { return kBoolCid; }
- virtual Definition* Canonicalize();
+ virtual Definition* Canonicalize(FlowGraphOptimizer* optimizer);
private:
const intptr_t token_pos_;
@@ -2248,7 +2252,7 @@
virtual bool AttributesEqual(Instruction* other) const;
virtual bool AffectedBySideEffect() const { return false; }
- virtual Definition* Canonicalize();
+ virtual Definition* Canonicalize(FlowGraphOptimizer* optimizer);
virtual intptr_t ResultCid() const { return kBoolCid; }
@@ -3387,7 +3391,7 @@
Value* right() const { return inputs_[1]; }
- virtual Instruction* Canonicalize();
+ virtual Instruction* Canonicalize(FlowGraphOptimizer* optimizer);
private:
DISALLOW_COPY_AND_ASSIGN(CheckEitherNonSmiInstr);
@@ -3999,7 +4003,7 @@
const ICData& unary_checks() const { return unary_checks_; }
- virtual Instruction* Canonicalize();
+ virtual Instruction* Canonicalize(FlowGraphOptimizer* optimizer);
virtual void PrintOperandsTo(BufferFormatter* f) const;
@@ -4032,7 +4036,7 @@
virtual bool AffectedBySideEffect() const { return false; }
- virtual Instruction* Canonicalize();
+ virtual Instruction* Canonicalize(FlowGraphOptimizer* optimizer);
Value* value() const { return inputs_[0]; }

Powered by Google App Engine
This is Rietveld 408576698