[turbofan] Change RawMachineAssembler to take a CallDescriptor instead of a MachineSignature.

[turbofan] Change RawMachineAssembler to take a CallDescriptor instead of a MachineSignature.

The InterpreterAssembler needs to specify a specific CallDescriptor type
instead of using the SimplifiedCDescriptor type. This CL makes it possible
to specify the CallDescriptor used by the RawMachineAssembler instead of
specifying a MachineSignature.

Also removes instruction-selector-tester.h which was erroneously resurrected
at some point.

BUG=v8:4280
LOG=N

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

[rmcilroy, 2015-07-15 16:57:00]

rmcilroy@chromium.org changed reviewers:
+ bmeurer@chromium.org

[rmcilroy, 2015-07-15 16:57:00]

Benedikt: this change is required for
https://codereview.chromium.org/1239793002/. Could you please take a look?
Thanks.

[rmcilroy, 2015-07-20 13:48:17]

On 2015/07/15 16:57:00, rmcilroy wrote:
> Benedikt: this change is required for
> https://codereview.chromium.org/1239793002/. Could you please take a look?
> Thanks.

Ping Benedikt?

[rmcilroy, 2015-07-21 11:14:16]

rmcilroy@chromium.org changed reviewers:
+ mstarzinger@chromium.org
- bmeurer@chromium.org

[rmcilroy, 2015-07-21 11:14:17]

Michael, since Benedikt is OOO could you take a look at this please? It is a
precursor to the interpreter framework CL.

[Michael Starzinger, 2015-07-21 11:46:23]

LGTM with one comment to address.

https://codereview.chromium.org/1235313002/diff/1/test/cctest/compiler/instru...
File test/cctest/compiler/instruction-selector-tester.h (right):

https://codereview.chromium.org/1235313002/diff/1/test/cctest/compiler/instru...
test/cctest/compiler/instruction-selector-tester.h:46: machine_sig)) {}
There seems to be a syntax error here. Also this entire file is actually dead
code, and was erroneously resurrected recently. Let's delete it please.

[rmcilroy, 2015-07-21 15:16:41]

Patchset #2 (id:20001) has been deleted

[rmcilroy, 2015-07-21 15:16:46]

The CQ bit was checked by rmcilroy@chromium.org

[rmcilroy, 2015-07-21 15:16:46]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1235313002/#ps40001
(title: "instruction-selector-tester.h must die!")

[commit-bot: I haz the power, 2015-07-21 15:16:54]

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

[rmcilroy, 2015-07-21 15:16:55]

https://codereview.chromium.org/1235313002/diff/1/test/cctest/compiler/instru...
File test/cctest/compiler/instruction-selector-tester.h (right):

https://codereview.chromium.org/1235313002/diff/1/test/cctest/compiler/instru...
test/cctest/compiler/instruction-selector-tester.h:46: machine_sig)) {}
On 2015/07/21 11:46:23, Michael Starzinger wrote:
> There seems to be a syntax error here. Also this entire file is actually dead
> code, and was erroneously resurrected recently. Let's delete it please.

Done.

[commit-bot: I haz the power, 2015-07-21 15:54:20]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-07-21 15:54:36]

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