Refactor bailout reasons and disable optimization in more cases.

Refactor bailout reasons and disable optimization in more cases.

R=jkummerow@chromium.org

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

[Jakob Kummerow, 2014-09-23 16:21:00]

lgtm

https://codereview.chromium.org/596783002/diff/1/src/lithium.cc
File src/lithium.cc (right):

https://codereview.chromium.org/596783002/diff/1/src/lithium.cc#newcode7
src/lithium.cc:7: #include "src/bailout-reason.h"
This shouldn't be necessary as lithium.h includes it already.

[Sven Panne, 2014-09-24 06:00:18]

svenpanne@chromium.org changed reviewers:
+ svenpanne@chromium.org

[Sven Panne, 2014-09-24 06:00:19]

https://codereview.chromium.org/596783002/diff/1/src/lithium.cc
File src/lithium.cc (right):

https://codereview.chromium.org/596783002/diff/1/src/lithium.cc#newcode7
src/lithium.cc:7: #include "src/bailout-reason.h"
On 2014/09/23 16:21:00, Jakob wrote:
> This shouldn't be necessary as lithium.h includes it already.

IWYU...

[Sven Panne, 2014-09-24 06:52:47]

On 2014/09/24 06:00:19, Sven Panne wrote:
> https://codereview.chromium.org/596783002/diff/1/src/lithium.cc
> File src/lithium.cc (right):
> 
> https://codereview.chromium.org/596783002/diff/1/src/lithium.cc#newcode7
> src/lithium.cc:7: #include "src/bailout-reason.h"
> On 2014/09/23 16:21:00, Jakob wrote:
> > This shouldn't be necessary as lithium.h includes it already.
> 
> IWYU...

As discussed offline with Yang, Jakob's comment was OK, because we are in FOO.cc
and FOO.h already includes it. I was confused, because the include of FOO.h
within FOO.cc was not at the top where it should be (to be done).

cpplint actually tries to check the include order (FOO.h, c system, c++ system,
other), but it obviously fails to enforce that FOO.h must come first.

[Yang, 2014-09-24 07:09:01]

Committed patchset #2 (id:20001) manually as 24161 (presubmit successful).