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

Side by Side Diff: src/hydrogen.cc

Issue 14364010: Lithium: avoid registers for constants when possible. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Last comments 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « src/arm/lithium-codegen-arm.cc ('k') | src/hydrogen-instructions.h » ('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 1476 matching lines...) Expand 10 before | Expand all | Expand 10 after
1487 1487
1488 1488
1489 HInstruction* HGraphBuilder::BuildStoreMap(HValue* object, 1489 HInstruction* HGraphBuilder::BuildStoreMap(HValue* object,
1490 HValue* map) { 1490 HValue* map) {
1491 Zone* zone = this->zone(); 1491 Zone* zone = this->zone();
1492 Factory* factory = isolate()->factory(); 1492 Factory* factory = isolate()->factory();
1493 Handle<String> map_field_name = factory->map_field_string(); 1493 Handle<String> map_field_name = factory->map_field_string();
1494 HInstruction* store_map = 1494 HInstruction* store_map =
1495 new(zone) HStoreNamedField(object, map_field_name, map, 1495 new(zone) HStoreNamedField(object, map_field_name, map,
1496 true, JSObject::kMapOffset); 1496 true, JSObject::kMapOffset);
1497 store_map->ClearGVNFlag(kChangesInobjectFields);
1497 store_map->SetGVNFlag(kChangesMaps); 1498 store_map->SetGVNFlag(kChangesMaps);
1498 AddInstruction(store_map); 1499 AddInstruction(store_map);
1499 return store_map; 1500 return store_map;
1500 } 1501 }
1501 1502
1502 1503
1503 HInstruction* HGraphBuilder::BuildStoreMap(HValue* object, 1504 HInstruction* HGraphBuilder::BuildStoreMap(HValue* object,
1504 Handle<Map> map) { 1505 Handle<Map> map) {
1505 Zone* zone = this->zone(); 1506 Zone* zone = this->zone();
1506 HValue* map_constant = 1507 HValue* map_constant =
(...skipping 10668 matching lines...) Expand 10 before | Expand all | Expand 10 after
12175 } 12176 }
12176 } 12177 }
12177 12178
12178 #ifdef DEBUG 12179 #ifdef DEBUG
12179 if (graph_ != NULL) graph_->Verify(false); // No full verify. 12180 if (graph_ != NULL) graph_->Verify(false); // No full verify.
12180 if (allocator_ != NULL) allocator_->Verify(); 12181 if (allocator_ != NULL) allocator_->Verify();
12181 #endif 12182 #endif
12182 } 12183 }
12183 12184
12184 } } // namespace v8::internal 12185 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/arm/lithium-codegen-arm.cc ('k') | src/hydrogen-instructions.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698