Move input/output counts directly into Operators, simplying OperatorProperties.

Move input/output counts directly into Operators, simplying OperatorProperties.

This is a first step to refactoring OperatorProperties out of existence.
The next step is to inline OperatorProperties::GetXXXCount into the callers.

R=rossberg@chromium.org
BUG=

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

[rossberg, 2014-10-29 09:43:21]

I love simplying stuff. LGTM, just nits.

https://codereview.chromium.org/680313003/diff/20001/src/compiler/common-oper...
File src/compiler/common-operator.cc (right):

https://codereview.chromium.org/680313003/diff/20001/src/compiler/common-oper...
src/compiler/common-operator.cc:292: static inline size_t
CE(Operator::Properties properties) {
Nit: Make this a static private method of class CallOperator. Or move to
Operator class, so that you don't need multiple implementations.

https://codereview.chromium.org/680313003/diff/20001/src/compiler/operator.h
File src/compiler/operator.h (right):

https://codereview.chromium.org/680313003/diff/20001/src/compiler/operator.h#...
src/compiler/operator.h:54: size_t value_in, size_t effect_in, size_t
control_in,
Using size_t here but int for the getters seems inconsistent. Can you add a
TODO?

[titzer, 2014-10-29 11:59:26]

https://codereview.chromium.org/680313003/diff/20001/src/compiler/common-oper...
File src/compiler/common-operator.cc (right):

https://codereview.chromium.org/680313003/diff/20001/src/compiler/common-oper...
src/compiler/common-operator.cc:292: static inline size_t
CE(Operator::Properties properties) {
On 2014/10/29 09:43:21, rossberg wrote:
> Nit: Make this a static private method of class CallOperator. Or move to
> Operator class, so that you don't need multiple implementations.

Added Operator::ZeroIfPure(Operator::Properties)

https://codereview.chromium.org/680313003/diff/20001/src/compiler/operator.h
File src/compiler/operator.h (right):

https://codereview.chromium.org/680313003/diff/20001/src/compiler/operator.h#...
src/compiler/operator.h:54: size_t value_in, size_t effect_in, size_t
control_in,
On 2014/10/29 09:43:21, rossberg wrote:
> Using size_t here but int for the getters seems inconsistent. Can you add a
> TODO?

Done.

[titzer, 2014-10-29 14:41:12]

Committed patchset #5 manually as 24983.