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

Merge "Fix using kArm instead of kThumb2."

Fix using kArm instead of kThumb2.

Change-Id: I0f3bf93b38d910bc57d017693675829b6a39bc18

Merge "art_quick_lock_object uses registers incorrectly"

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

Merge "Improve invalid root dumping."

Improve invalid root dumping.

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

Change-Id: Ie821296d569f34909ba6e2705f5c347cd2143a3a

Merge "Disable use of R4 as a promotable register"

Merge "Inlining synthetic accessors."

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

Inlining synthetic accessors.

Bug: 11549140
Change-Id: Ie0034a6840b1beaa3df92f26bf9d315119c81e34

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

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

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

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

Use target-specific word instead of runtime word.

Change-Id: Ia11dc3cc520a1a5c7bd017013e5699af9570ce91

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

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>

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

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

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

Merge "art: Handle x86_64 architecture equal to x86"

Merge "Fix Unsafe.getLong()/putLong() for x86, add test."

Merge "Runtime::PreZygoteFork returns void, not boolean."

Fix Unsafe.getLong()/putLong() for x86, add test.

Change-Id: Id6841ae50e1047543c04dcdae30254dd6ce82082

Merge "Profile: renamed kThresholdPercent and make it configurable"

Merge "Profile: repurposed kTresholdPercent"

art: Handle x86_64 architecture equal to x86

This patch forces FE/ME to treat x86_64 as x86 exactly.
The x86_64 logic will be revised later when assembly will be ready.

Change-Id: I4a92477a6eeaa9a11fd710d35c602d8d6f88cbb6
Signed-off-by: Dmitry Petrochenko <dmitry.petrochenko@intel.com>

Merge "Use temp register in Unsafe Get/Put intrisics for displaced offset."

Merge "Make SpaceBitmap cross-compiling tolerant"

Use temp register in Unsafe Get/Put intrisics for displaced offset.

Previously, the object register would get clobberd by the adding the
offset for wide UnsafeGets and wide UnsafePuts. This resulted in heap
corruption in AtomicLongFieldUpdater.compareAndSet and possibly other
places.

Bug: 13744490
Change-Id: Ic368d930b855c5c57e22060b32b73d0745328ef2

Merge "Add support for -Xverify:none mode."

Add support for -Xverify:none mode.

This mode skips all verification and compilation.
Public bug: https://code.google.com/p/android/issues/detail?id=67664

Change-Id: Idd00ab8e9e46d129c02988b063c41a507e07bf5b

Make SpaceBitmap cross-compiling tolerant

Change the order of bits in SpaceBitmap to be the intuitive one:
Offset 0 is bit 0, instead of the MSB. Then compiling on 32b for
64b works as expected.

Change-Id: Iee2491eaf06d4b5f8b534b7c980d5719633cb64c

Merge "Keep the symbol table in the shared libraries."

Merge "Pass instruction-set from runtime through to spawned dex2oat."

Pass instruction-set from runtime through to spawned dex2oat.

Change-Id: I1727af7beb9f710c29124d4d6bc9175e4856f3cc

Profile: renamed kThresholdPercent and make it configurable

Bug: 12877748
Change-Id: If266f20f77faad4a726ef8df451a0c42f541392f

Merge "force dex2oat target executables to be compiled for 32-bit"

Merge "Add r4 as promotable register if implicit suspend checks"

Add r4 as promotable register if implicit suspend checks

If we are doing implicit suspend checks we should be able
to use r4 as a target for promotion.

Also bump OAT version

Change-Id: Ia27d14ece3b3259dbb74bcf89feaa9da2cda6db8

force dex2oat target executables to be compiled for 32-bit

64-bit dex2oat doesn't support 32-bit targets yet, force dex2oat
target executables to compile 32-bit for now by passing in a
value for LOCAL_MULTILIB to build-art-executable.

Bug: 13764385
Change-Id: I41adba1e64f373e4b61a3c13af2592ed4f7d0b57

Profile: repurposed kTresholdPercent

Previously kTresholdPercent was the percentage of samples of the total
that a method must comprise before compiling.

I changed it to mean the threshold for a running total...i.e. compile
all the methods that comprise K% of the samples cumulatively.

(in the process fixed ProfileData#percent doc and changed its name)

Bug: 12877748
Change-Id: Ib0e18e525a16c11b189afc3d840c09183ac629de

Merge "Avoid debug check violation in thread registration."

Avoid debug check violation in thread registration.

Bug: 13757855
Change-Id: Ied83a7fc43e2b9a89a8b99aa4f74069ef8a369b1

Merge "Fixed total method count in profile file parsing."

Fixed total method count in profile file parsing.

Change-Id: Iae38b3ca296378d17f054ac330301035ec684084

Runtime::PreZygoteFork returns void, not boolean.

This method aborts on failure (as it should) and unconditionally
returns true, so making it return void simplifies callers.

Change-Id: Iae39bd327f20311579ece47efa8afd1be7defce9

Merge "Include <sys/resource.h> for rlimit."

Include <sys/resource.h> for rlimit.

Change-Id: I0e487f8191e284052d8b82eb942d723147d780a1

Merge "Update missed DEX2OAT -> DEX2OATD changes."

Update missed DEX2OAT -> DEX2OATD changes.

Change-Id: I9556ac9ce30fe057e8b278858303e25c547e86ba

Merge "Move most zygote related code to frameworks/base."

Merge "Use debug version of DEX2OAT explicitly."

Use debug version of DEX2OAT explicitly.

Change-Id: I57b4c8caedcc6bfbeaa541d8171f8b5b3069c584

Merge "Fix ClassHelper::GetDirectInterface for proxy class"

Merge "Cleanup transaction support"

Cleanup transaction support

Updates Thread::CreateInternalStackTrace to support both transactional and
non-transactional modes using template.

Generalizes non-transactional mode for invariant fields (which are set only
once).

Removes ArrayLog::VisitRoots as we never create Array logs of ObjectArray. As
ObjectArray elements are set using Object::SetFieldObject, they are already
recorded in the object logs: the object is the array itself and the offset
corresponds to the element index in this array. And also checks we never log
ObjectArray in array logs.

Fixes location of thrown exception when calling native method during class
initialization.

Change-Id: Idbc368d3b8292b85ff40bc8a7c559e085477bf89

Keep the symbol table in the shared libraries.

In order to allow the unwinder code to have meaningful names for
art functions, leave the symbol table.

Here are the size increases:

Platform    Size Increase
--------    -------------
arm         ~631K
arm64       ~631K
mips        ~424K
x86         ~565K
x86_64      ~565K

Bug: 12958251
Change-Id: Ib8bf11bef648f8296f4d6544f7b2af0d7f969ad8

Merge "Use TEST_ART_RUN_TESTS consistently"

Use TEST_ART_RUN_TESTS consistently

Change-Id: I0242702886d53a39f2604e209a0763ff82278e1c

Merge "Add GenBarrier() calls to terminate all IT blocks."

Add GenBarrier() calls to terminate all IT blocks.

This is needed to prevent things like load hoisting from putting
instructions inside the IT block.

Bug: 13749123
Change-Id: I98a010453b163ac20a90f626144f798fc06e65a9

Merge "Fix ThreadStress, unify ANR lock descriptions."

Merge "Fix Thread offsets for ARM64 assembly"

Merge "Fix running with DlMalloc instead of RosAlloc."

Fix ThreadStress, unify ANR lock descriptions.

Bug: 13323656
Change-Id: I902825a847af9aa570ed79701409298e3532bebf

Fix running with DlMalloc instead of RosAlloc.

Bug: 13748830
Bug: 13744490

Change-Id: Ieec97cf58ebf9864a40ee85aae582cdcf3f27c0c

Fix Thread offsets for ARM64 assembly

Change-Id: I8d0516091b5a865f415e1446425203785e607f8a

Merge "Build changes to build multilib tests"