Merge "Factor out common code for SetPendingNext."

Merge "ART: Add missing swaps to MIPS codegen"

ART: Add missing swaps to MIPS codegen

Change-Id: I0fb50280ddf43f817d991c15d3b6cdeb4635d6c2

Merge "Do not enforce deterministic compilation with read barriers."

Merge "Add a comment to oatdump for 0 size symbols"

Merge "Change log to avoid polluting logcat."

Add a comment to oatdump for 0 size symbols

The symbolizer inside oatdump generates 0 size symbols for the
trampoline symbols. Add a comment explaining why we emitting them with
0 size.

Change-Id: I16ed87554681ce57aafc26517a1e674647b5f9ac

Merge "ART: Fix gtest after liveness CL"

ART: Fix gtest after liveness CL

Change-Id: I2d029044cebe6e1ee7d7efb2e20541060f88c07c

Merge "Implement first kind of polymorphic inlining."

Implement first kind of polymorphic inlining.

Add HClassTableGet to fetch an ArtMethod from the vtable or imt,
and compare it to the only method the profiling saw.

Change-Id: I76afd3689178f10e3be048aa3ac9a97c6f63295d

Merge "ART: Implement HSelect"

ART: Implement HSelect

This patch adds a new HIR instruction to Optimizing. HSelect returns
one of two inputs based on the outcome of a condition.

This is only initial implementation which:
 - defines the new instruction,
 - repurposes BooleanSimplifier to emit it,
 - extends InstructionSimplifier to statically resolve it,
 - updates existing code and tests accordingly.

Code generators currently emit fallback if/then/else code and will be
updated in follow-up CLs to use platform-specific conditional moves
when possible.

Change-Id: Ib61b17146487ebe6b55350c2b589f0b971dcaaee

Merge "ART: Implement support for instruction inlining"

ART: Implement support for instruction inlining

Optimizing HIR contains 'non-materialized' instructions which are
emitted at their use sites rather than their defining sites. This
was not properly handled by the liveness analysis which did not
adjust the use positions of the inputs of such instructions.
Despite the analysis being incorrect, the current use cases never
produce incorrect code.

This patch generalizes the concept of inlined instructions and
updates liveness analysis to set the compute use positions correctly.

Change-Id: Id703c154b20ab861241ae5c715a150385d3ff621

Merge "Emit symbol names for trampoline symbols in oatdump"

Merge "Fix ARM & ARM64 UnsafeCASObject intrinsic with heap poisoning."

Do not enforce deterministic compilation with read barriers.

Forcing determinism of the boot image compilation (on host)
does not work when read barriers are enabled, as the former
switches the GC to a non-concurrent one by passing the
option `-Xgc:nonconcurrent`.

Also make dex2oat abort when invoked with
`--force-determinism` while read barriers are enabled.

Bug: 12687968
Bug: 26687569
Change-Id: I37b388fccbe4502a90def8031a898f1a40df6a9f

Merge "Re-enable some run-tests with CC after mterp fix."

Merge "Don't crash if the perf map creation failed"

Don't crash if the perf map creation failed

The perf map file is used in the jit compiler to emit the symbol name
for the start address of each jitted function. The file is stored in
/data/misc/trace (previously /data) what is only accessible on
userdebug builds. This change modifies the code to work on user builds
with not using the perf map file if we failed to create it.

Change-Id: Icfecd4bdab94ffc528ec218f3ac2b872fbdacf37

Change log to avoid polluting logcat.

Change-Id: I804afe9e3a78876f4fc2d74ae084419530435840

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

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

This reverts commit 2818dbcd75ea9beadcba9d18e2f68523108d0cf5.

Change-Id: I92b2b60b4f08f50cacfea4132f1c28cfbd628f1a

Merge changes from topic 'pointer_conversion'

* changes:
  Pass template args down
  Fix pointer conversion error in SetElementPtrSize

Pass template args down

Fixes failing DCHECK in dex2oatd

(cherry picked from commit 3df47c1c675e62cf0e5853699149c1181a48b28f)

Change-Id: I96a3b582705e4bffa39ee8e24ec43ff0d7a57ccd

Fix pointer conversion error in SetElementPtrSize

SetElementPtrSize used to cast from pointer to uint64_t which
could sign extend with 32 bit dex2oat for 64 bit app image.
Also removed C style casts and added more logging.

Bug: 22858531

(cherry picked from commit d22886357d4c53f74ca298faa7ad2555794e17e0)

Change-Id: I479124328fbc497cfcae20c43b4b55827e446527

Merge "ART: Add option to ensure deterministic compilation"

ART: Add option to ensure deterministic compilation

To ensure reliable stable generation of a boot image, add a flag
for forced determinism, trading compile time for a deterministic
output.

We have to run certain passes in the compiler-driver single-threaded.
It is also necessary to try to make the heap layout deterministic.
Switch to nonconcurrent GC, use the free-list implementation for LOS,
and try to allocate the main space at a known location. This is best
effort at the moment.

To allow the compiler phase to be parallelized, const-strings need
to be created beforehand.

The identity hashcode seed needs to be pinned.

Besides the Dex object we also need to null the DexFile pointer in
dex caches.

For classes, we need to remove the clinit thread ID.

Fix oatdump alignment padding accounting.

Bug: 26687569
Change-Id: Ia82120e8f715bb3691d861817b12778ac677355a

Merge "reflection: Add new 1.8 AnnotatedElement methods and tests"

Merge "ART: Make sure dex files are verified in the compiler"

reflection: Add new 1.8 AnnotatedElement methods and tests

Also:
* Bump the size of the Class vtable size since 3 new virtuals were
added to java.lang.Class
* Refactor Method#isDefault test into a separate file within 048 test.

Change-Id: I8420a4d208bb60874a9cf996766313c7d5058c45

ART: Make sure dex files are verified in the compiler

Follow-up to 9bdf108885a27ba05fae8501725649574d7c491b. Make sure
that dex files from the oat writer are verified with the dex file
verifier.

Bug: 26793137
Bug: 26808512
Change-Id: I1a5f51751491eead21d8f9f1b31e37c7374c72a5

Merge "Move heap trim outside of zygote creation lock region"

Merge "Clear dalvik-cache before running test on host."

Merge "Implemented checker/unit test for various operations."

Move heap trim outside of zygote creation lock region

Fixes lock order violation due if non moving space is a RosAllocSpace
since the trim requires acquiring the mutator lock.

Change-Id: Iff0b8c943e261702eda65b3c255fda24a6fe14c2

Clear dalvik-cache before running test on host.

The test 137-cfi is run twice and the stale data causes problems.

Change-Id: I631cc73c8e832534d0a276b7d23a654adeacebee

Merge "ART: Allow --no-inline-from to specify multiple dex files."

ART: Allow --no-inline-from to specify multiple dex files.

This will allow tests to specify arbitrary dex files from
which we should not inline, in addition to core-oj, so that
we can test the related functionality. Additionally, should
the core-oj.jar core grow beyond a single classes.dex and
require a multi-dex .jar, this change makes sure that we
prevent inlining also from the extra classes<N>.dex files.

Change-Id: I74da4839bf9bb405dd62ad80563bf646a7a65dd9

Emit symbol names for trampoline symbols in oatdump

Modify the symbolizer functionality of oatdump to emit symbols for
the trampolines in the begining of each oat file.

Change-Id: I083c160ff54784f904743f87bcc18881e6ce0603

Re-enable some run-tests with CC after mterp fix.

These tests might have been affected by a moving GC issue in
the fast interpreter:
- 004-ThreadStress
- 141-class-unload

Re-enable them so that we can monitor them on the concurrent
collector configurations.

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

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."

Fix ARM & ARM64 UnsafeCASObject intrinsic with heap poisoning.

Ensure aliasing does not make us poison/unpoison the same
value twice.

Also extend testing of Unsafe.compareAndSwap* routines.

Bug: 26204023
Bug: 12687968
Change-Id: I29d7e5dd2a969845e054798f77837d20e3c18483

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

Factor out common code for SetPendingNext.

Change-Id: I6c1f9cd6da7b2c21714175455e61479273d3669f

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

Implemented checker/unit test for various operations.

Rationale:
It's test time! These new checker tests verify that various
common integer operations are recognized as intrinsic.
Furthermore, some functionality testing is done to ensure
that architectural specific implementations are correct.

Change-Id: I8ac02a083ef1110c9baf274fbf75d931167e9ba5

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