X87: [compiler] Mark shared functions for optimization.

X87: [compiler] Mark shared functions for optimization.

  port 4a31323e973e0a03403a53c601dfd4f0237532e8 (r40506)

  original commit message:
  The current method of marking functions for optimization, which replaces
  the JSFunction's code object with one that triggers optimization, would
  never allow unnamed functions to be optimized. This is an issue for a
  style of programming which heavily relies on passing around closures.

  This patch sets a bit on the SharedFunctionInfo when a JSFunction is
  marked. When another JSFunction referring to the same SharedFunctionInfo
  is lazily compiled, it immediately triggers a non-concurrent optimize.

BUG=

Committed: https://crrev.com/ac8318e2e03b4ddb0c7fb82a71cb37cfcc72c1fc
Cr-Commit-Position: refs/heads/master@{#40520}

[zhengxing.li, 2016-10-24 02:47:14]

zhengxing.li@intel.com changed reviewers:
+ chunyang.dai@intel.com, leszeks@chromium.org, weiliang.lin@intel.com

[zhengxing.li, 2016-10-24 02:47:14]

PTAL, thanks!

[Weiliang, 2016-10-24 04:32:14]

lgtm

[zhengxing.li, 2016-10-24 06:15:52]

The CQ bit was checked by zhengxing.li@intel.com

[commit-bot: I haz the power, 2016-10-24 06:16:02]

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-10-24 06:42:56]

Committed patchset #1 (id:1)

[Leszek Swirski, 2016-10-24 14:46:31]

Thanks for catching and fixing this.

[commit-bot: I haz the power, 2016-11-17 22:10:31]

Description was changed from

==========
X87: [compiler] Mark shared functions for optimization.

  port 4a31323e973e0a03403a53c601dfd4f0237532e8 (r40506)

  original commit message:
  The current method of marking functions for optimization, which replaces
  the JSFunction's code object with one that triggers optimization, would
  never allow unnamed functions to be optimized. This is an issue for a
  style of programming which heavily relies on passing around closures.

  This patch sets a bit on the SharedFunctionInfo when a JSFunction is
  marked. When another JSFunction referring to the same SharedFunctionInfo
  is lazily compiled, it immediately triggers a non-concurrent optimize.

BUG=
==========

to

==========
X87: [compiler] Mark shared functions for optimization.

  port 4a31323e973e0a03403a53c601dfd4f0237532e8 (r40506)

  original commit message:
  The current method of marking functions for optimization, which replaces
  the JSFunction's code object with one that triggers optimization, would
  never allow unnamed functions to be optimized. This is an issue for a
  style of programming which heavily relies on passing around closures.

  This patch sets a bit on the SharedFunctionInfo when a JSFunction is
  marked. When another JSFunction referring to the same SharedFunctionInfo
  is lazily compiled, it immediately triggers a non-concurrent optimize.

BUG=

Committed: https://crrev.com/ac8318e2e03b4ddb0c7fb82a71cb37cfcc72c1fc
Cr-Commit-Position: refs/heads/master@{#40520}
==========

[commit-bot: I haz the power, 2016-11-17 22:10:32]

Patchset 1 (id:??) landed as
https://crrev.com/ac8318e2e03b4ddb0c7fb82a71cb37cfcc72c1fc
Cr-Commit-Position: refs/heads/master@{#40520}