Merge "Do not include image header in decompressed size"
am: 11ad252355

* commit '11ad2523557c35acfbf829aa203d14914e61884e':
  Do not include image header in decompressed size

Merge "Do not include image header in decompressed size"

Do not include image header in decompressed size

Could cause a buffer overflow since we told LZ4 to decompress more
bytes than necessary.

Bug: 27561308

(cherry picked from commit 324eb2c6b049f1677133f0b708730e904c5e73ab)

Change-Id: I524c03b8f0e8a84814dbd8678285184e9d4da9f1

Merge "Do not include image header in decompressed size" into nyc-dev
am: 3a1b33df0d

* commit '3a1b33df0df7db2b8685f8616b9240accd6a23c5':
  Do not include image header in decompressed size

Merge "Do not include image header in decompressed size" into nyc-dev

Do not include image header in decompressed size

Could cause a buffer overflow since we told LZ4 to decompress more
bytes than necessary.

Bug: 27561308
Change-Id: Ic938f3e67f8c0627f203ad7261d4342ce43157cd

Merge "Fix cases where we miss instrumentation changes"
am: 1b876790ec

* commit '1b876790ec3e9801ef40fa26630ca925f319956c':
  Fix cases where we miss instrumentation changes

Merge "Fix cases where we miss instrumentation changes"

Fix cases where we miss instrumentation changes

Moved allocation stack push to after we record the allocation since
it can cause thread suspension.

Added handling in entrypoint utils for thread suspension cases.

Keep the AllocRecordObjectMap around since we do not want to delete
it if there are any threads waiting on new_record_condition_. The
condition guards adding stack traces while the GC is running. If we
delete the map and there are still waiters that did not resume, it
caused a CHECK failure. This could happen in cases where one thread
disables allocation tracking while other threads are about to
resume from the condition.

Bug: 27506909

(cherry picked from commit 0f394021d6abfdf9ebea6c7f405ec936a812ea62)

Change-Id: I1dc51e9f000684b4032b57beab59d317ece26f06

Merge "Don\'t be so agressive in quering the code cache for new methods" into nyc-dev
am: 04bca8a8c4

* commit '04bca8a8c47600bf1f5f5eff1e347a72d118ab04':
  Don't be so agressive in quering the code cache for new methods

Merge "Don't be so agressive in quering the code cache for new methods" into nyc-dev

Don't be so agressive in quering the code cache for new methods

Interim fix for apps which JIT a lot and trigger a lot of I/O because of
profile data (e.g. gmscore).

This reduces the amount of data re-written to disk by a few order of
magnitude. (e.g. 43k instead of 3.5MB)

Proper fix which will make the writing smarter is coming.

Bug:27600652
Change-Id: I7582bfab9abee6efebfe99da6979f7e118f82915

Merge "Optimizing: Fix TypeConversion(And(x, const)) simplification."
am: 0b2c1922cc

* commit '0b2c1922cc29a7939f747f60d80240a9fb22547c':
  Optimizing: Fix TypeConversion(And(x, const)) simplification.

Merge "Optimizing: Fix TypeConversion(And(x, const)) simplification."

Optimizing: Fix TypeConversion(And(x, const)) simplification.

Avoid introducing implicit conversions when simplifying the
expression TypeConversion(And(x, const)). Previously, when
we dropped the And, we could end up with a TypeConversion to
the same type which should be eliminated on subsequent pass
of the block's instructions; however, a subsequent dependent
TypeConversion in the same block would be processed earlier
and we would unexpectedly see its input as the conversion to
the same type, failing a DCHECK().

Bug: 27626509
Change-Id: I5874a9ceafbf635cf3391beea807ede8468ab5c3

Merge "Clean up the valgrind test targets."
am: de83659474

* commit 'de836594741c8547afb2a680e1d442dcc27cfbe5':
  Clean up the valgrind test targets.

Merge "Clean up the valgrind test targets."

Merge "Fix cases where we miss instrumentation changes" into nyc-dev
am: dd6558da4b

* commit 'dd6558da4bbd11a430268bff7d886b7fcb071dc4':
  Fix cases where we miss instrumentation changes

Merge "Fix cases where we miss instrumentation changes" into nyc-dev

Clean up the valgrind test targets.

Allow the 32-bit or the 64-bit valgrind tests to run separately or both.

Bug: 27156726
Change-Id: Ib439eb2f57d3302be152cb5af1a8960a41725f31

Merge "Optimizing: Do not re-record standby checks for dynamic BCE."
am: 70bc6de7e2

* commit '70bc6de7e27dd7d82a2bac6685e0f3096df07b83':
  Optimizing: Do not re-record standby checks for dynamic BCE.

ART: Check alignment of section offsets
am: 3a52b22cf7

* commit '3a52b22cf71446dd47cc1a2775dd0aabbf5e96f7':
  ART: Check alignment of section offsets

Merge "Optimizing: Do not re-record standby checks for dynamic BCE."

Fix cases where we miss instrumentation changes

Moved allocation stack push to after we record the allocation since
it can cause thread suspension.

Added handling in entrypoint utils for thread suspension cases.

Keep the AllocRecordObjectMap around since we do not want to delete
it if there are any threads waiting on new_record_condition_. The
condition guards adding stack traces while the GC is running. If we
delete the map and there are still waiters that did not resume, it
caused a CHECK failure. This could happen in cases where one thread
disables allocation tracking while other threads are about to
resume from the condition.

Bug: 27506909
Change-Id: I097689ca35ad408121c6b8dabd28e75cce1a43a0

Merge "Revert "Enable compilation of secondary dex files""
am: e70cda6147

* commit 'e70cda6147e2e49384a219b167a6c734a8db28f5':
  Revert "Enable compilation of secondary dex files"

Merge "Revert "Enable compilation of secondary dex files""

Revert "Enable compilation of secondary dex files"

Reverting until we fix the buildbot failures on jdwp and libcore.

Bug: 26719109

This reverts commit 770b39eb564ec16c672ace83515535b055edb2b1.

Change-Id: Ifba366d2ddc9311737b6ea29c007475a40cc803f

Merge "ART: Check alignment of section offsets"
am: 14cc147fbd

* commit '14cc147fbd09d5ef0ba5c724a7287ac6770ae330':
  ART: Check alignment of section offsets

ART: Check alignment of section offsets

Make sure the sections mentioned in the header are aligned according
to the Dalvik File Format specification.

Ensure the same for annotations.

Bug: 27275385
Bug: https://code.google.com/p/android/issues/detail?id=201384

(cherry picked from commit b512c0e5da0767d00a0ee9b7201668ab9977e21b)

Change-Id: Ifdd98377f8468e78c1c2198223ad58cab302dd37

Merge "ART: Check alignment of section offsets"

Merge "Add symbolizer option to generate debug information only."
am: fbc61e1957

* commit 'fbc61e19578d281d05728bcd120e1ace57c2fbd8':
  Add symbolizer option to generate debug information only.

Merge "Add symbolizer option to generate debug information only."

Merge "Ignore empty maps when emitting DWARF variable locations."
am: 7f77eb4858

* commit '7f77eb485810a64cde897933677d9a0e44588647':
  Ignore empty maps when emitting DWARF variable locations.

Merge "Ignore empty maps when emitting DWARF variable locations."

Optimizing: Do not re-record standby checks for dynamic BCE.

Adding the checks to the standby vector invalidates the
vector storage used by range-based loop in the
BCEVisitor::Finish() as exposed by valgrind in image_test.

Bug: 27597089
Change-Id: Ib0f0e8cdfdb7211a64a26836e085cb99fb2ce8b8

Merge remote-tracking branch \'goog/master\' into nyc-dev
am: 6ec2c6e54d

* commit '6ec2c6e54d301ef84ec0c064c310a153b9465aed':

Merge "Enable compilation of secondary dex files"
am: fb8932eec4

* commit 'fb8932eec4c09c34dfe759ba151e751293245c0d':
  Enable compilation of secondary dex files

Merge "Enable compilation of secondary dex files"

Merge "Revert "Re-enable test that was causing TimeoutExceptions on ARM64.""
am: 971c69db91

* commit '971c69db916d66096dccd481df621cee7147d20c':
  Revert "Re-enable test that was causing TimeoutExceptions on ARM64."

Merge "Revert "Re-enable test that was causing TimeoutExceptions on ARM64.""

Merge remote-tracking branch 'goog/master' into nyc-dev

bug:27398183
bug:26846185
bug:27424509

Ignore empty maps when emitting DWARF variable locations.

This is rewrite of https://android-review.googlesource.com/#/c/202115

The aim in both cases is to avoid gaps in generated locations.
(which helps to keep the size of the generated DWARF down).

However, the previous CL was a bit too eager with extending of
variable scope and reporting locations. We might have reported
variable in scope when in fact, it was not.

This CL implements simpler solution by just filtering stackmaps
without dex register maps at first opportunity. This should
ensure that locations for breakpoint locations are completely
accurate as originally intended.

Change-Id: I98378716c0ef5ef46b12181502904621eb6ecf2f

Revert "Re-enable test that was causing TimeoutExceptions on ARM64."

The test is failing again on the ARMv8 concurrent collector
configuration
(https://build.chromium.org/p/client.art/builders/armv8-concurrent-collector/builds/1833/steps/test%20libcore/logs/stdio).

This reverts commit 9e9411cc6edf5fa8be0cabd11203fac09e4bac13.

Bug: 26155567
Change-Id: I81b543a48cbbc1eca99efcc430965222ca6edfec

Enable clang for ARM builds.
am: 48c3ce6733

* commit '48c3ce6733efbc545605b1bfe2ddbdf95a04edf3':
  Enable clang for ARM builds.

ART: Allow unwinding unattached threads
am: b9d4f99977

* commit 'b9d4f999773754ab4e84496edf60af60b05119de':
  ART: Allow unwinding unattached threads

Merge "ART: mterp arm/arm64 cleanup"
am: e717e2c2c4

* commit 'e717e2c2c436da1d6daa572bd3142e34aef25b1c':
  ART: mterp arm/arm64 cleanup

Merge "ART: mterp arm/arm64 cleanup"

Enable clang for ARM builds.

Bug: http://b/25130937

We disable the integrated assembler for the ARM interpreter because
Clang can't handle the syntax present.

(cherry picked from commit a388c5a6291dbfb9a8366181a752050f4898d4ff)

Change-Id: Iefd5ba922eec54a2167e8a057c459b289aaf2ef3

ART: mterp arm/arm64 cleanup

Assembly code cleanup in response to comments from already-submitted
CL: https://android-review.googlesource.com/#/c/188977/

Change-Id: I0ea85c5759a08cb50ef3e97dc5cf79b3ba041640

Merge "Enable clang for ARM builds."
am: 2f74be0856

* commit '2f74be0856b56942614ec65896853a58a37b7aff':
  Enable clang for ARM builds.

Merge "Assembly region TLAB allocation fast path for arm."
am: 91cdf71e16

* commit '91cdf71e166630eb5e28e02298b0ba6ed4037553':
  Assembly region TLAB allocation fast path for arm.

Merge "Enable clang for ARM builds."

Merge "Assembly region TLAB allocation fast path for arm."

Assembly region TLAB allocation fast path for arm.

This is for the CC collector.

Share the common fast path code with the tlab fast path code.

Speedup (on N5):
        BinaryTrees:  2291 ->  902 ms (-60%)
        MemAllocTest: 2137 -> 1845 ms (-14%)

Bug: 9986565
Bug: 12687968

Change-Id: Ica63094ec2f85eaa4fd04d202a20090399275d85

Merge "ART: Allow unwinding unattached threads"
am: 43bad209f9

* commit '43bad209f99df82412dadab5e3b735bb12d257ab':
  ART: Allow unwinding unattached threads

ART: Allow unwinding unattached threads

Partial revert of commit ed8b723c5f3989d2593ec21c65c96d6d8bf25579.

Make it (constexpr) configurable whether we allow unwinding native
stacks of unattached threads.

Bug: 27449879

(cherry picked from commit 8d1594da6e97cd11580baf30ee4b75d4e7e2616a)

Change-Id: Iad3f3dde63155993bb4efb7bbbbde3e36f1379cc

Merge "ART: Allow unwinding unattached threads"

Merge "Integrate BitwiseNegated into shared framework."
am: 95f931eefd

* commit '95f931eefd6a694a27548c3503ca56559a2b868b':
  Integrate BitwiseNegated into shared framework.

Merge "Integrate BitwiseNegated into shared framework."

Merge "Typo in comment in art::mirror::Object::IdentityHashCode."
am: cdd3991f82

* commit 'cdd3991f821f027c9ac6674057603ba4ad9c2bb0':
  Typo in comment in art::mirror::Object::IdentityHashCode.

Merge "Fix DCHECK in OatQuickMethodHeader::GetFrameSizeInBytes()"
am: 5e447d872a

* commit '5e447d872a9529e5002d08dd97643c400ba9176c':
  Fix DCHECK in OatQuickMethodHeader::GetFrameSizeInBytes()

Merge "Typo in comment in art::mirror::Object::IdentityHashCode."

Merge "Fix DCHECK in OatQuickMethodHeader::GetFrameSizeInBytes()"

Fix DCHECK in OatQuickMethodHeader::GetFrameSizeInBytes()

We should check whether the frame is divisible by kStackAlignment.

Change-Id: I5936a6e90b38c09775b4c3a8212641e639c4181c

Merge "Tests for round() method."
am: fe9242b97d

* commit 'fe9242b97d3270e28d88971be71593787342cc4f':
  Tests for round() method.

Merge "Tests for round() method."

Merge "Suppress a valgrind libc issue."
am: 24129be9f8

* commit '24129be9f8cb6a80d246eeda20b2ef1397190f29':
  Suppress a valgrind libc issue.

Merge "Suppress a valgrind libc issue."

Merge "Revert "Revert "Allow duplicated methods in different DWARF line tables."""
am: 2de1e8a52e

* commit '2de1e8a52ef8d4b2e033f3404ca9cc970bb5149c':
  Revert "Revert "Allow duplicated methods in different DWARF line tables.""

Merge "DWARF: Compensate for compiler\'s off-by-one instruction."
am: 9faab4dd50

* commit '9faab4dd50d2bdda12284c6b8d3837943a73b5da':
  DWARF: Compensate for compiler's off-by-one instruction.

Suppress a valgrind libc issue.

To prevent our valgrind tests from failing.

Bug: 27596582
Bug: 27156726
Change-Id: Ic55256bd09fdffcf2a0c8fa4fa7951bc648b86a8

Merge "Revert "Revert "Allow duplicated methods in different DWARF line tables."""

Revert "Revert "Allow duplicated methods in different DWARF line tables.""

This reverts commit 8862fac4a0b97d827d2808146d2d79b8d799b998.

Change-Id: I4d5629df4580b7ac08a5cb04924c56eecad3ad25

Merge "DWARF: Compensate for compiler's off-by-one instruction."

Merge "Symbolize native debug information for AOTed methods in boot.oat."
am: 6030b15ce8

* commit '6030b15ce83ac2a9fb78cd9d137b6f4b441e6417':
  Symbolize native debug information for AOTed methods in boot.oat.

Merge "Add trampoline symbols to native debug info."
am: b2359d4fb2

* commit 'b2359d4fb2c9de12830301faede59f4d5fa46278':
  Add trampoline symbols to native debug info.

Merge "Symbolize native debug information for AOTed methods in boot.oat."

Symbolize native debug information for AOTed methods in boot.oat.

The oatdump symbolizer will produce all DWARF information now,
not just the symtab. This allows us to recover almost all native
debug information after the fact, even for non-debuggable oat files.

The main drawback is that non-debuggable oat files do not have
enough stackmaps to provide reasonable line stepping experience.
We might also be missing inlined methods from backtraces.

Change-Id: I73ff6f43a3a0e5a0907af353f74f04e9b0129590

Merge "Minor optimizations of debug::WriteCFISection."
am: fdaa583e78

* commit 'fdaa583e785383cca68df63c7ef53fcaf667b8e2':
  Minor optimizations of debug::WriteCFISection.

Merge "Add trampoline symbols to native debug info."

Add trampoline symbols to native debug info.

The debugger needs them to unwind through the trampolines and to
understand what is happening in the call stack.

Change-Id: Ia554058c3796788adcd7336d620a7734eb366905

Merge "Minor optimizations of debug::WriteCFISection."

Minor optimizations of debug::WriteCFISection.

The method might be passed method infos without CFI in some cases.
Use the sorting phase as a chance to filter them out.
This makes sure we do not allocate memory and sort methods if there
is in fact no work to do.

Also change the sort to stable - for the sake of determinism.

Change-Id: I97d57d77e8b709d0d49d6971f66b955efcbb57b0

Typo in comment in art::mirror::Object::IdentityHashCode.

Change-Id: I5380d1c34deda32c668e48ed5f80e96b29f5b0b5

Merge "MIPS: Don\'t use  for calling entrypoints"
am: fa3857ee61

* commit 'fa3857ee61e193471ad3547e02bafccdc712b556':
  MIPS: Don't use $t8 for calling entrypoints

Merge "MIPS: Don't use $t8 for calling entrypoints"

Merge "Revert "Revert "Revert "Revert "Change condition to opposite if lhs is constant"""""
am: a9e68021a8

* commit 'a9e68021a88566f31b0ba1a997ac3c61d3081c40':
  Revert "Revert "Revert "Revert "Change condition to opposite if lhs is constant""""

Merge "Revert "Revert "Revert "Revert "Change condition to opposite if lhs is constant"""""

Merge "Fix invariant in reference type propagation."
am: 62a9841469

* commit '62a9841469f99810b2b39b0748cb113af0e8913a':
  Fix invariant in reference type propagation.

DWARF: Compensate for compiler's off-by-one instruction.

The compiler generates stackmaps with PC value which is *after*
the instruction rather PC of the instruction itself. This causes
trouble when generating native line number mapping since the
branch instruction itself will not be mapped, and consequently
we get incorrect line number in backtraces. Add code to compensate
for this. See the in-code comment for more details.

Change-Id: I72c992e6d08a767f314290d562421b251ae60732

MIPS: Don't use $t8 for calling entrypoints

When jumping to entrypoints, $t9 register must be used. Value of $gp
is calculated based on value from $t9 and it should contain the
address of the first instruction of the entrypoint.

This CL enables MIPS32 and MIPS64 to boot.

Change-Id: Idf0f7f479cceafabb88b1b087da190dbda86e1c3

Merge "Fix invariant in reference type propagation."

Add symbolizer option to generate debug information only.

There is no need to copy the .rodata and .text sections since the
consumers can get their data form the original ELF file. We mark
those section as SHT_NOBITS instead, which is the standard way to
create ELF file with separate debug information only.

The new option is called --only-keep-debug and it essentially has
the same effect as --only-keep-debug option for "strip" command.

Change-Id: I4f7488c04bbad86288beeda11c3bf295890e740d

Merge "Revert "Allow duplicated methods in different DWARF line tables.""
am: 60e10c8564

* commit '60e10c8564f47de0b7f29b26d040c591d5a396b5':
  Revert "Allow duplicated methods in different DWARF line tables."

Merge "Revert "Allow duplicated methods in different DWARF line tables.""

Revert "Allow duplicated methods in different DWARF line tables."

This reverts commit 3c5462364daee17e651299f1c370f965f34baed8.

Change-Id: Icc63c47cd53abeff2a470ae84715012a1ef988ac

Merge "Allow duplicated methods in different DWARF line tables."
am: 8583b44f8b

* commit '8583b44f8b5788b57be4d4738072dc6a99c49823':
  Allow duplicated methods in different DWARF line tables.

Merge "Allow generation of native debug info for multiple methods."
am: cc22d41d4d

* commit 'cc22d41d4de2514cc3cb4ed569764a35f15da48d':
  Allow generation of native debug info for multiple methods.

Merge "Refactor MethodDebugInfo (input of DWARF writer)."
am: 1a008a1277

* commit '1a008a127730710f8f56c1ca66385e1b062a2dcb':
  Refactor MethodDebugInfo (input of DWARF writer).

Merge "Allow duplicated methods in different DWARF line tables."