[turbofan] Type::Contains() and Constants() is unnecessary.

[turbofan] Type::Contains() and Constants() is unnecessary.

Cleanup.

BUG=

Committed: https://crrev.com/9686d0811e05f416427989b088609643e07c78e9
Cr-Commit-Position: refs/heads/master@{#39830}

[mvstanton, 2016-09-28 13:22:11]

mvstanton@chromium.org changed reviewers:
+ jarin@chromium.org

[mvstanton, 2016-09-28 13:22:12]

Hi Jaro,
Here is a quick cleanup...

[mvstanton, 2016-09-28 13:25:08]

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

[commit-bot: I haz the power, 2016-09-28 13:25:19]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Jarin, 2016-09-28 13:27:02]

lgtm

https://codereview.chromium.org/2379573002/diff/1/src/compiler/types.h
File src/compiler/types.h (left):

https://codereview.chromium.org/2379573002/diff/1/src/compiler/types.h#oldcod...
src/compiler/types.h:545: int NumConstants();
Is this necessary?

[mvstanton, 2016-09-28 13:46:08]

thanks!

https://codereview.chromium.org/2379573002/diff/1/src/compiler/types.h
File src/compiler/types.h (left):

https://codereview.chromium.org/2379573002/diff/1/src/compiler/types.h#oldcod...
src/compiler/types.h:545: int NumConstants();
On 2016/09/28 13:27:02, Jarin wrote:
> Is this necessary?

It gets used by tests, I'd prefer to keep it for now as I make some additional
changes. Then we can remove it, for sure.

[commit-bot: I haz the power, 2016-09-28 13:56:04]

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

[commit-bot: I haz the power, 2016-09-28 13:56:05]

Dry run: This issue passed the CQ dry run.

[mvstanton, 2016-09-28 14:06:40]

The CQ bit was checked by mvstanton@chromium.org

[commit-bot: I haz the power, 2016-09-28 14:06:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-28 14:09:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-28 14:09:37]

Description was changed from

==========
[turbofan] Type::Contains() and Constants() is unnecessary.

Cleanup.

BUG=
==========

to

==========
[turbofan] Type::Contains() and Constants() is unnecessary.

Cleanup.

BUG=

Committed: https://crrev.com/9686d0811e05f416427989b088609643e07c78e9
Cr-Commit-Position: refs/heads/master@{#39830}
==========

[commit-bot: I haz the power, 2016-09-28 14:09:38]

Patchset 1 (id:??) landed as
https://crrev.com/9686d0811e05f416427989b088609643e07c78e9
Cr-Commit-Position: refs/heads/master@{#39830}