Revert of abstract_code: return compiled code for compiled shared funcs

Revert of abstract_code: return compiled code for compiled shared funcs (patchset #2 id:20001 of https://codereview.chromium.org/2592703002/ )

Reason for revert:
Breaks tree:

http://build.chromium.org/p/client.v8/builders/V8%20Linux64%20GC%20Stress%20-%20custom%20snapshot/builds/9970

Original issue's description:
> abstract_code: return compiled code for compiled shared funcs
> 
> SharedFunctionInfo's abstract_code was returning the bytecode array
> whenever SharedFunctionInfo had a bytecode array, even if the function
> was compiled (e.g. tiered up to FCG). This meant that abstract_code
> could return code that is not actually the code that will run, which was
> causing problems in profiling as the sampled PC did not match the known
> code offset.
> 
> This patch changes both SharedFunctionInfo and JSFunction to return the
> bytecode if-and-only-if they are not compiled and have a bytecode array
> to return, or they already point to the interpreter trampoline.
> 
> BUG=v8:5758
> 
> Review-Url: https://codereview.chromium.org/2592703002
> Cr-Commit-Position: refs/heads/master@{#41894}
> Committed: https://chromium.googlesource.com/v8/v8/+/679b31c21425ecdd86c40a8a02ac131b72f7bc6b

TBR=bmeurer@chromium.org,mstarzinger@chromium.org,mvstanton@chromium.org,mythria@chromium.org,rmcilroy@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5758
Review-Url: https://codereview.chromium.org/2591223002
Cr-Commit-Position: refs/heads/master@{#41896}
Committed: https://chromium.googlesource.com/v8/v8/+/7c18f1976a86b3a4c8ad815a569f144518cd2c5f

[Leszek Swirski, 2016-12-21 15:41:44]

The CQ bit was checked by leszeks@chromium.org

[Leszek Swirski, 2016-12-21 15:41:45]

Created Revert of abstract_code: return compiled code for compiled shared funcs

[commit-bot: I haz the power, 2016-12-21 15:41:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-21 15:42:07]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482334904953710, "parent_rev":
"7a6138ae11b7e1108bb3ced923993a3dc2def4eb", "commit_rev":
"7c18f1976a86b3a4c8ad815a569f144518cd2c5f"}

[commit-bot: I haz the power, 2016-12-21 15:42:11]

Description was changed from

==========
Revert of abstract_code: return compiled code for compiled shared funcs
(patchset #2 id:20001 of https://codereview.chromium.org/2592703002/ )

Reason for revert:
Breaks tree:

http://build.chromium.org/p/client.v8/builders/V8%20Linux64%20GC%20Stress%20-...

Original issue's description:
> abstract_code: return compiled code for compiled shared funcs
> 
> SharedFunctionInfo's abstract_code was returning the bytecode array
> whenever SharedFunctionInfo had a bytecode array, even if the function
> was compiled (e.g. tiered up to FCG). This meant that abstract_code
> could return code that is not actually the code that will run, which was
> causing problems in profiling as the sampled PC did not match the known
> code offset.
> 
> This patch changes both SharedFunctionInfo and JSFunction to return the
> bytecode if-and-only-if they are not compiled and have a bytecode array
> to return, or they already point to the interpreter trampoline.
> 
> BUG=v8:5758
> 
> Review-Url: https://codereview.chromium.org/2592703002
> Cr-Commit-Position: refs/heads/master@{#41894}
> Committed:
https://chromium.googlesource.com/v8/v8/+/679b31c21425ecdd86c40a8a02ac131b72f...

TBR=bmeurer@chromium.org,mstarzinger@chromium.org,mvstanton@chromium.org,myth...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5758
==========

to

==========
Revert of abstract_code: return compiled code for compiled shared funcs
(patchset #2 id:20001 of https://codereview.chromium.org/2592703002/ )

Reason for revert:
Breaks tree:

http://build.chromium.org/p/client.v8/builders/V8%20Linux64%20GC%20Stress%20-...

Original issue's description:
> abstract_code: return compiled code for compiled shared funcs
> 
> SharedFunctionInfo's abstract_code was returning the bytecode array
> whenever SharedFunctionInfo had a bytecode array, even if the function
> was compiled (e.g. tiered up to FCG). This meant that abstract_code
> could return code that is not actually the code that will run, which was
> causing problems in profiling as the sampled PC did not match the known
> code offset.
> 
> This patch changes both SharedFunctionInfo and JSFunction to return the
> bytecode if-and-only-if they are not compiled and have a bytecode array
> to return, or they already point to the interpreter trampoline.
> 
> BUG=v8:5758
> 
> Review-Url: https://codereview.chromium.org/2592703002
> Cr-Commit-Position: refs/heads/master@{#41894}
> Committed:
https://chromium.googlesource.com/v8/v8/+/679b31c21425ecdd86c40a8a02ac131b72f...

TBR=bmeurer@chromium.org,mstarzinger@chromium.org,mvstanton@chromium.org,myth...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5758

Review-Url: https://codereview.chromium.org/2591223002
Cr-Commit-Position: refs/heads/master@{#41896}
Committed:
https://chromium.googlesource.com/v8/v8/+/7c18f1976a86b3a4c8ad815a569f144518c...
==========

[commit-bot: I haz the power, 2016-12-21 15:42:12]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/7c18f1976a86b3a4c8ad815a569f144518c...