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

Side by Side Diff: runtime/vm/intermediate_language.h

Issue 135553005: Fix issue 16592: Do not attempt to merge operations that do not have uses, otherwise we add force u… (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 6 years, 10 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « runtime/vm/flow_graph_optimizer.cc ('k') | runtime/vm/intermediate_language.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #ifndef VM_INTERMEDIATE_LANGUAGE_H_ 5 #ifndef VM_INTERMEDIATE_LANGUAGE_H_
6 #define VM_INTERMEDIATE_LANGUAGE_H_ 6 #define VM_INTERMEDIATE_LANGUAGE_H_
7 7
8 #include "vm/allocation.h" 8 #include "vm/allocation.h"
9 #include "vm/ast.h" 9 #include "vm/ast.h"
10 #include "vm/growable_array.h" 10 #include "vm/growable_array.h"
(...skipping 4891 matching lines...) Expand 10 before | Expand all | Expand 10 after
4902 4902
4903 virtual bool AllowsCSE() const { return true; } 4903 virtual bool AllowsCSE() const { return true; }
4904 virtual EffectSet Effects() const { return EffectSet::None(); } 4904 virtual EffectSet Effects() const { return EffectSet::None(); }
4905 virtual EffectSet Dependencies() const { return EffectSet::None(); } 4905 virtual EffectSet Dependencies() const { return EffectSet::None(); }
4906 virtual bool AttributesEqual(Instruction* other) const { 4906 virtual bool AttributesEqual(Instruction* other) const {
4907 return kind() == other->AsMathUnary()->kind(); 4907 return kind() == other->AsMathUnary()->kind();
4908 } 4908 }
4909 4909
4910 virtual bool MayThrow() const { return false; } 4910 virtual bool MayThrow() const { return false; }
4911 4911
4912 Definition* Canonicalize(FlowGraph* flow_graph);
4913
4912 private: 4914 private:
4913 const MethodRecognizer::Kind kind_; 4915 const MethodRecognizer::Kind kind_;
4914 4916
4915 DISALLOW_COPY_AND_ASSIGN(MathUnaryInstr); 4917 DISALLOW_COPY_AND_ASSIGN(MathUnaryInstr);
4916 }; 4918 };
4917 4919
4918 4920
4919 // Represents Math's static min and max functions. 4921 // Represents Math's static min and max functions.
4920 class MathMinMaxInstr : public TemplateDefinition<2> { 4922 class MathMinMaxInstr : public TemplateDefinition<2> {
4921 public: 4923 public:
(...skipping 2198 matching lines...) Expand 10 before | Expand all | Expand 10 after
7120 ForwardInstructionIterator* current_iterator_; 7122 ForwardInstructionIterator* current_iterator_;
7121 7123
7122 private: 7124 private:
7123 DISALLOW_COPY_AND_ASSIGN(FlowGraphVisitor); 7125 DISALLOW_COPY_AND_ASSIGN(FlowGraphVisitor);
7124 }; 7126 };
7125 7127
7126 7128
7127 } // namespace dart 7129 } // namespace dart
7128 7130
7129 #endif // VM_INTERMEDIATE_LANGUAGE_H_ 7131 #endif // VM_INTERMEDIATE_LANGUAGE_H_
OLDNEW
« no previous file with comments | « runtime/vm/flow_graph_optimizer.cc ('k') | runtime/vm/intermediate_language.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698