Add initial integrated assembler w/ some Xmm ops.

Add initial integrated assembler w/ some Xmm ops.

Add a flag to use the integrated assembler.

Handle simple XMM binary op instructions as an initial example of how
instructions might be handled. This tests fixups in a very limited sense --
Track buffer locations of fixups for floating point immediates.

Patchset one shows the original dart assembler code (revision 39313), so that
it can be diffed.

BUG=none
R=stichnot@chromium.org

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=8acded039857a42b170c1cbc0ad4d1345419de3c

[jvoung (off chromium), 2014-09-20 18:04:39]

Patchset #5 (id:80001) has been deleted

[jvoung (off chromium), 2014-09-20 18:04:47]

Patchset #5 (id:100001) has been deleted

[jvoung (off chromium), 2014-09-20 18:06:21]

jvoung@chromium.org changed reviewers:
+ stichnot@chromium.org

[jvoung (off chromium), 2014-09-20 18:13:35]

Okay, this should be cleaner and more focused than the other CL I sent out
earlier, so hopefully it's reviewable.

The whole fixups structure may still end up changing when we stop using the

.byte ...
.long foo

hacks.

https://codereview.chromium.org/574133002/diff/120001/src/IceRegistersX8632.h
File src/IceRegistersX8632.h (left):

https://codereview.chromium.org/574133002/diff/120001/src/IceRegistersX8632.h...
src/IceRegistersX8632.h:22: class RegX8632 {
Changed.. since I can't do "using RegX8632::GPRRegister" if RegX8632 is a class
instead of a namespace.

https://codereview.chromium.org/574133002/diff/120001/src/assembler_ia32.cpp
File src/assembler_ia32.cpp (right):

https://codereview.chromium.org/574133002/diff/120001/src/assembler_ia32.cpp#...
src/assembler_ia32.cpp:339: EmitUint8(Ty == IceType_f32 ? 0xF3 : 0xF2);
I debated a bunch where to put the Type check. This seemed not-so-awful, since
the compiler is able to turn this into a:

sete ...
or ...
To flip the bit from F2 to F3.

[Jim Stichnoth, 2014-09-22 21:58:13]

LGTM.

https://codereview.chromium.org/574133002/diff/120001/src/IceInstX8632.cpp
File src/IceInstX8632.cpp (right):

https://codereview.chromium.org/574133002/diff/120001/src/IceInstX8632.cpp#ne...
src/IceInstX8632.cpp:490: (Asm->*(Emitter.XmmXmm))(Ty, VarReg, SrcReg);
My head is exploding trying to figure out what ->* means...

https://codereview.chromium.org/574133002/diff/120001/src/IceMemoryRegion.cpp
File src/IceMemoryRegion.cpp (right):

https://codereview.chromium.org/574133002/diff/120001/src/IceMemoryRegion.cpp...
src/IceMemoryRegion.cpp:7: //===- subzero/src/IceMemoryRegion.cpp - Memory
region ----------*- C++ -*-===//
Shouldn't need the ... C++ ... part

[jvoung (off chromium), 2014-09-23 00:26:16]

Thanks!

https://codereview.chromium.org/574133002/diff/120001/src/IceInstX8632.cpp
File src/IceInstX8632.cpp (right):

https://codereview.chromium.org/574133002/diff/120001/src/IceInstX8632.cpp#ne...
src/IceInstX8632.cpp:490: (Asm->*(Emitter.XmmXmm))(Ty, VarReg, SrcReg);
On 2014/09/22 21:58:13, stichnot wrote:
> My head is exploding trying to figure out what ->* means...

Acknowledged.

https://codereview.chromium.org/574133002/diff/120001/src/IceMemoryRegion.cpp
File src/IceMemoryRegion.cpp (right):

https://codereview.chromium.org/574133002/diff/120001/src/IceMemoryRegion.cpp...
src/IceMemoryRegion.cpp:7: //===- subzero/src/IceMemoryRegion.cpp - Memory
region ----------*- C++ -*-===//
On 2014/09/22 21:58:13, stichnot wrote:
> Shouldn't need the ... C++ ... part

Done.

[jvoung (off chromium), 2014-09-23 01:02:35]

Committed patchset #10 (id:220001) manually as 8acded0 (presubmit successful).