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

Unified Diff: runtime/vm/intermediate_language.h

Issue 13872020: Inline Float32x4 Getters (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 8 months 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
diff --git a/runtime/vm/intermediate_language.h b/runtime/vm/intermediate_language.h
index 05b0a215a6840f864b5ab372ccc2c27eb3176b56..829e24a4574d13d9ff8cc86b664e8a51cbfc81c4 100644
--- a/runtime/vm/intermediate_language.h
+++ b/runtime/vm/intermediate_language.h
@@ -71,6 +71,14 @@ class Range;
V(_Double, pow, DoublePow, 631903778) \
V(_Double, _modulo, DoubleMod, 437099337) \
V(::, sqrt, MathSqrt, 1662640002) \
+ V(_Float32x4, get:xxxx, Float32x4ShuffleXXXX, 925028482) \
+ V(_Float32x4, get:yyyy, Float32x4ShuffleYYYY, 925028482) \
+ V(_Float32x4, get:zzzz, Float32x4ShuffleZZZZ, 925028482) \
+ V(_Float32x4, get:wwww, Float32x4ShuffleWWWW, 925028482) \
+ V(_Float32x4, get:x, Float32x4ShuffleX, 502733830) \
+ V(_Float32x4, get:y, Float32x4ShuffleY, 502733830) \
+ V(_Float32x4, get:z, Float32x4ShuffleZ, 502733830) \
+ V(_Float32x4, get:w, Float32x4ShuffleW, 502733830) \
// Class that recognizes the name and owner of a function and returns the
// corresponding enum. See RECOGNIZED_LIST above for list of recognizable
@@ -505,6 +513,7 @@ class EmbeddedArray<T, 0> {
M(GuardField) \
M(IfThenElse) \
M(BinaryFloat32x4Op) \
+ M(Float32x4Shuffle) \
#define FORWARD_DECLARATION(type) class type##Instr;
FOR_EACH_INSTRUCTION(FORWARD_DECLARATION)
@@ -755,6 +764,7 @@ FOR_EACH_INSTRUCTION(INSTRUCTION_TYPE_CHECK)
friend class UnboxFloat32x4Instr;
friend class BinaryDoubleOpInstr;
friend class BinaryFloat32x4OpInstr;
+ friend class Float32x4ShuffleInstr;
friend class BinaryMintOpInstr;
friend class BinarySmiOpInstr;
friend class UnarySmiOpInstr;
@@ -4139,6 +4149,62 @@ class BinaryFloat32x4OpInstr : public TemplateDefinition<2> {
};
+class Float32x4ShuffleInstr : public TemplateDefinition<1> {
+ public:
+ Float32x4ShuffleInstr(MethodRecognizer::Kind op_kind, Value* value,
+ InstanceCallInstr* instance_call)
+ : op_kind_(op_kind) {
+ SetInputAt(0, value);
+ deopt_id_ = instance_call->deopt_id();
+ }
+
+ Value* value() const { return inputs_[0]; }
+
+ MethodRecognizer::Kind op_kind() const { return op_kind_; }
+
+ virtual void PrintOperandsTo(BufferFormatter* f) const;
+
+ virtual bool CanDeoptimize() const { return false; }
+
+ virtual bool HasSideEffect() const { return false; }
+
+ virtual bool AffectedBySideEffect() const { return false; }
+
+ virtual bool AttributesEqual(Instruction* other) const {
+ return op_kind() == other->AsFloat32x4Shuffle()->op_kind();
+ }
+
+ virtual Representation representation() const {
+ if (op_kind_ == MethodRecognizer::kFloat32x4ShuffleX ||
srdjan 2013/04/22 22:28:18 Use extra parentheses.
Cutch 2013/04/22 23:05:41 Done.
+ op_kind_ == MethodRecognizer::kFloat32x4ShuffleY ||
+ op_kind_ == MethodRecognizer::kFloat32x4ShuffleZ ||
+ op_kind_ == MethodRecognizer::kFloat32x4ShuffleW) {
+ return kUnboxedDouble;
+ }
+ return kUnboxedFloat32x4;
+ }
+
+ virtual Representation RequiredInputRepresentation(intptr_t idx) const {
+ ASSERT((idx == 0));
srdjan 2013/04/22 22:28:18 Don't use extra parentheses :-)
Cutch 2013/04/22 23:05:41 Done.
+ return kUnboxedFloat32x4;
+ }
+
+ virtual intptr_t DeoptimizationTarget() const {
+ // Direct access since this instruction cannot deoptimize, and the deopt-id
+ // was inherited from another instruction that could deoptimize.
+ return deopt_id_;
+ }
+
+ DECLARE_INSTRUCTION(Float32x4Shuffle)
+ virtual CompileType ComputeType() const;
+
+ private:
+ const MethodRecognizer::Kind op_kind_;
+
+ DISALLOW_COPY_AND_ASSIGN(Float32x4ShuffleInstr);
+};
+
+
class BinaryMintOpInstr : public TemplateDefinition<2> {
public:
BinaryMintOpInstr(Token::Kind op_kind,

Powered by Google App Engine
This is Rietveld 408576698