am 50ca506e: Merge "Inlining synthetic accessors."

* commit '50ca506e9191c2eafcf9564528712020f9531dc4':
  Inlining synthetic accessors.

Merge "Inlining synthetic accessors."

am 12c0273f: Merge "Use trampolines for calls to helpers"

* commit '12c0273fd394d72ae93424784b1450daea417472':
  Use trampolines for calls to helpers

Merge "Use trampolines for calls to helpers"

am 1415359a: Merge "Inlining setters that return one of their arguments."

* commit '1415359a4ca80a78507173103274a06cd85d223d':
  Inlining setters that return one of their arguments.

Merge "Inlining setters that return one of their arguments."

Inlining synthetic accessors.

Bug: 11549140
Change-Id: I0d6a38c51887f55563414c240ac42ee64bdb5426

Inlining setters that return one of their arguments.

Useful for builder classes and synthetic setters. The latter
are not inlined yet since they are static methods, i.e. they
don't use "this" as the object for IPUT.

Change-Id: I3d34156c3629b837a0a95ccc34a8aae3f4a636d5

am 427ca38b: Merge "Add support for taking parameters in optimizing compiler."

* commit '427ca38b0a6c6fd7dc0dbb380619e2b91b56cf1c':
  Add support for taking parameters in optimizing compiler.

Merge "Add support for taking parameters in optimizing compiler."

Add support for taking parameters in optimizing compiler.

- Fix stack layout to mimic Quick's.
- Implement some sub operations.

Change-Id: I8cf75a4d29b662381a64f02c0bc61d859482fc4e

am 7ab4e5c5: Merge "JDWP request is parsed incorrectly due to junk in the descriptor"

* commit '7ab4e5c5288e04b7beb6d8ddfd5e8bf878002732':
  JDWP request is parsed incorrectly due to junk in the descriptor

Merge "JDWP request is parsed incorrectly due to junk in the descriptor"

JDWP request is parsed incorrectly due to junk in the descriptor

The ClassHelper object is created and deleted on the same line.
Therefore it is wrong to refer to any class members of the object in
the next line. Any references to the object and the object itself
must have the same live ranges.

Change-Id: I792d7baa3828b0389254f2941705e006cab092cd
Signed-off-by: nikolay serdjuk <nikolay.y.serdjuk@intel.com>

am b70b09f6: Merge "Revert "Inlining setters that return one of their arguments.""

* commit 'b70b09f6cc1f1fbbf604da7b6b1ea5f74e43571c':
  Revert "Inlining setters that return one of their arguments."

am 90247d17: Merge "Revert "Inlining synthetic accessors.""

* commit '90247d17f7db0052c20515d363fe505be4fcaf89':
  Revert "Inlining synthetic accessors."

Merge "Revert "Inlining setters that return one of their arguments.""

Merge "Revert "Inlining synthetic accessors.""

Revert "Inlining setters that return one of their arguments."

Bug: 13817614

This reverts commit 8e40c3e662d852da87b6bcfe79355f96ab9e91c7.

Change-Id: Ia7ed2d933514781150b2e0b9855244e383c60187

Revert "Inlining synthetic accessors."

This reverts commit 505ebb0e7b42e7facc8354515b06333ee8b84b10.

am 818b4cf7: Merge "Fix wrong handling of Generic JNI not finding native method."

* commit '818b4cf753d6e8a294f0fbe2e3bd9168eea9f762':
  Fix wrong handling of Generic JNI not finding native method.

Merge "Fix wrong handling of Generic JNI not finding native method."

Fix wrong handling of Generic JNI not finding native method.

Code did not properly call JNIMethodEnd, such that locks etc
where not correctly handled.

Add a test case to jni_compiler_test.

Change-Id: If2d5c628517d65a56dd6bb5c4cabdff77c7664a1

am e8b4bbce: Merge "Fix using kArm instead of kThumb2."

* commit 'e8b4bbcef44f7fe76896850b2f5750cd46b1ade0':
  Fix using kArm instead of kThumb2.

Merge "Fix using kArm instead of kThumb2."

Fix using kArm instead of kThumb2.

Change-Id: I0f3bf93b38d910bc57d017693675829b6a39bc18

Use trampolines for calls to helpers

This is an ARM specific optimization to the compiler
that uses trampoline islands to make calls to runtime
helper functions.  The intention is to reduce the size
of the generated code (by 2 bytes per call) without
affecting performance.

By default this is on when generating an OAT file.  It is
off when compiling to memory.

To switch this off in dex2oat, use the command line option:
--no-helper-trampolines

Enhances disassembler to print the trampoline entry on the
BL instruction like this:

0xb6a850c0: f7ffff9e  bl      -196 (0xb6a85000)  ; pTestSuspend

Bug: 12607709
Change-Id: I9202bdb7cf21252ad807bd48701f1f6ce8e3d0fe

am 97a332b4: Merge "art_quick_lock_object uses registers incorrectly"

* commit '97a332b4476d5a2b4ad0650dacc6bfcff882fc57':
  art_quick_lock_object uses registers incorrectly

Merge "art_quick_lock_object uses registers incorrectly"

am 45157a41: Merge "ARM: enable optimisation for easy multiply, add modulus pattern."

* commit '45157a41b6c0ac9f73aeeb1f064c2270a6a68a60':
  ARM: enable optimisation for easy multiply, add modulus pattern.

am 312d70f1: Merge "Check the machine type of an ELF file when loading."

* commit '312d70f1f01ff0e8ef7dfd0ccb4a1c360c74d0c4':
  Check the machine type of an ELF file when loading.

Merge "ARM: enable optimisation for easy multiply, add modulus pattern."

Merge "Check the machine type of an ELF file when loading."

Check the machine type of an ELF file when loading.

This ensures that we reject wrong target ELF files,
and should result in a recompile for the right
target.

Change-Id: I898dddc4f2bb9b1607a7436083d0ba7619b6007b

am ebe5250f: Merge "Improve invalid root dumping."

* commit 'ebe5250f6162515c999f7966a918b1f86bb67396':
  Improve invalid root dumping.

Merge "Improve invalid root dumping."

Improve invalid root dumping.

The invalid root dumping now attempts to print the root type.

Change-Id: Ie821296d569f34909ba6e2705f5c347cd2143a3a

am 09379fd9: Merge "Disable use of R4 as a promotable register"

* commit '09379fd9f20e25ee71687e2c60f6a84c9ede8cd6':
  Disable use of R4 as a promotable register

am 54f192ca: Merge "Inlining synthetic accessors."

* commit '54f192cac80f6dd323be29bd7192898626ba2b01':
  Inlining synthetic accessors.

Merge "Disable use of R4 as a promotable register"

Merge "Inlining synthetic accessors."

am 34df691e: Merge "Inlining setters that return one of their arguments."

* commit '34df691e64a6491e7312a850a1ad255e4cf02702':
  Inlining setters that return one of their arguments.

Merge "Inlining setters that return one of their arguments."

Inlining synthetic accessors.

Bug: 11549140
Change-Id: Ie0034a6840b1beaa3df92f26bf9d315119c81e34

am 6cea2a6c: Merge "Re-enable DEX-to-DEX compilation path"

* commit '6cea2a6c42d3a4871e3b64fe55abcb5ffc8f09de':
  Re-enable DEX-to-DEX compilation path

Inlining setters that return one of their arguments.

Useful for builder classes and synthetic setters. The latter
are not inlined yet since they are static methods, i.e. they
don't use "this" as the object for IPUT.

Change-Id: I946cf570195be83ecec5fb32851bcaefad45f8a5

Merge "Re-enable DEX-to-DEX compilation path"

Re-enable DEX-to-DEX compilation path

Bug: 13807032
Change-Id: I6de7adb81a0f955a93257b83db9ad689ebcd90ad

am 764c798b: Merge "Fix LoadValueWide to not call MarkLive for high reg that is equal to low reg"

* commit '764c798b57b7baa04885fcdf5eba5708a01b9edc':
  Fix LoadValueWide to not call MarkLive for high reg that is equal to low reg

Merge "Fix LoadValueWide to not call MarkLive for high reg that is equal to low reg"

am 09dfce73: Merge "Clean up special method inlining."

* commit '09dfce73c4e849afda37035146dd2932220708d3':
  Clean up special method inlining.

Merge "Clean up special method inlining."

Clean up special method inlining.

Mark inlined getter/setter INVOKEs as NOP to allow implicit
null checks (SIGSEGV-based) rather than the explicit checks
in GenInvoke().

Avoid inlining wide setter and returning wide argument if
the wide source is not in consecutive dalvik registers in
INVOKE. This is valid dalvik bytecode and we should treat it
correctly even if we're currently unaware of any tools that
would generate such INVOKEs.

Remove bogus MIR_INLINED checks from LVN.

Change-Id: I7e75a832fcf9bd0550e21b1c8b3813c6166197dd

am 3734bffd: Merge "Use target-specific word instead of runtime word."

* commit '3734bffdf77e2bc293eb9722f9bd4389fbaa0d92':
  Use target-specific word instead of runtime word.

Merge "Use target-specific word instead of runtime word."

Use target-specific word instead of runtime word.

Change-Id: Ia11dc3cc520a1a5c7bd017013e5699af9570ce91

am 7efad5d3: Merge "Disassemble Thumb2 shifts and more VFP instructions."

* commit '7efad5d3a806a15166109837439f2e149031feef':
  Disassemble Thumb2 shifts and more VFP instructions.

am 885e813e: Merge "Add support for new-instance and invoke-direct."

* commit '885e813edc121cd8392eb066981522424480253c':
  Add support for new-instance and invoke-direct.

am 6f31ce38: Merge "Please clang by making explicit instantiations non-static."

* commit '6f31ce382777d82952d019908866ff28ef3bbc76':
  Please clang by making explicit instantiations non-static.

am 839f43a4: Merge "Explicitly instantiate artInvokeCommon to please analysis tool."

* commit '839f43a40c9fe170266f42a9726c02195c73d043':
  Explicitly instantiate artInvokeCommon to please analysis tool.

am 541dd231: Merge "Insert an empty read barrier call."

* commit '541dd231f9a7282f7fd1092349a322dc67bafaf2':
  Insert an empty read barrier call.

Merge "Disassemble Thumb2 shifts and more VFP instructions."

Merge "Add support for new-instance and invoke-direct."

Merge "Please clang by making explicit instantiations non-static."

Please clang by making explicit instantiations non-static.

Change-Id: I4112bf62c69b4f28ca369c4bf3c626be43692afd

Add support for new-instance and invoke-direct.

Change-Id: I2daed646904f7711972a7da15d88be7573426932

Merge "Explicitly instantiate artInvokeCommon to please analysis tool."

Explicitly instantiate artInvokeCommon to please analysis tool.

Change-Id: I5c078a197c5026e9bd886d0ba1e188054b5ffa75

Merge "Insert an empty read barrier call."

Insert an empty read barrier call.

Bug: 12687968
Change-Id: Ie1d28658e16e09f6a983cb5c1f0d5b375b7ae069

am e9236f03: Merge "Fix Object::Clone()\'s pre-fence barrier."

* commit 'e9236f03d8c4348a8f1a8767baa1aea60ecdc1f5':
  Fix Object::Clone()'s pre-fence barrier.

Merge "Fix Object::Clone()'s pre-fence barrier."

art_quick_lock_object uses registers incorrectly

x86 implementation of art_quick_lock_object uses registers
incorrectly in the case of .Lalready_thin. eax points to object
while it is used as holding the lock word.

The patch fixes this.

Change-Id: Iacf1a40c6570fae78a5504ca3b2f1218631f7a8f
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

am e5893f8f: Merge "Restore the args in the invoke trampoline slow paths."

* commit 'e5893f8fb70c58fe9950c1fc8b1023e32ca34637':
  Restore the args in the invoke trampoline slow paths.

Merge "Restore the args in the invoke trampoline slow paths."

am 03bafa4b: Merge "Do not output ARM64 assembler debug code."

* commit '03bafa4b80e130664295a8eada93d20bc6f6cdeb':
  Do not output ARM64 assembler debug code.

am 77093dc3: Merge "Make frame size dependent on arch-specific constants."

* commit '77093dc31ddce272e7856a20ece595bf5554e0a6':
  Make frame size dependent on arch-specific constants.

am 0666dcbc: Merge "Add OpEndIT() for marking the end of OpIT blocks"

* commit '0666dcbc600db012760218247aed24c5eff36128':
  Add OpEndIT() for marking the end of OpIT blocks

am 0537c5ea: Merge "Fix off-by-1 error in new SpaceBitmap"

* commit '0537c5ea92a4fb60a04024cc2b7247f08a3d9096':
  Fix off-by-1 error in new SpaceBitmap

am 7f74c43c: Merge "Add monitor_enter_object as root."

* commit '7f74c43c2c621512e7331e2876ae4d4c44d28ff0':
  Add monitor_enter_object as root.

Fix Object::Clone()'s pre-fence barrier.

Pass in a pre-fence barrier object that sets in the array length
instead of setting it after returning from AllocObject().

Fix another potential bug due to the wrong default pre-fence barrier
parameter value. Since this appears error-prone, removed the default
parameter value and make it an explicit parameter.

Fix another potential moving GC bug due to a lack of a SirtRef.

Bug: 13097759
Change-Id: I466aa0e50f9e1a5dbf20be5a195edee619c7514e

Restore the args in the invoke trampoline slow paths.

Previously, there were errors caused by not restoring the args for
the slow path in artInvokeInterfaceTrampoline and artInvokeCommon. The
slow path in artInvokeInterfaceTrampoline occurs when you attempt
to do an interface dispatch of a method in another dex file. The slow
path in artInvokeCommon occurs when the method in dex cache is not
already resolved.

Since FindMethodFromCode may resolve a method, it can occasionally
cause thread suspension/GC. If a moving GC occurred at this point it
resulted in this_object being invalid after the trampoline returned
the code pointer. This caused a native crash in one of the money runs
since there was an unbundeled app which did a List.iterator call.

Bug: 12934910
Change-Id: Ib454faad14bef0d7732a6d7f1ca5803472d502a6

Disable use of R4 as a promotable register

When we are using implicit suspend checks we can potentially
use r4 as a register into which variables can be promoted.  However
the runtime doesn't save this and thus will corrupt it.  Not good.

This disables the promotion of r4 until we can figure out how
to make the runtime save it properly.

Change-Id: Ib95ce93579e1c364de5ecc8e728f2cb7990da77a

Fix LoadValueWide to not call MarkLive for high reg that is equal to low reg

For x86 double FP registers, LoadValueWide should not call MarkLive for
high reg that is equal to low reg.

Change-Id: Ie6a59307c9ff93303bd489c15529432cfdeceaa4
Signed-off-by: Chao-ying Fu <chao-ying.fu@intel.com>

Merge "Do not output ARM64 assembler debug code."

Merge "Make frame size dependent on arch-specific constants."

Merge "Add OpEndIT() for marking the end of OpIT blocks"

Make frame size dependent on arch-specific constants.

Necessary for proper cross-compiling.

Change-Id: I852901ee6ca5121e480b83a8e318bdc9c7d615e8

Disassemble Thumb2 shifts and more VFP instructions.

Disassemble Thumb2 instructions LSL, LSR, ASR, ROR and VFP
instructions VABS, VADD, VSUB, VMOV, VMUL, VNMUL, VDIV.

Clean up disassembly of VCMP, VCMPE, VNEG and VSQRT. These
could have been erroneously used for other insns (VSQRT for
VMOV was encountered) and one VSQRT branch was unreachable.

Remove duplicate VMOV opcodes from compiler.

Change-Id: I160a1e3e4b6eabb6a5101ce348ffd49c0573257d

Do not output ARM64 assembler debug code.

This breaks the run tests, which are a simple diff against an
expected output. Please do such changes in a local change.

Change-Id: Ib961919600ba79eca1356278bc6c09ca17041c7c

Add OpEndIT() for marking the end of OpIT blocks

In ARM we need to prevent code motion to the inside of an
IT block.  This was done using a GenBarrier() to mark the end, but
it wasn't obvious that this is what was happening.  This CL adds
an explicit OpEndIT() that takes the LIR of the OpIT for future
checks.

Bug: 13751744
Change-Id: If41d2adea1f43f11ebb3b72906bd308252ce3d01

Merge "Fix off-by-1 error in new SpaceBitmap"

Fix off-by-1 error in new SpaceBitmap

Do not visit_end in the VisitMarkedRange code.

Change-Id: Iaf02788509b21a102cd1c0e2db3cbd09d0522bfa

Merge "Add monitor_enter_object as root."

Add monitor_enter_object as root.

The monitor enter object was not a root, this caused issues with
compactions since the object could move and then verify object would
fail.

Bug: 13769139
Change-Id: Ia732376fb9d1de8bf137ef9894cb2bb3241d8bb2

ARM: enable optimisation for easy multiply, add modulus pattern.

Fix the issue when src/dest registers overlap in easy multiply.

Change-Id: Ie8cc098c29c74fd06c1b67359ef94f2c6b88a71e

am a7b2826f: Merge "Support passing arguments to invoke-static* instructions."

* commit 'a7b2826fa469c626ff2c3ff26fd848c28bccc092':
  Support passing arguments to invoke-static* instructions.

Merge "Support passing arguments to invoke-static* instructions."

Support passing arguments to invoke-static* instructions.

- Stop using the frame pointer for accessing locals.
- Stop emulating a stack when doing code generation. Instead,
  rely on dex register model, where instructions only reference
  registers.

Change-Id: Id51bd7d33ac430cb87a53c9f4b0c864eeb1006f9

am 5a4139fd: Merge "art: Handle x86_64 architecture equal to x86"

* commit '5a4139fd33547d09c94d9650157e3a4e4c9eede4':
  art: Handle x86_64 architecture equal to x86