 Chromium Code Reviews
 Chromium Code Reviews Issue 1287383003:
  Re-reland: Remove register index/code indirection  (Closed) 
  Base URL: https://chromium.googlesource.com/v8/v8.git@master
    
  
    Issue 1287383003:
  Re-reland: Remove register index/code indirection  (Closed) 
  Base URL: https://chromium.googlesource.com/v8/v8.git@master| OLD | NEW | 
|---|---|
| 1 // Copyright 2015 the V8 project authors. All rights reserved. | 1 // Copyright 2015 the V8 project authors. All rights reserved. | 
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be | 
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. | 
| 4 | 4 | 
| 5 // Flags: --noflush-optimized-code-cache --allow-natives-syntax | 5 // Flags: --noflush-optimized-code-cache --allow-natives-syntax | 
| 6 | 6 | 
| 7 // The following triggers a GC in SharedFunctionInfo::AddToOptimizedCodeMap. | 7 // The following triggers a GC in SharedFunctionInfo::AddToOptimizedCodeMap. | 
| 8 // Flags: --gc-interval=1234 --gc-global | 8 // Flags: --gc-interval=1234 --gc-global | 
| 9 | 9 | 
| 10 function makeFun() { | 10 function makeFun() { | 
| 11 function fun(osr_fuse) { | 11 function fun(osr_fuse) { | 
| 12 for (var i = 0; i < 3; ++i) { | 12 for (var i = 0; i < 3; ++i) { | 
| 13 if (i == osr_fuse) %OptimizeOsr(); | 13 if (i == osr_fuse) %OptimizeOsr(); | 
| 14 } | 14 } | 
| 15 for (var i = 3; i < 6; ++i) { | 15 for (var i = 3; i < 6; ++i) { | 
| 16 if (i == osr_fuse) %OptimizeOsr(); | 16 if (i == osr_fuse) %OptimizeOsr(); | 
| 17 } | 17 } | 
| 18 } | 18 } | 
| 19 return fun; | 19 return fun; | 
| 20 } | 20 } | 
| 21 | 21 | 
| 22 makeFun()(7); // Warm up. | 22 makeFun()(7); // Warm up. | 
| 
titzer
2015/09/24 09:54:45
What's up here?
 
danno
2015/09/24 10:43:26
The test change was probably a strange merge hiccu
 | |
| 23 makeFun()(4); // Optimize once. | |
| 24 makeFun()(1); // Optimize again. | |
| OLD | NEW |