[wasm] Int64Lowering of Word64Popcnt.

[wasm] Int64Lowering of Word64Popcnt.

Word64Popcnt is lowered to Word32Popcnt(low-word) + Word32Popcnt(high_word).
Since the optional Word64Popcnt operator does not exist on 32 bit platforms,
I introduced a new operator "Word64PopcntPlaceholder" which is generated
in the WasmCompiler and then lowered in the Int64Lowering.

R=titzer@chromium.org

Committed: https://crrev.com/2e6f3e86078ac28e9974d62f0afc4085a32a2cb0
Cr-Commit-Position: refs/heads/master@{#34777}

[titzer, 2016-03-14 09:50:31]

lgtm

https://codereview.chromium.org/1803453003/diff/1/src/compiler/machine-operat...
File src/compiler/machine-operator.cc (right):

https://codereview.chromium.org/1803453003/diff/1/src/compiler/machine-operat...
src/compiler/machine-operator.cc:474: const Operator*
MachineOperatorBuilder::Word64PopcntPlaceholder() {
Please leave a comment here about why we are doing this, i.e. that we need to
get a reference to a Word64Popcnt operator for later lowering, even though the
machine doesn't support it.

[ahaas, 2016-03-14 12:33:03]

On 2016/03/14 at 09:50:31, titzer wrote:
> lgtm
> 
>
https://codereview.chromium.org/1803453003/diff/1/src/compiler/machine-operat...
> File src/compiler/machine-operator.cc (right):
> 
>
https://codereview.chromium.org/1803453003/diff/1/src/compiler/machine-operat...
> src/compiler/machine-operator.cc:474: const Operator*
MachineOperatorBuilder::Word64PopcntPlaceholder() {
> Please leave a comment here about why we are doing this, i.e. that we need to
get a reference to a Word64Popcnt operator for later lowering, even though the
machine doesn't support it.

Done.

[ahaas, 2016-03-15 10:34:51]

The CQ bit was checked by ahaas@chromium.org

[ahaas, 2016-03-15 10:34:52]

The patchset sent to the CQ was uploaded after l-g-t-m from titzer@chromium.org
Link to the patchset: https://codereview.chromium.org/1803453003/#ps40001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-03-15 10:35:00]

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

[commit-bot: I haz the power, 2016-03-15 10:42:14]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-15 10:42:15]

Description was changed from

==========
[wasm] Int64Lowering of Word64Popcnt.

Word64Popcnt is lowered to Word32Popcnt(low-word) + Word32Popcnt(high_word).
Since the optional Word64Popcnt operator does not exist on 32 bit platforms,
I introduced a new operator "Word64PopcntPlaceholder" which is generated
in the WasmCompiler and then lowered in the Int64Lowering.

R=titzer@chromium.org
==========

to

==========
[wasm] Int64Lowering of Word64Popcnt.

Word64Popcnt is lowered to Word32Popcnt(low-word) + Word32Popcnt(high_word).
Since the optional Word64Popcnt operator does not exist on 32 bit platforms,
I introduced a new operator "Word64PopcntPlaceholder" which is generated
in the WasmCompiler and then lowered in the Int64Lowering.

R=titzer@chromium.org

Committed: https://crrev.com/2e6f3e86078ac28e9974d62f0afc4085a32a2cb0
Cr-Commit-Position: refs/heads/master@{#34777}
==========

[commit-bot: I haz the power, 2016-03-15 10:42:15]

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