Merge "Add MapAnonymous handling for null error_str"

Add MapAnonymous handling for null error_str

We use MapAnonymous with null error_str for app image loading when
we want to fail quickly. Also avoid doing CheckNonOverlapping in
CheckMapRequest if error_msg is null. The result from
CheckNonOverlapping is unused and CheckNonOverlapping is slow since
it creates a backtrace map.

Bug: 22858531
Bug: 26746779
Change-Id: I8605ec0b9d9ae554a4b74f2606fa7dd81ade9021

Merge "Don't make this script fail a buildbot run."

Don't make this script fail a buildbot run.

Change-Id: I000d69c1a33eca6ca98f7fd1096bc9af7d22b4e8

Merge "ARM: Implement Reverse bits and bytes intrinsic."

Merge "MIPS32: Implement intrinsics from java.lang.Math:"

Merge "Remove unreachable code paths in constant folding."

Merge "Fix indentation of copyright headers in some ART run-tests."

Fix indentation of copyright headers in some ART run-tests.

Reported by Kevin Brodsky <kevin.brodsky@linaro.org>.

Change-Id: Iec3d69aa33d3b6a31f1298607e0ee6bc4cb6d9c0

Merge "ART: Enable JitProfiling for x86 Mterp"

Remove unreachable code paths in constant folding.

Change-Id: I7ffb361711c87f6b1b98d172d2cfdf9b2ba65607

ARM: Implement Reverse bits and bytes intrinsic.

 - IntegerReverse
 - LongReverse
 - IntegerReverseBytes
 - LongReverseBytes
 - ShortReverseBytes

Change-Id: I3ec202696b245148a0237ff6e46ac3f1a3f8402a

Merge "Add a script to symbolize buildbot crashes."

Add a script to symbolize buildbot crashes.

Change-Id: I6e85a2f37db84a8d06c7258de1c60eaffc78e44b

Merge "ART: Allow method references across oat files for multi-image, 2nd attempt."

ART: Allow method references across oat files for multi-image, 2nd attempt.

These were disabled because we didn't have sufficient
information about the multi-image layout when processing
link-time patches in OatWriter. This CL refactors the
ELF file creation so that the information is available.

Also clean up ImageWriter to use oat file indexes instead
of filenames and avoid reopening the oat file to retrieve
the checksum.

Change-Id: Icc7b528deca29da1e473c8f079521a36d6c4892f

Merge "Please Clang with respect to stack frame limits in dex2oat."

Merge "Introduce partial code cache collection."

Merge "Associate slow paths with the instruction that they belong to."

Associate slow paths with the instruction that they belong to.

Almost all slow paths already know the instruction they belong to,
this CL just moves the knowledge to the base class as well.

This is needed to be be able to get the corresponding dex pc for
slow path, which allows us generate better native line numbers,
which in turn fixes some native debugging stepping issues.

Change-Id: I568dbe78a7cea6a43a4a71a014b3ad135782c270

Merge "Remove HNativeDebugInfo from start of basic blocks."

Remove HNativeDebugInfo from start of basic blocks.

We do not require full environment at the start of basic block.
The dex pc contained in basic block is sufficient for line mapping.

Change-Id: I5ba9e5f5acbc4a783ad544769f9a73bb33e2bafa

Merge "Show battery level fo devices on buildbot."

ART: Enable JitProfiling for x86 Mterp

Adds branch profiling and enables for x86.

Change-Id: I875034d5bc6b639df08a0236e415195521994238

Show battery level fo devices on buildbot.

Change-Id: Ic8279d1905c8a6efe08ca190f1bb7332842e1904

Merge "Make JNI work correctly with default methods."

Make JNI work correctly with default methods.

Also adds some tests for JNI and DefaultMethods.

Bug: 27259142
Bug: 24618811

Change-Id: I31222e3e41059d803be1dbb0f40e1144ac4bf457

Merge "Improved instruction + offset hunting."

Improved instruction + offset hunting.

Rationale:
This is generally useful for anything using this method
but in particular for deopting something like

 bs[  off] = (byte)(n >>> 24);
 bs[++off] = (byte)(n >>> 16);
 bs[++off] = (byte)(n >>>  8);
 bs[++off] = (byte)(n       );

where the base + offset is hidden in the increments.
Occurs quite often in real-life code.

Change-Id: I3fa7d285a7368a179a26e590e8eee37f3b64c25d

Merge "Fix allocation tracking race"

Fix allocation tracking race

Need to instrument the stubs before calling
Heap::SetAlloctrackingEnabled or else we can get this DCHECK failing
DCHECK(!IsAllocTrackingEnabled())

Change-Id: I1fb5df217a2785baf22b4f57887cd323f13f0973

Merge "Revert "Compile checker run-tests with Jack""

Revert "Compile checker run-tests with Jack"

Tests still faill.

Bug: 25635944
Bug: 19467889

This reverts commit fbeeb47fc63ec85891198b51b2309bb15f26527e.

Change-Id: I8f49dc6e8d3eb5f0de10636ef2685b7a2e08f297

Please Clang with respect to stack frame limits in dex2oat.

This change enables Clang to compile dex2oat on MIPS64
without complaining about stack frames larger than what
`-Wframe-larger-than` allows.

Bug: 27310199
Change-Id: I441a4be499959a089d3c2eae1135eb0273c1e80b

Merge "Compile checker run-tests with Jack"

Merge "Fix profman path for the gtest."

Fix profman path for the gtest.

Change-Id: Id332c50658b7dc1eb1edf6a58fcbdde4c2887d27

Compile checker run-tests with Jack

Bug: 25635944
Bug: 19467889
Change-Id: I3c3fcbcbd2f6c142dbea279a9ecd9d372e79b3b9

Introduce partial code cache collection.

It will collect code that is known unused (because it deoptimized),
and osr code.

bug:26846185

Change-Id: Ic27dfeb944efb2ca464039007ba365c1e0d4a040

Merge "Revert "Revert "Add profman tool: responsible to process profiles"""

Merge "Re-enable tests that were causing TimeoutExceptions on x86/x86-64."

Re-enable tests that were causing TimeoutExceptions on x86/x86-64.

Some libcore tests have been failing with a
java.util.concurrent.TimeoutException on ART Builbot's
x86/x86-64 concurrent collector configurations.  No longer
ignore these failures to see whether recent changes (namely
https://android-review.googlesource.com/#/c/204125) fixed
the issue.

Bug: 26507762
Change-Id: Ib188e194195fc11b539bd53a03db78ab80626f33

Merge "Don't allocate mspaces of less than a page."

Don't allocate mspaces of less than a page.

Fixes jit tests in debug mode.

Change-Id: I34565e03683cee063e26975a461d8e75ad4a205f

Revert "Revert "Add profman tool: responsible to process profiles""

This reverts commit 3da74687e42de7d33a8e75df9bd64374e650f75e.

Change-Id: Id005096bd8063c6c602744d4476d5eb7e0d34e90

Merge "Dex-wide ArenaPool scoping for AOT compilation"

Merge "Add a new kReservedCapacity to decide when to start GC code."

Merge "Fix static dex2oat  build"

Merge "Revert "Force Jack to consider multidex annotations""

Merge "Add regression test for alloc stack trace and unloading"

Add regression test for alloc stack trace and unloading

Test creates allocations in a different loader, then unloads it.
This would free the ArtMethod except that we now visit them as
roots.

Bug: 26849503
Change-Id: Ied9f5942ff9022b1cd39052363f119a630ffbaef

Merge "Visit stack trace roots"

Visit stack trace roots

We need to visit the declaring classes of all the methods in the
allocation stack traces to prevent class unloading for these
methods. If the class gets unloaded, it will free the linear alloc
resulting in hprof crashing during dumping.

Also a bit of clean up.

Bug: 26849503

Change-Id: I5cc57436294e4399388cb497819148b8a3b973d9

Merge "Revert "Add profman tool: responsible to process profiles""

Revert "Add profman tool: responsible to process profiles"

Needs a profile_assistant_test fix.

Bug: 26719109
Bug: 26563023

This reverts commit 6caefd983a800a063b219f1d3ed71b1416cecd70.

Change-Id: Ibdeb7385737dd7846ed861e0a95f083abb9aa974

Merge "Fix class_linker_test for valgrind."

Fix class_linker_test for valgrind.

Fix a memory leak.

Bug: 27156726
Change-Id: I4d9a2dc9cff447b6f2fd02432e018ae49766563a

Merge "Fix CC collector thread flip and JNI critical section deadlocks."

Merge "ART: Add Class.getEnclosingClass() to UnstartedRuntime"

MIPS32: Implement intrinsics from java.lang.Math:

- abs(double)           - abs(float)            - abs(int)
- abs(long)             - max(double, double)   - max(float, float)
- max(int, int)         - max(long, long)       - min(double, double)
- min(float, float)     - min(int, int)         - min(long, long)
- sqrt(double)

The math intrinsics:

- ceil(double)          - floor(double)         - rint(double)
- round(double)         - round(float)

aren't implemented because they require instructions which only exist
for MIPS64, or for MIPS32r6.

Change-Id: I943be3592b52a423fcb7ac40f46f38a5e2a58c50

Fix CC collector thread flip and JNI critical section deadlocks.

There's a bug in the synchronization between the CC collector's thread
flip and JNI critical sections where it incorrectly attempts to make
the synchronization scheme to be writer (GC) preference to avoid
starvation of GC in the presence of frequent JNI critical section
enter/exit. This could cause a deadlock between them if a thread
enters a nested JNI critical section after a thread flip occurs. This
is reproduced in the added test.

The fix is to use a thread local disable counter in addition to the
global counter to detect a nested enter by the same thread and avoid
waiting if nested.

Bug: 19235243
Bug: 12687968
Change-Id: Idf7720a6906c9ea508219935af3727f76680d2d8

ART: Add Class.getEnclosingClass() to UnstartedRuntime

Required to initialize classes calling getSimpleName().

Bug: 27265238

(cherry picked from commit fb5c22e28030cc9ba8424b7f6f70677ec3ba3afc)

Change-Id: Id63dac5b75b3bbcdfe80f73f2d9d76dea3996823

Merge "ART: Enable JitProfiling for Arm Mterp"

Merge "Optimizing/x86-64: Use MOVL in Load64BitValue() if IsUint<32>()."

Fix static dex2oat  build

Change-Id: I1a5fb726fa72d9ccd083f90a818e245f075fc89c

Optimizing/x86-64: Use MOVL in Load64BitValue() if IsUint<32>().

Change-Id: Ie8bfb1861a384d0906f2aff9e8a94be0925c65b6

Merge "Add Math.round tests for large integers."

Merge "Revert "Workaround JDWP failures""

Revert "Force Jack to consider multidex annotations"

This reverts commit 8cc049c2354af3964d2c15223a9c8d4f57a6691b.
The bug that necessitated that workaround was fixed.
Bug: 26441710

Change-Id: Idf238e497052cb983fd56f50a36f0f7998240fa8

Merge "Add profman tool: responsible to process profiles"

Add profman tool: responsible to process profiles

This pulls out profile parsing from dex2oat into a separate tool.
Some additional refactoring:
- better return codes
- dex2oat now accepts only one profile file

This is the first step towards support secondary dex files and
extracting profiles out of the code_cache directory.

Bug: 26719109
Bug: 26563023

(cherry picked from commit d81c289e58f4ced8ec7674fd377c356669f97227)

Change-Id: I34d408faa318e866e1a23d4d04c369131ee5012b

Merge "Track libcore change b5d045f7e4023c0ba9a42fb95efad4c30dfc085f"

Track libcore change b5d045f7e4023c0ba9a42fb95efad4c30dfc085f

Change-Id: I40bfd7f85c3972cb21c1ddde07ae9b5727df8909

Add a new kReservedCapacity to decide when to start GC code.

Change-Id: Ib9d0ad9bc86c66ac29975f7f907f79f842342500

Revert "Workaround JDWP failures"

This reverts commit 8af4d1c84682a3108b1e5b66fd7442a69eb715c0.

Bug: 27262745
Change-Id: I4254fad87cfdd2f9c044722c2f5766c8b14aaa9c

Merge "Ignore GZIPInputStreamTest#testLongMessage failures with CC."

Ignore GZIPInputStreamTest#testLongMessage failures with CC.

This test sometimes fails with a
java.util.concurrent.TimeoutException on ART Builbot's
x86-64 concurrent collector configuration:
* libcore.java.util.zip.GZIPInputStreamTest#testLongMessage

Disable it while we investigate.

Bug: 26507762
Change-Id: I319a812ff7e6e95857879b2e653c886020cb462b

Merge "Track memory usage of code cache."

Merge "Use adds instead of add to set the N flag."

Use adds instead of add to set the N flag.

That flag will be used by the following bmi instruction.

Fixes 109-suspend-check on arm32.

Change-Id: I07ee069294e7a0d9709718375bce1e21e3db6529

Merge "Enable bytecode tracing in ART FI"

Merge "x86 ART FI clean-up"

Enable bytecode tracing in ART FI

Trace bytecode execution in Fast Interpreter similar to
other interpreters.

Update TraceExecutionEnabled function to switch on tracing.

Change-Id: Icabc17871c8198b11cd4c3dbfaa901e4fbf67946
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

Merge "Support image classes filtering in image writer"

Support image classes filtering in image writer

New logic prunes classes that have any dependency on a non-image
class. This enables creating smaller images with class profiling.
Code is from class profiling CL.

Added a CHECK for guarding against creating an app boot image with
existing boot image(s).

Bug: 22858531

(cherry picked from commit 7fccc2de5a7ea1bb8241d50869a2a4e44b35767f)

Change-Id: Idcc1de4367b5368bbbd2881cbd63975a646d0831

Merge "Add VLOG(collector) to reduce log spam"

Add VLOG(collector) to reduce log spam

VLOG(gc) is enabled from AndroidRuntime, using it too much produces
log spam.

Bug: 22858531

(cherry picked from commit a6e1c126299586932ecd3c1133a55a6f8e1107fc)

Change-Id: If60902a8d7855cdabd80208b9b385ce4ba4d9e5f

Merge "Revert "Do not build art with clang for mips64""

ART: Enable JitProfiling for Arm Mterp

Also, fix missing shadow frame clear operation for the 64-bit
shift operations.

Change-Id: Icea95b3aeb1d6d36ea92336fb738cf56edd92da4

Merge "Workaround JDWP failures"

Revert "Do not build art with clang for mips64"

This reverts commit 610a1420fe400d3f638fc0dadb1a012d2ac3ad19.

Bug: 25928358
Change-Id: I3fe8777ead2bd067c071277a7c15b2442f15567f

Workaround JDWP failures

Compile JDWP tests in 1.7 to workaround debug info issues.

Bug: 27262745
Change-Id: I4630a0346d5a99f94895557d705dad310735b45f

Merge "Revert "Revert "Revert "Revert "ART: Enable Jit Profiling in Mterp for arm/arm64"""""

Track memory usage of code cache.

Needed once we start having a strategy based on how full the code
cache is.

Change-Id: I181ad05e1c216afabf106f91ac54bbf144d7406a

Merge "The JIT does not need VerifedMethod nor CompilerCallbacks."

Revert "Revert "Revert "Revert "ART: Enable Jit Profiling in Mterp for arm/arm64""""

This reverts commit 5d03317a834efdf3b5240c401f1bc2ceac7a2f25.

We need to catch all possible cases in which new instrumentation appears
or the debugger is attached, and then switch to the reference interpreter
if necessary.  We may, in a future CL, use the alt-mterp mechanism to accompish
this (as did Dalvik).

Only enables Arm64 for now.  Once it survives extended testing, will enable
arm and update x86.

Updated OSR handling to match other interpreters.

Change-Id: I076f1d752d6f59899876bab26b18e2221cd92f69

Merge "Fix braino in single frame deopt."

Dex-wide ArenaPool scoping for AOT compilation

The Arena Pool is a structure that holds memory allocated by the Arena
Allocator, preventing system allocations and deallocations to occur too often
during AOT compilation. Currently, the Arena Pool holds memory it allocates
during the whole AOT compilation process. Unfortunately, such a behavior
generates memory usage overhead, which is exacerbated by bigger applications
such as Facebook.

In this particular app, method size imbalance provokes unnecessary memory
pressure, as one method might require a lot of arena allocations that won't be
used in the remaining compilation. Because the compiler memory footprint keeps
increasing during AOT compilation, the memory pressure becomes very high.

The proposed patch is an attempt to find a tradeoff between allocations /
deallocations time overhead, and the aforementioned memory pressure overhead
resulting of the allocations being held by the Arena Pool. The patch adds a
feature freeing up all memory allocated in the Arena Pool after each dex file
is done compiling.

We have measured no significant AOT compile-time overhead with the patch
(<0.3%), because the selected dex-file granularity is coarse enough. In the
meantime, it provides significant memory footprint improvements. The impact is
especially big with Facebook, because this app has a few methods generating
huge memory footprint peaks (peak native heap footprint goes down 34%).

Change-Id: I27e867e6a20b8a6c28a82cb83140941a8c2b5847
Signed-off-by: Jean-Philippe Halimi <jean-philippe.halimi@intel.com>

Fix braino in single frame deopt.

handler_method_header_ is actually the OatQuickMethodHeader
of the caller of the method that is deoptimizing (as we will
resume at its call site). We should use the OatQuickMethodHeader
of the method that is deoptimizing and pass it to the code cache
to invalidate the method.

Change-Id: I2b61a96b8415e9477e3ac902c54a314aeb0811a0

Merge "Use image oat file instead of image header for immune spaces"