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

Side by Side Diff: src/hydrogen-instructions.cc

Issue 6260035: Fix a bug in the placement of minus-zero checks and in GVN.... (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: '' Created 9 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/hydrogen.cc ('k') | test/mjsunit/compiler/regress-1085.js » ('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 2010 the V8 project authors. All rights reserved. 1 // Copyright 2010 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 235 matching lines...) Expand 10 before | Expand all | Expand 10 after
246 } 246 }
247 } 247 }
248 return false; 248 return false;
249 } 249 }
250 250
251 251
252 bool HValue::Equals(HValue* other) const { 252 bool HValue::Equals(HValue* other) const {
253 if (other->opcode() != opcode()) return false; 253 if (other->opcode() != opcode()) return false;
254 if (!other->representation().Equals(representation())) return false; 254 if (!other->representation().Equals(representation())) return false;
255 if (!other->type_.Equals(type_)) return false; 255 if (!other->type_.Equals(type_)) return false;
256 if (other->flags() != flags()) return false;
256 if (OperandCount() != other->OperandCount()) return false; 257 if (OperandCount() != other->OperandCount()) return false;
257 for (int i = 0; i < OperandCount(); ++i) { 258 for (int i = 0; i < OperandCount(); ++i) {
258 if (OperandAt(i)->id() != other->OperandAt(i)->id()) return false; 259 if (OperandAt(i)->id() != other->OperandAt(i)->id()) return false;
259 } 260 }
260 bool result = DataEquals(other); 261 bool result = DataEquals(other);
261 ASSERT(!result || Hashcode() == other->Hashcode()); 262 ASSERT(!result || Hashcode() == other->Hashcode());
262 return result; 263 return result;
263 } 264 }
264 265
265 266
(...skipping 1182 matching lines...) Expand 10 before | Expand all | Expand 10 after
1448 1449
1449 1450
1450 void HCheckPrototypeMaps::Verify() { 1451 void HCheckPrototypeMaps::Verify() {
1451 HInstruction::Verify(); 1452 HInstruction::Verify();
1452 ASSERT(HasNoUses()); 1453 ASSERT(HasNoUses());
1453 } 1454 }
1454 1455
1455 #endif 1456 #endif
1456 1457
1457 } } // namespace v8::internal 1458 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/hydrogen.cc ('k') | test/mjsunit/compiler/regress-1085.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698