i#1551 Port DynamoRIO to ARM: update instr_shared.c to build for ARM

i#1551 Port DynamoRIO to ARM: update instr_shared.c to build for ARM
- add INSTR_CREATE_mov(), INSTR_CREATE_load_mm(),
  INSTR_CREATE_store_mm(), and INSTR_CREATE_jmp_ind_mem().
- add OP_load and OP_store
- update instr_shared.c to build for ARM

R=bruening@google.com
BUG=1551

Committed: https://code.google.com/p/dynamorio/source/detail?r=2937

[bruening, 2014-11-03 16:56:26]

commit msg looks like it has some mistakes: INSTR_CREATE_mov and
INSTR_CREATE_load_mm are listed twice

[zhaoqin, 2014-11-03 17:09:05]

On 2014/11/03 16:56:26, bruening wrote:
> commit msg looks like it has some mistakes: INSTR_CREATE_mov and
> INSTR_CREATE_load_mm are listed twice

updated

[bruening, 2014-11-04 03:14:07]

On 2014/11/03 17:09:05, zhaoqin wrote:
> On 2014/11/03 16:56:26, bruening wrote:
> > commit msg looks like it has some mistakes: INSTR_CREATE_mov and
> > INSTR_CREATE_load_mm are listed twice
> 
> updated

INSTR_CREATE_mov is still listed twice

[zhaoqin, 2014-11-04 03:16:44]

On 2014/11/04 03:14:07, bruening wrote:
> On 2014/11/03 17:09:05, zhaoqin wrote:
> > On 2014/11/03 16:56:26, bruening wrote:
> > > commit msg looks like it has some mistakes: INSTR_CREATE_mov and
> > > INSTR_CREATE_load_mm are listed twice
> > 
> > updated
> 
> INSTR_CREATE_mov is still listed twice

updated

[bruening, 2014-11-04 19:20:56]

LGTM if the 4 spelling errors in user-facing docs are fixed

https://codereview.appspot.com/165330043/diff/1/core/arch/arm/instr_create.h
File core/arch/arm/instr_create.h (right):

https://codereview.appspot.com/165330043/diff/1/core/arch/arm/instr_create.h#...
core/arch/arm/instr_create.h:114: #define INSTR_CREATE_load_mm(dc, r, m)
INSTR_CREATE_vldr(dc, r, m)
vldr doesn't seem to take 128-bit regs -- so I'm not sure how to load/store
those.

https://codereview.appspot.com/165330043/diff/1/core/arch/arm/opcode.h
File core/arch/arm/opcode.h (right):

https://codereview.appspot.com/165330043/diff/1/core/arch/arm/opcode.h#newcod...
core/arch/arm/opcode.h:398: #define OP_store     OP_str    /**<
Platform-independend opcode name for store. */
spelling, on both lines

https://codereview.appspot.com/165330043/diff/1/core/arch/x86/opcode.h
File core/arch/x86/opcode.h (right):

https://codereview.appspot.com/165330043/diff/1/core/arch/x86/opcode.h#newcod...
core/arch/x86/opcode.h:1344: #define OP_store  OP_mov_st    /**<
Platform-independend opcode name for store. */
2 spelling errors again

[bruening, 2014-11-04 19:20:56]

LGTM if the 4 spelling errors in user-facing docs are fixed

https://codereview.appspot.com/165330043/diff/1/core/arch/arm/instr_create.h
File core/arch/arm/instr_create.h (right):

https://codereview.appspot.com/165330043/diff/1/core/arch/arm/instr_create.h#...
core/arch/arm/instr_create.h:114: #define INSTR_CREATE_load_mm(dc, r, m)
INSTR_CREATE_vldr(dc, r, m)
vldr doesn't seem to take 128-bit regs -- so I'm not sure how to load/store
those.

https://codereview.appspot.com/165330043/diff/1/core/arch/arm/opcode.h
File core/arch/arm/opcode.h (right):

https://codereview.appspot.com/165330043/diff/1/core/arch/arm/opcode.h#newcod...
core/arch/arm/opcode.h:398: #define OP_store     OP_str    /**<
Platform-independend opcode name for store. */
spelling, on both lines

https://codereview.appspot.com/165330043/diff/1/core/arch/x86/opcode.h
File core/arch/x86/opcode.h (right):

https://codereview.appspot.com/165330043/diff/1/core/arch/x86/opcode.h#newcod...
core/arch/x86/opcode.h:1344: #define OP_store  OP_mov_st    /**<
Platform-independend opcode name for store. */
2 spelling errors again

[zhaoqin, 2014-11-04 20:46:28]

Committed patchset #1 (id:1) manually as 2937 (presubmit successful).