• vogelheim's avatar
    Fix expression positions for for-loops. · de4f3d3e
    vogelheim authored
    FullCodegen generates 2 statement positions for the loop init block, like so:
    
      for(var i = 0; i....
          ^   ^
    
    This change removes the first of those, updates unit tests,
    and removes text expectations for Ignition.
    
    ---
    An alternative would be to emulate the existing behaviour in Ignition, but:
    - The new behaviour seems more logical,
    - Ignition generates no bytecodes for the 'var', meaning there is no code position to attach the break position to.
    
    BUG=v8:4690
    LOG=Y
    
    Review URL: https://codereview.chromium.org/1784883002
    
    Cr-Commit-Position: refs/heads/master@{#34717}
    de4f3d3e
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
fuzzer Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
promises-aplus Loading commit data...
simdjs Loading commit data...
test262 Loading commit data...
unittests Loading commit data...
webkit Loading commit data...
bot_default.gyp Loading commit data...
bot_default.isolate Loading commit data...
default.gyp Loading commit data...
default.isolate Loading commit data...
ignition.gyp Loading commit data...
ignition.isolate Loading commit data...
optimize_for_size.gyp Loading commit data...
optimize_for_size.isolate Loading commit data...
perf.gyp Loading commit data...
perf.isolate Loading commit data...