AbortOptimization, RetryOptimization preserve first bailout reason

AbortOptimization, RetryOptimization preserve first bailout reason

The current logic appears to have been introduced in
https://codereview.chromium.org/596783002.

R=yangguo@chromium.org
BUG=

Committed: https://crrev.com/79a8caec540ad5b8504aa6e387ccf07fb7a75ec1
Cr-Commit-Position: refs/heads/master@{#26043}

[wingo, 2015-01-12 17:48:03]

Yang, does this look right to you?  I get confused when looking at the pipeline
:)

[wingo, 2015-01-12 18:06:50]

https://codereview.chromium.org/836033005/diff/20001/src/compiler.h
File src/compiler.h (left):

https://codereview.chromium.org/836033005/diff/20001/src/compiler.h#oldcode333
src/compiler.h:333: if (bailout_reason_ != kNoReason) bailout_reason_ = reason;
Because bailout_reason_ is initialized to kUnknown and not kNoReason, and
nothing sets bailout_reason_ to kNoReason, that I can see, this condition will
always be true.

[Yang, 2015-01-13 09:49:18]

On 2015/01/12 18:06:50, wingo wrote:
> https://codereview.chromium.org/836033005/diff/20001/src/compiler.h
> File src/compiler.h (left):
> 
> https://codereview.chromium.org/836033005/diff/20001/src/compiler.h#oldcode333
> src/compiler.h:333: if (bailout_reason_ != kNoReason) bailout_reason_ =
reason;
> Because bailout_reason_ is initialized to kUnknown and not kNoReason, and
> nothing sets bailout_reason_ to kNoReason, that I can see, this condition will
> always be true.

lgtm.

[wingo, 2015-01-13 14:37:18]

Committed patchset #3 (id:40001) manually as
79a8caec540ad5b8504aa6e387ccf07fb7a75ec1 (presubmit successful).

[commit-bot: I haz the power, 2015-01-13 14:37:22]

Patchset 3 (id:??) landed as
https://crrev.com/79a8caec540ad5b8504aa6e387ccf07fb7a75ec1
Cr-Commit-Position: refs/heads/master@{#26043}