[Interpreter] Move BytecodeHasHandler() from Interpreter to Bytecodes.

[Interpreter] Move BytecodeHasHandler() from Interpreter to Bytecodes.

BUG=v8:4280
LOG=N

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

[Stefano Sanfilippo, 2016-03-24 15:42:51]

ssanfilippo@chromium.org changed reviewers:
+ oth@chromium.org

[oth, 2016-03-24 15:50:35]

lgtm, one minor comment.

https://codereview.chromium.org/1830593002/diff/1/src/interpreter/bytecodes.h
File src/interpreter/bytecodes.h (right):

https://codereview.chromium.org/1830593002/diff/1/src/interpreter/bytecodes.h...
src/interpreter/bytecodes.h:515: // Returns true if a handler is generated for a
bytecode at a given
Could we also say something like "All bytecodes have handlers at
OperandScale::kSingle, but only bytecodes with scalable operands have handlers
with larger OperandScale values."

[Stefano Sanfilippo, 2016-03-24 16:07:15]

https://codereview.chromium.org/1830593002/diff/1/src/interpreter/bytecodes.h
File src/interpreter/bytecodes.h (right):

https://codereview.chromium.org/1830593002/diff/1/src/interpreter/bytecodes.h...
src/interpreter/bytecodes.h:515: // Returns true if a handler is generated for a
bytecode at a given
On 2016/03/24 15:50:35, oth wrote:
> Could we also say something like "All bytecodes have handlers at
> OperandScale::kSingle, but only bytecodes with scalable operands have handlers
> with larger OperandScale values."

Done.

[Stefano Sanfilippo, 2016-03-24 16:17:00]

The CQ bit was checked by ssanfilippo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-24 16:17:05]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1830593002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1830593002/40001

[commit-bot: I haz the power, 2016-03-24 17:12:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-24 17:12:32]

Dry run: This issue passed the CQ dry run.

[Stefano Sanfilippo, 2016-04-05 09:42:19]

The CQ bit was checked by ssanfilippo@chromium.org

[Stefano Sanfilippo, 2016-04-05 09:42:20]

The patchset sent to the CQ was uploaded after l-g-t-m from oth@chromium.org
Link to the patchset: https://codereview.chromium.org/1830593002/#ps40001
(title: "Expand function doc.")

[commit-bot: I haz the power, 2016-04-05 09:42:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1830593002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1830593002/40001

[commit-bot: I haz the power, 2016-04-05 10:05:40]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-05 10:06:30]

Description was changed from

==========
[Interpreter] Move BytecodeHasHandler() from Interpreter to Bytecodes.

BUG=v8:4280
LOG=N
==========

to

==========
[Interpreter] Move BytecodeHasHandler() from Interpreter to Bytecodes.

BUG=v8:4280
LOG=N

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

[commit-bot: I haz the power, 2016-04-05 10:06:31]

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