 Chromium Code Reviews
 Chromium Code Reviews Issue 12385014:
  Hydrogen stubs for array constructors  (Closed) 
  Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
    
  
    Issue 12385014:
  Hydrogen stubs for array constructors  (Closed) 
  Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge| Index: src/ia32/macro-assembler-ia32.cc | 
| diff --git a/src/ia32/macro-assembler-ia32.cc b/src/ia32/macro-assembler-ia32.cc | 
| index 3228e8370e1ea9848ddbe0885462234784e885ce..46f75b5a9be412a440a97058ef5c91c3fbd88712 100644 | 
| --- a/src/ia32/macro-assembler-ia32.cc | 
| +++ b/src/ia32/macro-assembler-ia32.cc | 
| @@ -2531,10 +2531,13 @@ void MacroAssembler::VerifyX87StackDepth(uint32_t depth) { | 
| and_(eax, kTopMask); | 
| shr(eax, 11); | 
| cmp(eax, Immediate(tos)); | 
| - Label all_ok; | 
| - j(equal, &all_ok); | 
| + // These lines if you want to break right away | 
| + // Label all_ok; | 
| + // j(equal, &all_ok); | 
| + // int3(); | 
| 
Hannes Payer (out of office)
2013/04/18 11:14:39
is this debugging leftover code?
 
mvstanton
2013/04/18 13:39:26
Yes and no, it is actually a bug because the j(equ
 | 
| + // The check line otherwise | 
| Check(equal, "Unexpected FPU stack depth after instruction"); | 
| - bind(&all_ok); | 
| + // bind(&all_ok); | 
| fnclex(); | 
| pop(eax); | 
| } |