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

Side by Side Diff: src/ia32/lithium-codegen-ia32.cc

Issue 14996004: Track heap objects. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 7 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-instructions.h ('k') | src/ia32/lithium-ia32.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 4221 matching lines...) Expand 10 before | Expand all | Expand 10 after
4232 if (!IsInteger32(operand_value)) { 4232 if (!IsInteger32(operand_value)) {
4233 DeoptimizeIf(no_condition, instr->environment()); 4233 DeoptimizeIf(no_condition, instr->environment());
4234 } 4234 }
4235 } else { 4235 } else {
4236 Register value = ToRegister(instr->value()); 4236 Register value = ToRegister(instr->value());
4237 __ SmiTag(value); 4237 __ SmiTag(value);
4238 if (!instr->hydrogen()->value()->range()->IsInSmiRange()) { 4238 if (!instr->hydrogen()->value()->range()->IsInSmiRange()) {
4239 DeoptimizeIf(overflow, instr->environment()); 4239 DeoptimizeIf(overflow, instr->environment());
4240 } 4240 }
4241 } 4241 }
4242 } else if (FLAG_track_heap_object_fields && representation.IsHeapObject()) {
4243 if (instr->value()->IsConstantOperand()) {
4244 LConstantOperand* operand_value = LConstantOperand::cast(instr->value());
4245 if (IsInteger32(operand_value)) {
4246 DeoptimizeIf(no_condition, instr->environment());
4247 }
4248 } else {
4249 if (!instr->hydrogen()->value()->type().IsHeapObject()) {
4250 Register value = ToRegister(instr->value());
4251 __ test(value, Immediate(kSmiTagMask));
4252 DeoptimizeIf(zero, instr->environment());
4253 }
4254 }
4242 } else if (FLAG_track_double_fields && representation.IsDouble()) { 4255 } else if (FLAG_track_double_fields && representation.IsDouble()) {
4243 ASSERT(transition.is_null()); 4256 ASSERT(transition.is_null());
4244 ASSERT(instr->is_in_object()); 4257 ASSERT(instr->is_in_object());
4245 ASSERT(!instr->hydrogen()->NeedsWriteBarrier()); 4258 ASSERT(!instr->hydrogen()->NeedsWriteBarrier());
4246 if (CpuFeatures::IsSupported(SSE2)) { 4259 if (CpuFeatures::IsSupported(SSE2)) {
4247 CpuFeatureScope scope(masm(), SSE2); 4260 CpuFeatureScope scope(masm(), SSE2);
4248 XMMRegister value = ToDoubleRegister(instr->value()); 4261 XMMRegister value = ToDoubleRegister(instr->value());
4249 __ movdbl(FieldOperand(object, offset), value); 4262 __ movdbl(FieldOperand(object, offset), value);
4250 } else { 4263 } else {
4251 __ fstp_d(FieldOperand(object, offset)); 4264 __ fstp_d(FieldOperand(object, offset));
(...skipping 2251 matching lines...) Expand 10 before | Expand all | Expand 10 after
6503 FixedArray::kHeaderSize - kPointerSize)); 6516 FixedArray::kHeaderSize - kPointerSize));
6504 __ bind(&done); 6517 __ bind(&done);
6505 } 6518 }
6506 6519
6507 6520
6508 #undef __ 6521 #undef __
6509 6522
6510 } } // namespace v8::internal 6523 } } // namespace v8::internal
6511 6524
6512 #endif // V8_TARGET_ARCH_IA32 6525 #endif // V8_TARGET_ARCH_IA32
OLDNEW
« no previous file with comments | « src/hydrogen-instructions.h ('k') | src/ia32/lithium-ia32.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698