Merge "Show registered native allocations in ahat."

Merge "Cast method pointer to uintptr_t before uint64_t"

Merge "Change the order of functions in .debug_frame to save space."

Merge "Do not write full method signature in mini-debug-info."

Merge "Revert "Revert "Add option to generate compressed backtrace info."""

Merge "Add option to never interpret."

Add option to never interpret.

Change-Id: Ib6d6170fa60c77c2e6a8844964f14fa0de4c9e7b

Merge "Optimizing: Lazy creation of Handle<>s in RTP."

Optimizing: Lazy creation of Handle<>s in RTP.

This reduces the number of StackHandleScope<>s that we
allocate, especially when we're inlining many methods.
Previously, each RTP would create 4 handles in the ctor
which is actually the same as kNumReferencesPerScope.

Change-Id: Ib6767993b1e89913a91275f0ef66dac786bccb6c

Cast method pointer to uintptr_t before uint64_t

Previously we were casting a pointer to uint64_t. This sign extends.
This change fixes 64 bit app image crashes on angler.

Bug: 22858531
Change-Id: I03e973a0c5a1d74db7e8db0f13f5b1825d8d8ab5

Merge "Remove GetImageRoots read barrier for image relocation"

Remove GetImageRoots read barrier for image relocation

Fixes CC assertion that failed if an app image was loaded when the GC
was running.

Bug: 26786304
Change-Id: I47bea7b4b947332b57be2e2db604c2c25d92b7e2

Merge "Convert to UTF8 before doing dex file location comparison"

Convert to UTF8 before doing dex file location comparison

Need to have strings in the same format or else the lengths
might not match due to UTF8 conversion.

Added regression test.

Bug: 26799552

(cherry picked from commit 3776db4e51ab61a6e535772417c2adf95920b569)

Change-Id: Ifbc8c38418c9da307a0ff4446d18cea4fb5e71bc

Merge "Blacklist new test due to so loading problems."

Blacklist new test due to so loading problems.

Change-Id: I3f519cb2ab02afe3852f59a65de7d2ff2d4070fd

Merge "Temporarily use -fstack-protector for the read barrier config."

Temporarily use -fstack-protector for the read barrier config.

Instead of -fstack-protector-strong.

To avoid a major performance regression.

Bug: 26744236
Bug: 12687968

Change-Id: I299247589e2bb6939393d30bdc54c15ce0f04b2a

Merge "Use mutator lock to guard adding and removing heap spaces"

Use mutator lock to guard adding and removing heap spaces

Too hard to add a new lock since dlmalloc ArtMoreCore requires
looping through the spaces while holding the allocator lock.

Bug: 22858531
Change-Id: Ieac2136da02c766b6795cd604a58798bee37ef2a

Merge "Recognize common utilities as intrinsics."

Merge "Some minor simplifications in code and tests."

Merge "Enable interface default methods by default."

Enable interface default methods by default.

This also enables interface static methods.

This removes the -Xexperimental:default-methods flag and all places
where we explicitly check for its presence.

Bug: 24618811

Change-Id: Icd91e377bd6e1a45a1645f810d15de1b0312e31d

Recognize common utilities as intrinsics.

Rationale:
Recognizing these method calls as intrinsics already has
major advantages (compiler knows about no-side-effects/no-throw
properties). Next step is, of course, to implement these
with native instructions on each architecture.

Change-Id: I06fd12973238caec00d67b31b195d7f8807a538e

Some minor simplifications in code and tests.

Background:
This is actually a resubmit of an earlier cl that was
reverted because was test was less robust against
inlining changes (it assumed a virtual call would
never be inlined).

original cl: If8ada79dfd70bea991c11d2b18661b951b6c4cd4
revert cl:   I739aaaccd0509d02a62ef01e797a6d45bfe941df

Change-Id: I952680d60ff488874907f066bfdf156a45b409ba

Merge "Revert "Lift the spill at each irreducible loop block restriction.""

Revert "Lift the spill at each irreducible loop block restriction."

This reverts commit 79e9f43951c3cfa9ab3b0fea93e5bfdfa7aa5950.

Change-Id: I0670618b4076e06bd3f6bf8c385abfd1b651393c

Merge "Lift the spill at each irreducible loop block restriction."

Merge "ART: Remove another unnecessary mutator lock annotation."

ART: Remove another unnecessary mutator lock annotation.

Missed in https://android-review.googlesource.com/198690 .

Change-Id: I56a6dfe1126315957658962d51290e3bfa873c79

Merge "Disable interpreter run-tests occasionally failing with CC."

Merge "ART: Remove some unnecessary mutator lock annotations."

Disable interpreter run-tests occasionally failing with CC.

These tests sometimes fail on the concurrent collector
configurations:
- 004-ThreadStress
- 141-class-unload

Disable them while we investigate.

Also adjust the list of Quick run-tests failing with CC.

Bug: 26786154
Bug: 25866001
Bug: 12687968
Change-Id: I7a37184888b7c992ab636bba44760c1aeab51aad

Merge "Add profile support for apps with shared runtime."

Add profile support for apps with shared runtime.

- extend ProfileSaver to track different dex locations to different
profile files.
- allow repeated calls to Runtime::RegisterAppInfo() which will
register a new pair <profile_file, locations_to_track> with the profile
saver.

Bug: 26080105
Change-Id: I58c4587d5f462c557411ad7126a00c29cd7db3a6

Merge "Add dependency on liblzma."

Lift the spill at each irreducible loop block restriction.

It was not intended to have it this way anyway. This also
required to fix GetSiblingAt, to take into account interval
holes, and ConnectSplitSibling to re-materialize a constant
or a method.

Change-Id: Ia5534a93a5413cd0458a251c022d0b655369502b

Change the order of functions in .debug_frame to save space.

Order the methods to help the compression of mini-debug-info.
This reduces the overhead of mini-debug-info from 3% to 2.5%.

Also remove oat_patches from the mini-debug-info since it is
does not make sense there.

Change-Id: I5077f664c3b01f07047b251489d2e9b05c8c9379

Do not write full method signature in mini-debug-info.

Write only fully qualified method name to reduce the size.
This approximately halves the amount of string data which
in turn reduces the overhead of mini-debug-info from 3.5% to 3%.

Change-Id: Iba9b1d5af77bd90a6c93912ff10bded243d716ce

Revert "Revert "Add option to generate compressed backtrace info.""

This reverts commit 8546cc9aeb05e866e1fb6a9e4130d53ea330baa8.

Change-Id: I676fdf9af27fa3b16fa8921778ff8832ab8c437d

Merge "[WIP] ART Mterp: fix for hidden gc roots"

ART: Remove some unnecessary mutator lock annotations.

The StackReference<> pointer held by a Handle<> can be used
without holding the mutator lock. We already do that when
we copy Handle<>s around. Only accessing the actual content
of the pointed-to StackReference<> needs to be done while
holding the mutator lock.

Change-Id: I5f93bd7e277383192f1f16dff6883ecb26387414

Merge "Generate DWARF line numbers only for points with dex register map."

Add dependency on liblzma.

It will be needed for compression of debug information.

Change-Id: I77340a3b328aae08d82018f611615af9aebc5aca

Merge "Disable run-tests causing occasional timeouts with CC."

Disable run-tests causing occasional timeouts with CC.

These tests sometimes time out on the concurrent collector
configurations:
- 004-ThreadStress
- 496-checker-inlining-and-class-loader

Disable them while we investigate.

Bug: 26786154
Bug: 26786304
Bug: 12687968
Change-Id: I169766eac656cd1281a169337c2205a9592c32a1

Merge "Fix non-Linux builds."

Fix non-Linux builds.

Do no expose a constant only used in Linux builds in
runtime/jdwp/jdwp_socket.cc.

Change-Id: I0f1de4fd36a9013b46b1393ea0cded41bae2b751

Merge "Improve gethostbyname_r use in art::JDWP::JdwpSocketState::Establish."

[WIP] ART Mterp: fix for hidden gc roots

To support moving gc, we must not hold onto any references solely
in registers across potential gc points.  This was happening during
returns, instance-of and check-cast.

[testing in progress]

Change-Id: I367750658c5716960737f0666e46800240fd392d

Merge "Reduce app image log spam"

Reduce app image log spam

Change to VLOG or file exists.

Bug: 26782386
Bug: 22858531
Change-Id: Ic73d110d44216c2846e4ea789070bea2af8043a3

Merge "Revert "Revert "Load app images"""

Revert "Revert "Load app images""

This reverts commit 1bc977cf2f8199311a97f2ba9431a184540e3e9c.

Bug: 22858531

Change-Id: Ide00bf3a73a02cba3bb364177204ad1b13f70295

Merge "Only create non-moving space at fixed address if zygote"

Only create non-moving space at fixed address if zygote

Pulled from app image CL, hopefully fixes dex2oat asan failure.

Change-Id: I022f7509b534679e5a4ef99ec7a0c622472cf740

Improve gethostbyname_r use in art::JDWP::JdwpSocketState::Establish.

Try a range of values for the work buffer size, instead of
using a fixed size.

Also, turn a couple of constants defined as macros into
constexprs.

Change-Id: I2354bfb51935bb0cad775c13d6cae06902b4a934

Merge "Use dlmalloc from external/dlmalloc instead of bionic/."

Merge "Revert "Add option to generate compressed backtrace info.""

Revert "Add option to generate compressed backtrace info."

This reverts commit 5fdcc3c931b70204fd8c491afa66f57f8428490f.

Change-Id: I9c1f5aad6933a46af6717e3a90a51f76111f9c8a

Generate DWARF line numbers only for points with dex register map.

Stackmaps without dex register map are useful to the runtime,
but they are not a reasonable stopping point for the debugger.

Change-Id: I393d2d293e6faf339c7560b250bcf3dad39d54cd

Merge "Fix MIPS64 booting problem"

Merge "Add option to generate compressed backtrace info."

Merge "Revert "Optimizing: double-negated bitwise operations simplifications""

Revert "Optimizing: double-negated bitwise operations simplifications"

Fails compiling the Wallet.apk with:

dex2oatd F 40736 41007 art/compiler/optimizing/optimizing_compiler.cc:194] Error after instruction_simplifier: art::SSAChecker: Instruction Add:59 in block 4 does not dominate use Or:153 in block 4.

This reverts commit 96798493170521691d709be50dd2102ead47b083.

Change-Id: Ia4b02e62e6133aa104f5db12ba82d5561b6fc090

Merge "Revert "X86: Use the constant area for more operations.""

Revert "X86: Use the constant area for more operations."

Hits a DCHECK:

dex2oatd F 19461 20411 art/compiler/optimizing/pc_relative_fixups_x86.cc:196] Check failed: !invoke_static_or_direct->HasCurrentMethodInput()

This reverts commit dc00454f0b9a134f01f79b419200f4044c2af5c6.

Change-Id: Idfcacf12eb9e1dd7e68d95e880fda0f76f90e9ed

Merge "Optimizing: double-negated bitwise operations simplifications"

Merge "X86: Use the constant area for more operations."

Merge "Revert "Load app images""

Revert "Load app images"

Fails when a method is duplicated (see test 097-duplicate-method)

Bug: 22858531

This reverts commit f7fd970244f143b1abb956e29794c446e4d57f46.

Change-Id: Ib30ae5be00cc568e799290be6b3c8f29cbbe4c20

Merge "Disable the CC collector verbose log."

Merge "Temporarily disable some libcore tests for the CC collector."

Disable the CC collector verbose log.

To reduce the amount of buildbot libcore test logs.

Bug: 12687968
Change-Id: Idadca6622c745ef12629ccb26dcbdc9ebc379a00

Temporarily disable some libcore tests for the CC collector.

Bug: 26711853
Bug: 12687968
Change-Id: Id791ac14afd7aff74f92aff88adefe4dcad7a20f

Merge "Bump image version"

Bump image version

Version bump in previous CL was rebased out.

Change-Id: Ifcd76b852716cc2a6c08287e83b9861621df7605

Merge "Load app images"

Load app images

Support in-place patching of the app image based on boot image
location and app oat location. Only loads for art run test so far
since we do not automatically generate app images for app installs.

N5 maps launch time (~200 runs):
Before: 930ms
After: 878.18ms
After + image class table: 864.57ms

TODO:
Oatdump support.
Store class loaders as class roots in image.

Bug: 22858531

Change-Id: I9cbc645645e62ea2ed1ad8e139e91af7d88514c1

X86: Use the constant area for more operations.

Allow FP HNeg to use the constant area to hold the constant to flip the
sign bit.

Enhance some math intrinsics to allow the use of the constant
area: Abs{Float,Double}, {Min,Max}{FloatFloat,DoubleDouble}.

Allow compares of floats/doubles to constants using the constant area.

These eliminate almost all uses of loading constants from the stack.

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

Merge "Raise ulimit for run-test 961."

Use dlmalloc from external/dlmalloc instead of bionic/.

Bug: http://b/17207577
Change-Id: I86d731e5236947c2cc21cbd17de29bd310a60a5b

Raise ulimit for run-test 961.

Run-test 961 was rubbing against the ulimit, causing spurious
failures. This raises the ulimit the test runs under.

Bug: 26733951

Change-Id: I43f7c5f8e5e9e23ed139bd1fcc6bf24c526c546b

Add option to generate compressed backtrace info.

Add flag --generate-mini-debug-info which generates
LZMA compressed .symtab and .debug_frame, which are
sufficient to print java backtraces in libunwind.

If enabled, it increases the size of boot.oat by about 3.5%.

Change-Id: Ic3c2ef7704c05fa328720c6781ca2a9b8e3935a3

Merge "Improve documentation and assertions of read barrier instrumentation."

Merge "Allow x86 memory operands for 64-bit popcnt on x86."

Show registered native allocations in ahat.

Bug: 23130675
Change-Id: I1d7f41a47a956b30611429b9bd395ec5f9580209

Merge "Generate unique ID for each type during native debugging."

Generate unique ID for each type during native debugging.

This allows the native debugger to find dynamic types of variables.

Change-Id: I901022b7db7d3c1db9f4b678ebafcb4eefed4ba7

Merge "Create helper class for DWARF expressions."

Optimizing: double-negated bitwise operations simplifications

Generic instruction simplifications applying to bitwise operations when
both inputs are Not's. And and Or are handled by De Morgan's laws,
removing one instruction:
~a & ~b -> ~(a | b)
~a | ~b -> ~(a & b)
Xor is handled by this trivial relation, removing two instructions:
~a ^ ~b = a ^ b

The simplifications only happen when neither Not is used by other
instructions.

Change-Id: I5d5187af2f625c475c3e49466af6bc3e87595f8f

Create helper class for DWARF expressions.

This ensures we generate valid DWARF opcodes and
it slightly simplifies the debug info writer.

Change-Id: I2ef8a9222c9a129c58f500741acd209b67e2ecff

Merge "Revert "Revert "ARM64 Baker's read barrier fast path implementation."""

Fix MIPS64 booting problem

Add missing MarkGCCard in compareAndSwapObject intinsic.

Additionaly, don't do a null test in MarkGCCard if the value
cannot be null for MIPS64.

Change-Id: Iad50f9e6be8cd27fedb31abb00d5829498941696

Merge "Revert "Revert "Fast ART x86 interpreter"""

Merge "Revert "Revert "Inline methods with loops."""

Revert "Revert "Inline methods with loops.""

Bug: 26689526

This reverts commit 451ad8d1be9a1949ea3c3e3a713a9e76198a8b2d.

Change-Id: If484fe4c0744254dd7568fd5006e574d621a1855

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

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

Fails two checker tests:

458-checker-instruction-simplification
537-checker-jump-over-jump

This reverts commit 884e54c8a45e49b58cb1127c8ed890f79f382601.

Change-Id: I22553e4e77662736b8b453d911a2f4e601f3a27e