am 24c24d43: Merge "Fix updating of JNI references for String.<init>."

* commit '24c24d43f53224bf41a41eb214c5256dda93668d':
  Fix updating of JNI references for String.<init>.

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

am 023e6f6d: Merge "Add --generate-debug-info flag and remove the other two flags."

* commit '023e6f6d67d6ab520e194d142f3adaafb6e3a11d':
  Add --generate-debug-info flag and remove the other two flags.

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

am 6f84e504: Merge "ART: Distinguish Checker lines from comments"

* commit '6f84e50443f1ead31efb25ecbdfb51c1037e4519':
  ART: Distinguish Checker lines from comments

Merge "ART: Distinguish Checker lines from comments"

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

am b4e2fbbe: Merge "Add a HCurrentMethod node."

* commit 'b4e2fbbed28c3bcdb8fd0fee5a201ba78e1edf28':
  Add a HCurrentMethod node.

Merge "Add a HCurrentMethod node."

am 31db3432: Merge "Fix single-step in native thread"

* commit '31db3432157caac55584a59827c69b303ba09d13':
  Fix single-step in native thread

Merge "Fix single-step in native thread"

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

* commit '2b2288f2abbcdcd1201b322008fa30495d724d51':
  Fix rule: the exe must be first dependency.

am 01908ebc: Merge "Refactor makefiles to allow just building tests."

* commit '01908ebcea21a8cbdfe7ba0cb4ddb2661292c69d':
  Refactor makefiles to allow just building tests.

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

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

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

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

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

Change-Id: I8a3277835f22a23c31063eda9b7f1fdc9423a5b0

am 38bb2ab8: Merge "Fix valgrind includes"

* commit '38bb2ab825f83488ec38c9b07072da0f53777867':
  Fix valgrind includes

Merge "Fix valgrind includes"

am d99aae11: Merge "Reset libc targetSdkVersion"

* commit 'd99aae117984dc655abbd408d1caa7cc09e4e6c8':
  Reset libc targetSdkVersion

am 33d75bc7: Merge "ART: Fix VerifyObject runtime verification"

* commit '33d75bc75c72db7e5b1ea4ad0be287b1dfd72e69':
  ART: Fix VerifyObject runtime verification

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

am 65efe919: Merge "Split .oat_patches to multiple sections."

* commit '65efe9192390187674108b9c895ba225628bb07b':
  Split .oat_patches to multiple sections.

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

am 13df71ce: Merge "Bring Reference Type Propagation to NewArray"

* commit '13df71cea8c774d28e834497288f67cb2bbb16b0':
  Bring Reference Type Propagation to NewArray

am 55bf5b4f: Merge "Improve art::CodeInfo dumping."

* commit '55bf5b4f1a570fb10d7ba8c91ac49c6bc16639ac':
  Improve art::CodeInfo dumping.

am 19b6ab4c: Merge "Rename VisitCondition\'s argument in code generators."

* commit '19b6ab4c71a1eafad755648820fa23306a4d0fb4':
  Rename VisitCondition's argument in code generators.

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

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

* commit '6c70104dc3ecd66b46f56cae068a14e12021c9f2':
  [optimizing] x86: Prefer add over lea if possible

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>

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

* commit 'f109cd2215ba23dac23a15eb64ad21ef2b15843c':
  ART: Use kBadFprBase on mips64 and arm64.

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

am 382f5c24: Merge "Fix another source of undeterministic inlining."

* commit '382f5c24eb663ca8fa39a94a038349138a00272a':
  Fix another source of undeterministic inlining.

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

am 00f09ff9: Merge "Pass the dex method index directly to interface trampoline."

* commit '00f09ff93480d5f43aaf657650a59d57330184aa':
  Pass the dex method index directly to interface trampoline.

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

am ec1170a4: Merge "Add missing oatdump test dependency"

* commit 'ec1170a4bf9d1c0baa00640d2cbc1e6870bc7f68':
  Add missing oatdump test dependency

Merge "Add missing oatdump test dependency"

Add missing oatdump test dependency

Bug: 18000316
Change-Id: Ic828e8be8e5d3ef7e112facd0db57013740d449a

am 7c1a58bb: Merge "Add oatdump test"

* commit '7c1a58bb2316fc5bc47b35ed9772454d2d061352':
  Add oatdump test

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

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

* commit 'eabafcefca8ead8309dccc1c5510e6e0845e471d':
  Fix alloc-dealloc-mismatch (new[] vs delete) in tests.

am 2dde2a17: Merge "Switch to using .debug_frame for CFI."

* commit '2dde2a17bfeca795d2ab17795952770487d848b0':
  Switch to using .debug_frame for CFI.

am 6f3891ce: Merge "Generate just single ARM mapping symbol."

* commit '6f3891cea11de6e247111563f49136e222f02b1c':
  Generate just single ARM mapping symbol.

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

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

* commit 'e83635d05b80d7e933f868431f6043aef04fd589':
  Fix mac build: cast fpr callee save addresses.

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

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

* commit '5c592b598742569be4819bdc0e32242a4e1a7693':
  Fix mac build: include <sys/time.h> in time_utils.cc .

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

am dc31313c: Merge "ART: Clean up arm64 kNumberOfXRegisters usage."

* commit 'dc31313c4577bb4c00d90774159c84e738ed348d':
  ART: Clean up arm64 kNumberOfXRegisters usage.

Merge "ART: Clean up arm64 kNumberOfXRegisters usage."

am 03b5a398: Merge "Fix for potential moving GC bugs around proxy class."

* commit '03b5a398a96ea29c39ddfe3d810245b868d2871f':
  Fix for potential moving GC bugs around proxy class.

am 1b742fcc: Merge "Log the process memory maps on host SIGSEGV."

* commit '1b742fccf9ca68b783fdbd559a3fcbdafab48852':
  Log the process memory maps on host SIGSEGV.

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

am c06841da: Merge "Fix a few literals and assertions."

* commit 'c06841dac1443e99e92596f15caa82c99bfee010':
  Fix a few literals and assertions.

am 41f27292: Merge "Move the catalog after the stack maps."

* commit '41f27292e10792165bfdf67b9a35ccaea1a6872e':
  Move the catalog after the stack maps.

Merge "Fix a few literals and assertions."

Merge "Move the catalog after the stack maps."

am 9470f045: Merge "Make inlining deterministic."

* commit '9470f045c89cdb3de5175266bda523b8a1c2d96d':
  Make inlining deterministic.

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

am ce63cf65: Merge "ART: Update graph\'s exit block field if removed"

* commit 'ce63cf656bb5b25051d580a58086f4bf15fb878b':
  ART: Update graph's exit block field if removed

Merge "ART: Update graph's exit block field if removed"

am 092a5656: Merge "JDWP: fix breakpoint on catch statement"

* commit '092a5656937a319449346e8c356f3f4c2870d81c':
  JDWP: fix breakpoint on catch statement

Merge "JDWP: fix breakpoint on catch statement"

ART: Update graph's exit block field if removed

Running DCE on an infinite loop will delete the exit block but the
corresponding field is currently not cleared in the parent graph.
This does not cause any problems at the moment as that information is
only used in codegens to DCHECK that a block is not the exit block.
However, it will be necessary to update the inliner once we start to
inline methods with loops.

With this patch, DCE will update the HGraph::exit_block_ field. DCHECK
was also added to HGraph::InlineInto to make sure that the inlined
graph does have an exit block.

Change-Id: Ia8ddca375bbc6830cd919af6059a52cc9b73a023

JDWP: fix breakpoint on catch statement

Setting a breakpoint on a catch statement in the source actually
installs the breakpoint on a DEX move-exception instruction. At this
point, an exception is pending in the current thread.

The issue is no exception must be pending in the current thread to
report the breakpoint event. This is required to be able to call JNI
functions to create JDWP ids.

This CL fixes it by clearing the pending exception before reporting
event and restore it after reporting the event.

Bug: 21382373
Change-Id: Id3c60cca398135a3e0859a1ccb645b9c99d5ca76

am 8a8d8252: Merge "Do not use dex_compilation_unit after inlining."

* commit '8a8d8252f52bec14a8fe2b6f88e1f184c63cf351':
  Do not use dex_compilation_unit after inlining.