Tweak one hint and one split in the linear scan.

- Return a hinted register if it is available. Otherwise
  another move will be necessary.
- Use SplitBetween instead of raw split when a register
  is not fully available. This will find the best split
  position.

Change-Id: Ie464e536204ab556eb09345fe6426621eb86e5ac

Merge "Fix some ArtMethod related bugs"

Fix some ArtMethod related bugs

Added root visiting for runtime methods, not currently required
since the GcRoots in these methods are null.

Added missing GetInterfaceMethodIfProxy in GetMethodLine, fixes
--trace run-tests 005, 044.

Fixed optimizing compiler bug where we used a normal stack location
instead of double on ARM64, this fixes the debuggable tests.

TODO: Fix JDWP tests.

Bug: 19264997

Change-Id: I7c55f69c61d1b45351fd0dc7185ffe5efad82bd3

Merge "Fix test 115-native-bridge for Mips."

Fix test 115-native-bridge for Mips.

Change-Id: I3eee8df39e774d233f11c43a5578f407bfdebca1

Merge "Move mirror::ArtMethod to native"

Move mirror::ArtMethod to native

Optimizing + quick tests are passing, devices boot.

TODO: Test and fix bugs in mips64.

Saves 16 bytes per most ArtMethod, 7.5MB reduction in system PSS.
Some of the savings are from removal of virtual methods and direct
methods object arrays.

Bug: 19264997
Change-Id: I622469a0cfa0e7082a2119f3d6a9491eb61e3f3d

Merge "Add unstarted runtime test for String.<init>."

Add unstarted runtime test for String.<init>.

Bug: 21173514

(cherry picked from commit 73a937740841154eb32a0aa15f938f1c06f26091)

Change-Id: If7bd360c396e2594f34efe3a6237f77c7129112b

Merge "Make some parts of ELF more (pointer) aligned."

Make some parts of ELF more (pointer) aligned.

I am not aware of any problem at the moment, but better
be safe than sorry.  I have noticed that we align less
when comparing our output with output of other tools.

Bug: 20556771
Change-Id: I904e66f5b7776f64cc90ec2802ddb2c630b64d08

Merge "Fix double-exception in super-class method validation."

Merge "Link .dynamic to .dynstr"

Merge "Set correct size of PT_PHDR ELF segment."

Link .dynamic to .dynstr

This is what the ELF specification says we should do.
(see Figure 1-13: sh_link and sh_info Interpretation)

Also make the entity size of string sections 0 (i.e. not a table).
Size 1 makes some sense as well but it is not what other tools do.

Bug: 20556771
Change-Id: I780ba2e13c09c33c143fca70b38a35f3b8528fb2

Set correct size of PT_PHDR ELF segment.

This is just a nit pick, I do not believe that having it too
long could have caused any trouble since the correct number of
elements is stored in ELF header and extra space was zero padded.

Bug: 20556771
Change-Id: Ide56e6aa40f9b7175b55a830caf34c47e48213af

Merge "ART: Add support for special handlers in sigchainlib"

Merge "Support for inlining methods that call/throw."

Merge "Bring Reference Type Propagation to Instance/StaticInstanceField"

Support for inlining methods that call/throw.

Mostly fixes here and there to make it working.

Change-Id: I1b535e895105d78b65634636d675b818551f783e

Merge "Also add run-test dependencies to build-art-*-tests."

Merge "Use the new HCurrentMethod in HLoadString."

Also add run-test dependencies to build-art-*-tests.

Change-Id: If02c1a9426c2d04f4fad8892e387b2fb6c48ee2f

Merge "Fix typo in script."

Fix typo in script.

Change-Id: I853bb5758005f9f7c4940f815c3d72a1d48acd21

Use the new HCurrentMethod in HLoadString.

Change-Id: I23d27e5e10736d127519eb3238ff8f25df3843a2

Merge "Update libcore failures"

Update libcore failures

Default device date (1970) is causing some tests to fail. This
change updates libcore failures to reflect this knowledge.

Change-Id: Ia8151cfa1600fba4b45c203d468de9244b322d61

Bring Reference Type Propagation to Instance/StaticInstanceField

For this, we need the field index in FieldInfo, hence the add of the field.

Change-Id: Id219bd826d8496acf3981307a8c42e2eb6ddb712

Merge "Add a script to build exactly like the chrome buildbot."

Add a script to build exactly like the chrome buildbot.

The buildbot will use that script directly.

Change-Id: I837432f1e1e9b9d238d91de9af51ad955f7cd76d

Fix double-exception in super-class method validation.

Bug: 19333589
Change-Id: I2399c4058d488bbdbf6709c63e367f1b7c251b1d

Merge "ART: Fix valgrind Makefile"

ART: Fix valgrind Makefile

Add a missing dependency that lead to runtime errors.

Change-Id: Idbaed6bcc56a644713f10847f8600dc0f962202f

Merge "ART: Add Mips o32 callee-save registers to SaveAll frame"

Merge "Add JNI String.<init> test for global and weak global refs."

Add JNI String.<init> test for global and weak global refs.

Test for change:
https://android-review.googlesource.com/#/c/152265/

(cherry picked from commit 7be83025b62ab499ed8b2b59521f8fbe8e2b13ab)

Change-Id: If10a5fcf7a554d798d2bb21d493e5a7266ed0079

Merge "Fix updating of JNI references for String.<init>."

ART: Add support for special handlers in sigchainlib

Add support for a special managed handler in sigchainlib that
will be called as the first user handler.

Use this support for native bridge v2. Extend test 115-native-bridge
to test the functionality.

Bug: 20217701
Change-Id: I78cc01fbdabe169154ff6b94c9f3ddb95b5c7448

Merge "Add --generate-debug-info flag and remove the other two flags."

Add --generate-debug-info flag and remove the other two flags.

Replace the flags --include-debug-symbols and --include-cfi
with single flag called --generate-debug-info (with alias -g).

The name "symbol" was not ideal, since depending on context it
may be interpreted as "ELF symbols", or "debugging information".

This CL also means that we have only the options to include
either all debugging information or none.  This should be fine,
since we can use standard tools to strip anything we do not want.

Change-Id: I721fded56d755d7eed0ef36aa84e841a1f5747f8

Merge "ART: Distinguish Checker lines from comments"

ART: Add Mips o32 callee-save registers to SaveAll frame

The floating point registers f20-f31 are callee-save in the Mips
ABI. While the managed code does not touch them, they need to be
saved when throwing an exception, so that they will be correctly
restored and not smashed.

Bug: 21266656
Change-Id: Ia96d52ce7fb41bf604da1797ce4d7a703e292415

ART: Distinguish Checker lines from comments

In order to prevent tests passing due to lines with hard-to-spot
formatting errors begin ignored, e.g. by forgetting the colon after
"//CHECK", Checker will now require its assertions to start with "///"
or "##", respectivelly for Java and Smali. Such lines will never be
ignored and will fail the test unless successfully parsed.

Change-Id: I0da9a8f13eb96d950af8c85df17d1899a853a299

Merge "Add a HCurrentMethod node."

Merge "Fix single-step in native thread"

Add a HCurrentMethod node.

This enables register allocation for the current method, so
that users of it don't always load it from the stack.

Currently only used by HLoadClass. Will make follow-up
CLs for the other users.

Change-Id: If73324d85643102faba47fabbbd2755eb258c59c

Fix single-step in native thread

If we attempt to single-step in a thread that is not running Java
code, there is no current method on the stack. So we need to check
for null before trying to decode debug info.

Bug: 21320157
Change-Id: Idef9954d14e1eb7185f25d2a6e238135ac52d35f

Merge "Fix rule: the exe must be first dependency."

Fix rule: the exe must be first dependency.

Change-Id: Ic59a0c38ae8d7d1efc439014e694a95063a7589d

Merge "Refactor makefiles to allow just building tests."

Refactor makefiles to allow just building tests.

Also add a way to avoid syncing to the device, in case
a previous sync did it.

Change-Id: I38673cad712f8ddff641589cea50ca0d0dc04850

Fix updating of JNI references for String.<init>.

Was missing updates to globals and weak globals.

Bug: 21288130
Bug: 21440428

(cherry picked from commit 1a302fb4f84525289c1cf7a437f5be1999a75251)

Change-Id: I41b64e8aae906f5986de483db096d8191e45f260

Merge "Fix build - large frame size of ElfWriterQuick<ElfTypes>::Write (again)."

Fix build - large frame size of ElfWriterQuick<ElfTypes>::Write (again).

Change-Id: I8a3277835f22a23c31063eda9b7f1fdc9423a5b0

Merge "Fix valgrind includes"

Merge "Reset libc targetSdkVersion"

Merge "ART: Fix VerifyObject runtime verification"

Fix valgrind includes

 external/valgrind structure has been changed
 to reflect upstream projects struture.

 Making corresponding changes to art includes.

Change-Id: I2434a270f88502528df82c4820afedb002b65b8d

Merge "Split .oat_patches to multiple sections."

Reset libc targetSdkVersion

 This will let libc/linker know which sdk version
 is targeted by the app. This is needed to work around
 incorrect DT_NEEDED entries in old apps.

Bug: http://b/21364029
Change-Id: I2f2907104106aac6c8831381ca171dc0302e6a9d

Merge "Bring Reference Type Propagation to NewArray"

Bring Reference Type Propagation to NewArray

Change-Id: Ieff4f38854e06b0ed4b5689ced94a4289053d80d

Merge "Improve art::CodeInfo dumping."

Merge "Rename VisitCondition's argument in code generators."

Rename VisitCondition's argument in code generators.

This argument is a condition instruction, not a comparison.

Change-Id: I026f799d2161df58b0c8a84600eb8fffd6f7b998

ART: Fix VerifyObject runtime verification

Update some bit-rotted code to work again. Most tests now work, for
some the verification overhead results in a timeout.

Change-Id: Ieab4f2de474a05e915e24abc93da3c2eeed996eb

Improve art::CodeInfo dumping.

- Refactor existing dumping code into a set of new routines.
- Use art::Indenter to properly indent in the output.
- Add an additional argument to art::CodeInfo::Dump to
  enable or disable the dumping of stack maps and Dex
  register maps.

Change-Id: I52814095ff465522302d5d69d416726369ca5258

Merge "[optimizing] x86: Prefer add over lea if possible"

[optimizing] x86: Prefer add over lea if possible

Looking at some generated code, I noticed an lea being used when an add
was sufficient.  Check for that case, and generate the add.

Fixed for x86 and x86_64.

Change-Id: I110304ff0fed8837ada96d34353a293d29022ce5
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>

Merge "ART: Use kBadFprBase on mips64 and arm64."

Merge "Fix another source of undeterministic inlining."

ART: Use kBadFprBase on mips64 and arm64.

Fix incorrect usage of kBadGprBase to avoid misleading and
ambiguous data in crash investigations.

Bug: 21266656
Change-Id: I1de6cd621d770be586f8140e86833769865b42c6

Fix another source of undeterministic inlining.

bug:20037935

Change-Id: Ifad234a331ec616f065aabc65576c5636ad8ec21

Merge "Pass the dex method index directly to interface trampoline."

Pass the dex method index directly to interface trampoline.

This avoids computing the dex pc and re-finding the method
index again. I have kept the code for kDebugBuild.

Change-Id: Icd60e0deade755e32b54021c0875b1af592b8c3e

Merge "Add missing oatdump test dependency"

Add missing oatdump test dependency

Bug: 18000316
Change-Id: Ic828e8be8e5d3ef7e112facd0db57013740d449a

Merge "Add oatdump test"

Add oatdump test

Bug: 18000316

Change-Id: Ic22d63dc64a9b7a492b5e6bfbd4c75f30c35dfd5

Split .oat_patches to multiple sections.

.oat_patches section currently contains encoded patch locations for
several other sections. Split it to several sections - one for each
of the destination sections. For example, .text.oat_patches section
contains patch locations for the .text section.

This ensures that if we strip some the sections using standard
tools, we strip the corresponding .oat_patches section as well.

It also makes the overall design simpler.
I should have done it this way in the first place.

Since ApplyOatPatches has been simplified and uses unaligned memory
access, this also fixes bug 21403573.

Bug:20556771
Bug:21403573
Change-Id: Iae7c423911507b524eec500e8d61744046fcd3ba

Merge "Fix alloc-dealloc-mismatch (new[] vs delete) in tests."

Merge "Switch to using .debug_frame for CFI."

Merge "Generate just single ARM mapping symbol."

Fix alloc-dealloc-mismatch (new[] vs delete) in tests.

Change-Id: I680eaf819f7d7d3a4149c833b6739cba4b7e21be

Merge "Fix mac build: cast fpr callee save addresses."

Fix mac build: cast fpr callee save addresses.

Bring back reinterpret_cast<>s removed by
    https://android-review.googlesource.com/151309 .
They are needed for mac builds.

Bug : 13925192

Change-Id: I7728c39a867a278000c84d75440b31e818c3bfcf

Merge "Fix mac build: include <sys/time.h> in time_utils.cc ."

Fix mac build: include <sys/time.h> in time_utils.cc .

Required for gettimeofday(). Not needed in utils.cc .

Bug: 13925192
Change-Id: If06840d8a47c9193ccc7bd4cee5c16fcf76d9729

Merge "ART: Clean up arm64 kNumberOfXRegisters usage."

Merge "Fix for potential moving GC bugs around proxy class."

Merge "Log the process memory maps on host SIGSEGV."

Fix for potential moving GC bugs around proxy class.

- Handlerize proxy_class which is live across multiple allocation
  points in ClassLinker::CreateProxyClass().

- In ClassLinker::CreateProxyClass(), insert a proxy class into the
  class table before creating ArtFields for it (and update it later in
  LinkClass()) because the field roots (ArtField::declaring_class_)
  won't be updated by GC unless the class is in the class table. If GC
  happens before they are updated by FixupTemporaryDeclaringClass()
  from LinkClass(), FixupTemporaryDeclaringClass() may not update the
  field roots correctly because the old class may already be moved but
  the fields roots may not. Reduce a window of time where the fields
  roots could be stale.

- In ClassLinker::LinkClass(), directly wrap a new class in a handle
  to avoid a window of time where new_class may be potentially stale.

- Print more diagnostic info about the holder of the field upon a mark
  sweep invalid ref crash.

- Add an additional sanity check in Field::GetArtField().

(cherry pick commit 08d1b5f2296c0f51507b8b443f4e39dfc161572c)

Bug: 20557050

Change-Id: I9ad32d304922da96b7e1fad262d97de21cbac776

ART: Clean up arm64 kNumberOfXRegisters usage.

Avoid undefined behavior for arm64 stemming from 1u << 32 in
loops with upper bound kNumberOfXRegisters.

Create iterators for enumerating bits in an integer either
from high to low or from low to high and use them for
<arch>Context::FillCalleeSaves() on all architectures.

Refactor runtime/utils.{h,cc} by moving all bit-fiddling
functions to runtime/base/bit_utils.{h,cc} (together with
the new bit iterators) and all time-related functions to
runtime/base/time_utils.{h,cc}. Improve test coverage and
fix some corner cases for the bit-fiddling functions.

Bug: 13925192
Change-Id: I704884dab15b41ecf7a1c47d397ab1c3fc7ee0f7

Merge "Fix a few literals and assertions."

Merge "Move the catalog after the stack maps."

Move the catalog after the stack maps.

Fetching a stack map should be cheap, and therefore we should
not compute the size of the catalog.

Change-Id: Iff6accb7832bdfa6d209a0928d6088b8b57c1aa4

Merge "Make inlining deterministic."

Fix a few literals and assertions.

Change-Id: I0a1e9db607ec7325e17568b034ba90e68d2298f9

Make inlining deterministic.

Only the case where two methods are not in the same dex
file could lead to undeterministic behavior.

bug:20037935

Change-Id: I1a7642a979302b17e76d196894437c1aacbbbe9d