TurboFan: commit dependencies only on update of the opt. code list.

TurboFan: commit dependencies only on update of the opt. code list.

TurboFan compilation was committing dependencies long before the
optimized function made it's way into the optimized code list for
the native context. The problem is that once the code pointer is out
there in dependency arrays, it is eligible for deopt. But the deopt
logic needs the code to be in the optimized code list to fully do it's
job.

BUG=
R=jarin@chromium.org

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

[mvstanton, 2015-05-08 08:29:14]

Hi Jaro, here is the issue we discussed, thx!
--Michael

[Jarin, 2015-05-08 08:34:16]

lgtm with a nit.

https://codereview.chromium.org/1136693003/diff/1/src/deoptimizer.cc
File src/deoptimizer.cc (right):

https://codereview.chromium.org/1136693003/diff/1/src/deoptimizer.cc#newcode500
src/deoptimizer.cc:500: ThreadId::Current().ToInteger(),
reinterpret_cast<void*>(code));
This PrintF should be removed, no?

[mvstanton, 2015-05-08 08:47:39]

doh! Thanks Jaro :)
--Michael

[mvstanton, 2015-05-08 08:49:58]

The CQ bit was checked by mvstanton@chromium.org

[mvstanton, 2015-05-08 08:49:58]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/1136693003/#ps20001
(title: "Removed errant printf.")

[commit-bot: I haz the power, 2015-05-08 08:50:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1136693003/20001

[commit-bot: I haz the power, 2015-05-08 09:13:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-08 09:13:29]

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