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

Side by Side Diff: src/hydrogen.h

Issue 12208013: Separated smi check from HBoundsCheck. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Removed platform specific code. Created 7 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 | « src/code-stubs-hydrogen.cc ('k') | src/hydrogen.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 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 857 matching lines...) Expand 10 before | Expand all | Expand 10 after
868 } 868 }
869 Zone* zone() const { return info_->zone(); } 869 Zone* zone() const { return info_->zone(); }
870 HGraph* graph() { return graph_; } 870 HGraph* graph() { return graph_; }
871 871
872 HGraph* CreateGraph(); 872 HGraph* CreateGraph();
873 873
874 // Adding instructions. 874 // Adding instructions.
875 HInstruction* AddInstruction(HInstruction* instr); 875 HInstruction* AddInstruction(HInstruction* instr);
876 void AddSimulate(BailoutId id, 876 void AddSimulate(BailoutId id,
877 RemovableSimulate removable = FIXED_SIMULATE); 877 RemovableSimulate removable = FIXED_SIMULATE);
878 HBoundsCheck* AddBoundsCheck(
879 HValue* index,
880 HValue* length,
881 BoundsCheckKeyMode key_mode = DONT_ALLOW_SMI_KEY,
882 Representation r = Representation::None());
878 883
879 protected: 884 protected:
880 virtual bool BuildGraph() = 0; 885 virtual bool BuildGraph() = 0;
881 886
882 HBasicBlock* CreateBasicBlock(HEnvironment* env); 887 HBasicBlock* CreateBasicBlock(HEnvironment* env);
883 HBasicBlock* CreateLoopHeaderBlock(); 888 HBasicBlock* CreateLoopHeaderBlock();
884 889
885 // Building common constructs 890 // Building common constructs
886 HInstruction* BuildExternalArrayElementAccess( 891 HInstruction* BuildExternalArrayElementAccess(
887 HValue* external_elements, 892 HValue* external_elements,
(...skipping 766 matching lines...) Expand 10 before | Expand all | Expand 10 after
1654 const char* filename_; 1659 const char* filename_;
1655 HeapStringAllocator string_allocator_; 1660 HeapStringAllocator string_allocator_;
1656 StringStream trace_; 1661 StringStream trace_;
1657 int indent_; 1662 int indent_;
1658 }; 1663 };
1659 1664
1660 1665
1661 } } // namespace v8::internal 1666 } } // namespace v8::internal
1662 1667
1663 #endif // V8_HYDROGEN_H_ 1668 #endif // V8_HYDROGEN_H_
OLDNEW
« no previous file with comments | « src/code-stubs-hydrogen.cc ('k') | src/hydrogen.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698