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

Side by Side Diff: src/hydrogen.h

Issue 10867033: Disable speculative LICM when it may lead to unnecessary deopts (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: feedback addressed Created 8 years, 3 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/heap.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 317 matching lines...) Expand 10 before | Expand all | Expand 10 after
328 } 328 }
329 329
330 ZoneList<HUnknownOSRValue*>* osr_values() { 330 ZoneList<HUnknownOSRValue*>* osr_values() {
331 return osr_values_.get(); 331 return osr_values_.get();
332 } 332 }
333 333
334 void set_osr_values(ZoneList<HUnknownOSRValue*>* values) { 334 void set_osr_values(ZoneList<HUnknownOSRValue*>* values) {
335 osr_values_.set(values); 335 osr_values_.set(values);
336 } 336 }
337 337
338 int update_type_change_checksum(int delta) {
339 type_change_checksum_ += delta;
340 return type_change_checksum_;
341 }
342
343 bool use_optimistic_licm() {
344 return use_optimistic_licm_;
345 }
346
347 void set_use_optimistic_licm(bool value) {
348 use_optimistic_licm_ = value;
349 }
350
338 void MarkRecursive() { 351 void MarkRecursive() {
339 is_recursive_ = true; 352 is_recursive_ = true;
340 } 353 }
341 354
342 bool is_recursive() const { 355 bool is_recursive() const {
343 return is_recursive_; 356 return is_recursive_;
344 } 357 }
345 358
346 private: 359 private:
347 HConstant* GetConstant(SetOncePointer<HConstant>* pointer, 360 HConstant* GetConstant(SetOncePointer<HConstant>* pointer,
(...skipping 30 matching lines...) Expand all
378 SetOncePointer<HConstant> constant_hole_; 391 SetOncePointer<HConstant> constant_hole_;
379 SetOncePointer<HArgumentsObject> arguments_object_; 392 SetOncePointer<HArgumentsObject> arguments_object_;
380 393
381 SetOncePointer<HBasicBlock> osr_loop_entry_; 394 SetOncePointer<HBasicBlock> osr_loop_entry_;
382 SetOncePointer<ZoneList<HUnknownOSRValue*> > osr_values_; 395 SetOncePointer<ZoneList<HUnknownOSRValue*> > osr_values_;
383 396
384 CompilationInfo* info_; 397 CompilationInfo* info_;
385 Zone* zone_; 398 Zone* zone_;
386 399
387 bool is_recursive_; 400 bool is_recursive_;
401 bool use_optimistic_licm_;
402 int type_change_checksum_;
388 403
389 DISALLOW_COPY_AND_ASSIGN(HGraph); 404 DISALLOW_COPY_AND_ASSIGN(HGraph);
390 }; 405 };
391 406
392 407
393 Zone* HBasicBlock::zone() const { return graph_->zone(); } 408 Zone* HBasicBlock::zone() const { return graph_->zone(); }
394 409
395 410
396 // Type of stack frame an environment might refer to. 411 // Type of stack frame an environment might refer to.
397 enum FrameType { 412 enum FrameType {
(...skipping 1059 matching lines...) Expand 10 before | Expand all | Expand 10 after
1457 const char* filename_; 1472 const char* filename_;
1458 HeapStringAllocator string_allocator_; 1473 HeapStringAllocator string_allocator_;
1459 StringStream trace_; 1474 StringStream trace_;
1460 int indent_; 1475 int indent_;
1461 }; 1476 };
1462 1477
1463 1478
1464 } } // namespace v8::internal 1479 } } // namespace v8::internal
1465 1480
1466 #endif // V8_HYDROGEN_H_ 1481 #endif // V8_HYDROGEN_H_
OLDNEW
« no previous file with comments | « src/heap.cc ('k') | src/hydrogen.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698