Minor compiler pipeline refactoring. Inline UpdateSharedFunctionInfo and make Parser::Parse respons…

Minor compiler pipeline refactoring. Inline UpdateSharedFunctionInfo and make Parser::Parse responsible for setting the strict mode of the CompilationInfo.

R=mstarzinger@chromium.org
BUG=

Committed: https://code.google.com/p/v8/source/detail?r=24001

[Michael Starzinger, 2014-09-17 12:10:10]

LGTM (if at least the first comment is addressed).

https://codereview.chromium.org/555553003/diff/1/src/compiler.cc
File src/compiler.cc (left):

https://codereview.chromium.org/555553003/diff/1/src/compiler.cc#oldcode820
src/compiler.cc:820: info.SetStrictMode(info.function()->strict_mode());
This removal also applies to the following files:
- src/compiler/js-inlining.cc
- test/cctest/compiler/test-changes-lowering.cc
- test/cctest/compiler/function-tester.h
- test/cctest/compiler/test-pipeline.cc
- test/cctest/compiler/test-codegen-deopt.cc

https://codereview.chromium.org/555553003/diff/1/test/mjsunit/compiler/osr-wa...
File test/mjsunit/compiler/osr-warm.js (right):

https://codereview.chromium.org/555553003/diff/1/test/mjsunit/compiler/osr-wa...
test/mjsunit/compiler/osr-warm.js:38: assertEquals(0, f1(200000));
Does this make it possible to remove all the exceptions (I count two) for
"compiler/osr-warm" from the mjsunit.status file?

[titzer, 2014-09-17 12:18:17]

https://codereview.chromium.org/555553003/diff/1/src/compiler.cc
File src/compiler.cc (left):

https://codereview.chromium.org/555553003/diff/1/src/compiler.cc#oldcode820
src/compiler.cc:820: info.SetStrictMode(info.function()->strict_mode());
On 2014/09/17 12:10:10, Michael Starzinger wrote:
> This removal also applies to the following files:
> - src/compiler/js-inlining.cc
> - test/cctest/compiler/test-changes-lowering.cc
> - test/cctest/compiler/function-tester.h
> - test/cctest/compiler/test-pipeline.cc
> - test/cctest/compiler/test-codegen-deopt.cc

Done.

https://codereview.chromium.org/555553003/diff/1/test/mjsunit/compiler/osr-wa...
File test/mjsunit/compiler/osr-warm.js (right):

https://codereview.chromium.org/555553003/diff/1/test/mjsunit/compiler/osr-wa...
test/mjsunit/compiler/osr-warm.js:38: assertEquals(0, f1(200000));
On 2014/09/17 12:10:10, Michael Starzinger wrote:
> Does this make it possible to remove all the exceptions (I count two) for
> "compiler/osr-warm" from the mjsunit.status file?

Done.

[titzer, 2014-09-17 12:35:00]

Committed patchset #2 manually as 24001 (presubmit successful).