Merge "Compile JNI stubs for verify-profile and interpret-only."

Compile JNI stubs for verify-profile and interpret-only.

This is the intended behavior to have a good JNI transition
performance.

Bug: 28902384

(cherry picked from commit 8c185bf0c6f18a5349bc87a7e3751ba06d90f461)

Change-Id: I52767909b916ada3c619206c8838b85bff5ac316

Merge "Enforce class definition ordering rules in v37 Dex files."

Merge "Avoid the need for the black color for the baker-style read barrier."

Avoid the need for the black color for the baker-style read barrier.

We used to set marked-through non-moving objects to black to
distinguish between an unmarked object and a marked-through
object (both would be white without black). This was to avoid a rare
case where a marked-through (white) object would be incorrectly set to
gray for a second time (and left gray) after it's marked
through (white/unmarked -> gray/marked -> white/marked-through ->
gray/incorrect). If an object is left gray, the invariant would be
broken that all objects are white when GC isn't running. Also, we
needed to have an extra pass over non-moving objects to change them
from black to white after the marking phase.

To avoid the need for the black color, we use a 'false gray' stack to
detect such rare cases and register affected objects on it and change
the objects to white at the end of the marking phase. This saves some
GC time because we can avoid the gray-to-black CAS per non-moving
object as well as the extra pass over non-moving objects.

Ritzperf EAAC (N6):
  Avg GC time:  232 ->  183 ms (-21%)
Total GC time: 15.3 -> 14.1 s  (-7.7%)

Bug: 12687968
Change-Id: Idb29c3dcb745b094bcf6abc4db646dac9cbd1f71

Merge "Revert "Revert "Move rewritten StringFactory call results into dex registers for deopt"""

Merge "Do not set property jpda.settings.syncPort in JDWP tests."

Merge "Only fill methods with 0xFE on debug builds"

Only fill methods with 0xFE on debug builds

The GC scans classes without holding any locks, we can not fill
the methods if it is running. Added a GC critical section to address
this. Fixes random crash when scanning classes' methods.

Only for debug builds to not hurt performance.

(cherry picked from commit 22bd2a1b5ec2a5038cc3ae1964781f30aef0315f)

Bug: 28699001
Change-Id: If96155eaf3fc0e6df31f57dcf32fbd4063b09345

Revert "Revert "Move rewritten StringFactory call results into dex registers for deopt""

Potential gc points can make the result value stale. We now set the result value
to null proactively once it's moved to shadow frame registers. IsStringInit()
is written in a way that does string comparison instead of requiring method
resolution so that it doesn't have a gc point. Also we don't cache the callee
method during frame unwinding since the method may be rewritten already.

Bug: 28555675

Change-Id: Ic51511a4a0fc84a852d8d907f91e7835f49ac478

Do not set property jpda.settings.syncPort in JDWP tests.

Let the OS choose the port automatically.

Bug: 28869570
Change-Id: I79775f9e72f7f8d0f9ec3d68c6f0bdb057ca13ff

Merge "ARM: Add new String.compareTo intrinsic."

ARM: Add new String.compareTo intrinsic.

Benchmarked on Nexus6P big, little, and all cores. The new intrinsic is
faster than pStringCompareTo for compare lengths on [1,512], so the
runtime call is no longer needed.

Change-Id: If853b592dfc5e561ea3389b51729f37a2c89c18e

Merge "ARM64: Move BIC after branch in StringCompareTo intrinsic."

ARM64: Move BIC after branch in StringCompareTo intrinsic.

Change-Id: Ic785dd93daab73b4b37917097c8c8ea2de797ed2

Merge "Add missing check for IsShuttingDown in profile saver"

Merge "ART: Check whether an oat file exists before opening"

Add missing check for IsShuttingDown in profile saver

Bug: 28814718

(cherry picked from commit 0233a413ba42aa34a92c357c8dcfbe48871788b9)

Change-Id: I709fc30147047c7a420cd6ff2f0c3b57c54c2021

ART: Check whether an oat file exists before opening

The two-stage process of dlopen and our own loader is expensive.
Considering that the OatFileAssistant needs to check two locations,
one of which is unlikely to have a file for the common case of apps
installed in data, do a file existence check before attempting to
load the file.

Remove an unused variable.

Saves about 0.25ms on a N6P.

Bug: 28801010

(cherry picked from commit d9a720bcb5df2c135f161c7c6da2d25db4400dca)

Change-Id: I0aff9c276c0f539723f8a92663f14ce72160c44d

Merge "Do not allow OSR jump while debugging is active."

Merge changes I0bbdf83d,I70156f98

* changes:
  ART: Fix Mac build
  ART: Optimize PreSetup pass for oat-file open

Do not allow OSR jump while debugging is active.

Rationale:
This prevents dangerous transitions between interpreter
and osr compiled code while e.g. single stepping through
a method that is becoming hot.

BUG=28672961

Change-Id: Ife8e641c05b3e3eb01284212270595c8677d6673

Merge "Clean up JNI calling convention callee saves."

Merge "MIPS32: java.lang.Math.round(float)"

Merge "Fix profile saver initial wait"

Clean up JNI calling convention callee saves.

Precalculate callee saves at compile time and return them
as ArrayRef<> instead of keeping then in a std::vector<>.

Change-Id: I4fd7d2bbf6138dc31b0fe8554eac35b0777ec9ef

Merge "Revert "Revert "ART: Reference.getReferent intrinsic for x86 and x86_64"""

Fix profile saver initial wait

Was using TimedWait for the initial wait, but this can return
early if the process receives any interrupt (due to futex). The
solution is to use a wait loop. If the wait returned early, not
enough classes were including in the profile. This negatively
affected application launch times in some scenarios.

Sample wait times before the change:
Waited 2.002s
Waited 97.808ms
Waited 182.676ms
Waited 2.000s
Waited 1.678s

Bug: 27688727

(cherry picked from commit 0ec065d55ccc4eb0956b0a2231bcc8c1a6d1d273)

Change-Id: I7cfdda9dcff942c7618d6022a0c31f872fb76fcf

ART: Fix Mac build

Follow-up to 9821cceebf077cb747aed543582b3d740a03ea7f.

Bug: 28801010

(cherry picked from commit 17fc4bda7987654709dcc24ea5a7bd6a859e0966)

Change-Id: I0bbdf83d64ea81e56824aa19bfae3baa341a2b73

ART: Optimize PreSetup pass for oat-file open

The PreSetup pass for opening an oatfile with dlopen parses all
shared objects and tries to retrieve the corresponding mem map
(so as to have a fake MemMap to reserve the space for any scanning
operations and so on). Optimistically optimize this pass by counting
the number of shared objects before the dlopen, and skipping as
many afterwards.

This saves about half a millisecond on an N6P.

Bug: 28801010

(cherry picked from commit 9821cceebf077cb747aed543582b3d740a03ea7f)

Change-Id: I70156f9875e89c5af3c9e36e8f01c5662e3f736e

Merge "Allow libcore and JDWP tests to be executed without JIT."

Allow libcore and JDWP tests to be executed without JIT.

To use AOT compiling, pass `--no-jit` option to
run-libcore-tests.sh and run-jdwp-tests.sh

Change-Id: I16efce1f48747ecbf8b900427be73c5694b5fb13

Merge "Catch classes inheriting from themselves in the class linker."

Merge "Revert "Move rewritten StringFactory call results into dex registers for deopt""

Revert "Move rewritten StringFactory call results into dex registers for deopt"

This reverts commit 8ca33bf04060fadd5b35fa93fa56547c62fe52e7.

597-deopt-new-string is failing.

Bug: 28846692
Bug: 28555675
Change-Id: Ibfb59ec36e089c987ea64e4af4ca3709e536412a

Merge "Make a fake libart for misbehaving apps."

Merge "Move rewritten StringFactory call results into dex registers for deopt"

Catch classes inheriting from themselves in the class linker.

(cherry picked from commit 90328ac545f65759a8e4fb217a75332906795518)

Bug: 28685551
Bug: 27682580
Bug: 28830038

Change-Id: If568013bf3c82c1df9b282522712d9af5ca5945d

Merge "Mark concrete HIR instructions as FINAL."

Mark concrete HIR instructions as FINAL.

This allows the compiler to apply more optimizations.

Change-Id: Ic7d8a457ea4e7d5853195cc4b56482703a1176d5

Enforce class definition ordering rules in v37 Dex files.

These new checks also reject classes extending or
implementing themselves directly or transitively within
a single Dex file.

Bug: 28812524
Bug: 28797012
Bug: 28685551
Bug: 27682580

Change-Id: I51c43f8b4b6a7abaade63682411e61f21767f5d4

Merge "ART: Fix systrace monitor logging"

ART: Fix systrace monitor logging

Thinlock unlocking was incorrectly unconditionally ending a block.

Bug: 28423466

(cherry picked from commit 825ab1c3f44fd84e5967f023c2dd8d36c6906f08)

Change-Id: Ifaebd9e959041e157e292d4cba05675a37e9c700

Merge "ART: Blacklist a libcore test for investigation"

ART: Blacklist a libcore test for investigation

Blacklist dalvik.system.DexClassLoaderTest#testDexThenPathClassLoader
for investigation.

Bug: 28826195
Change-Id: Ibff00293af3c7bc272e3b419f4065c0fd8c069cf

Make a fake libart for misbehaving apps.

Bug: 27775991
Change-Id: I22864aa22d4864d85e055a5b1d3ff1faf122011b
(cherry picked from commit b270b9e6cba453209e226b0f621c75f2e51f122a)

Move rewritten StringFactory call results into dex registers for deopt

Bug: 28555675
Change-Id: I9236df283f2e83ca5dcde01f73dc0522d745cd59

Merge "Fix an assert during jdwp debugging."

Merge "Keep oat file unique pointers until they are no longer used."

Merge "ART: Return native debuggable status of the app."

Keep oat file unique pointers until they are no longer used.

Before, the unique pointer would be deleted before we were finished
using the dex file we opened from the oat file.

Bug: 28359191

(cherry-picked from commit a6d46161aea07ebd1cbd6ab78b2b323f940e9c1e)

Change-Id: I737d17536967af48d8748bde2b266a82410830b7

ART: Return native debuggable status of the app.

Bug: 27942453

(cherry picked from commit dfbbeef151d52ec9c5bbc9f12449d4c74b8ec19e)

Change-Id: Ie474f4d830237d015436158829264b8bdf0a7194

Merge "Fix debug leftover in profman"

Fix debug leftover in profman

(cherry picked from commit 309eb7b11e6250b974c574f256d1c760cafa814e)

Bug: 27600652
Change-Id: I32466623379928cdaad839303e6ea3a39d673cd4

Merge "Support to pass <uses-library> option through to dex2oat."

Support to pass <uses-library> option through to dex2oat.

This change takes an app's shared libraries specified by <uses-library>
and passes it through to dex2oat to be used during compilation.

Part of a multi-project change.

Bug: 26880306

(cherry-picked from commit 26e8a2f150cd7f7195a10650ab8a5b6fa5014bc8)

Change-Id: I72a352abdfc37eacd8bedfa6c218e3809ca8e39c

Merge "Add null checks to weak DexCache references"

Add null checks to weak DexCache references

Bug: 28452385
Change-Id: I88243af47b8e091c360927d1bdb867770f092475

Merge "Clean up String.indexOf() intrinsics."

Merge "Revert "Reject classes inheriting from themselves directly or transitively.""

Revert "Reject classes inheriting from themselves directly or transitively."

This reverts commit 498bb81f8725e2cf8f5ce1aa6b49ab5a9c5bb4c9.

Reverting not to break apps that do not follow the class
definition order rules.

Bug: 28797012
Bug: 28685551
Bug: 27682580
Change-Id: I2491b5aefe3ca8a364a05f7dfbc899c5a5757712

Merge "Revert "Reject classes implementing themselves as interface directly or transitively.""

Revert "Reject classes implementing themselves as interface directly or transitively."

This reverts commit 34598460015962699eb6b85b5fdb39400759280e.

Reverting not to break apps that do not follow the class
definition order rules.

Bug: 28797012
Change-Id: I77844a77f2d070563294fa0b01f8f68383c93fb8

Clean up String.indexOf() intrinsics.

Additional cleanup after
    https://android-review.googlesource.com/223260

Bug: 28330359
Change-Id: I88def196babec70123896ef581ec8d61bb1b9a9a

Merge "Adjust ProfileSaver strategy"

Merge "Visit IMT of all classes during image writing"

Visit IMT of all classes during image writing

We still need to visit the IMT for classes that does not have any
methods. It is required to visit the IMT of every class to assign
image offsets for the conflict table methods.

Added regression test.

Bug: 28707801

(cherry picked from commit 54689b7d5568dcf01ff42d69435d0af7ae4e4d8e)

Change-Id: I4806685c32df7f2707f6018e21a489bfd48af6f7

Adjust ProfileSaver strategy

Change to a notification based strategy (instead of polling):
- the JIT notifies the ProfileSaver everytime it attempts a compilation
or generate (in a task) a ProfileInfo object for the method
- once a certain number of hot methods notification are received,
ProfileSaver wakes up the thread to do the analysis
- a wake up does not equal to I/O since the during processing we might
realized that's actually not enough new data.
- ProfileSaver throtles the request and ensures it doesn't do the
processing more often than kMinSavePeriodNs.

The new strategy avoids waking the thread unnecessary during extended
idle periods when we don't JIT.

Bug: 26815940

(cherry picked from commit 5fbb0fe0625a1f76aa33cd78ccf1a17b00d8f6d1)

Change-Id: Ifbac3a53c025e00a3faab6784bbfa679c6b42acb

Fix an assert during jdwp debugging.

When debugger attaches, we patch entry points of methods in framework code.
During that process, it's possible that some method's declaring class isn't
resolved yet. We need to relax one assert for that case.

Bug: 28630805
Change-Id: I59fd488ad75417a64b52321677ffcac6ccdc5ce1

Merge "SafeStack-friendly stack protection in ART."

Merge "Make quick frames of proxy methods visited if they're constructors."

SafeStack-friendly stack protection in ART.

This code takes an address of a stack variable to find the current
stack top. With SafeStack, such variables can be allocated on a
separate, "unsafe" stack, generating a completely bogus address.

Use __builtin_frame_address(0) instead.

Change-Id: I84fe3161bdd01cebbb4e9bebf2aca86783cc6f15

Make quick frames of proxy methods visited if they're constructors.

Also adds a test case for getting annotations during moving GC.

Bug: 28510922

(cherry-picked from commit 12c08f03f65acc1c308491fc7ab6c57f682e2f24)

Change-Id: I0d5cc9d5bb8bbff6bd6b108f7486a63402a15993

Merge "Typos and stylistic fixes."

Typos and stylistic fixes.

Change-Id: I1c88af9bd1d2cac6c803c8c61088c569a3847e01

Merge "Reject classes implementing themselves as interface directly or transitively."

Merge "Reject classes inheriting from themselves directly or transitively."

Reject classes implementing themselves as interface directly or transitively.

Also enforce class definition ordering with respect to
implemented interfaces within a Dex file.

Change-Id: I1cb991b1a976933f952d45ed08d8a189000ae4f5

Merge "Fix the benchmark fail by caliper's change"

Revert "Revert "ART: Reference.getReferent intrinsic for x86 and x86_64""

This reverts commit 0997d24e67d78f2146ebae2888eda0d7d254789a.

ART_HEAP_POISONING=true mode is fixed.

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

Merge "Fix JNIEnv-on-wrong-thread abort message."

Fix JNIEnv-on-wrong-thread abort message.

Update an incorrect abort message that claimed that thread A
was using the JNIEnv from thread A (when the JNIEnv was in
fact from another thread). Now, the message is e.g.

  JNI DETECTED ERROR IN APPLICATION: thread
  Thread[1,tid=628,Native,Thread*=0xb038b400,peer=0x75efc1f0,"main"]
  using JNIEnv* from thread
  Thread[6,tid=640,Runnable,Thread*=0xaa890800,peer=0x12c00ca0,"Binder:628_1"]

Change-Id: I877a78c47e88d1e8a28bade8ea40be6bed58439f

Merge "Use _exit instead of exit for the system exit."

Reject classes inheriting from themselves directly or transitively.

Also enforce class definition ordering with respect to super
classes within a Dex file.

Bug: 28685551
Bug: 27682580
Change-Id: If3eba782538eb4328d4b8a542236632379e7c050

Merge "Ensure that java.lang.reflect.Field is initialized"

Ensure that java.lang.reflect.Field is initialized

Recent change in libcore (a move of clinit code
to register function for java.net.InetAddress
class) was triggering a failure in 119-noimage-patchoat
test. Failure was occurring when the test was running
without the image. This was caused by the
java.lang.reflect.Field not being initialized when
running InetAddress.<clinit>, which is a side effect
of calling JNIEnv->GetMethodId inside the libcore JNI_OnLoad.

This changes makes sure that this class is initialized
when we're running without image.

Bug: 28153851
Change-Id: I8aeb5eb312a163c452c40b851112544a8340e488

Merge "Revert "ART: Reference.getReferent intrinsic for x86 and x86_64""

Revert "ART: Reference.getReferent intrinsic for x86 and x86_64"

Fails heap poisoning configuration.

This reverts commit afdc97ebcb4e58afb7cf54d846d30314e6499d83.

Change-Id: I50e53756a2b85059b89cfb8950f8c9e2b032743c

Merge "Fix oatdump crash on arm64/arm code. Also adds 16 bit literal information."

Merge "Don't expect permission to open foreign dex use marks."

Merge "Hold a ClassLoader reference in NativeAllocationRegistry."

Fix oatdump crash on arm64/arm code.
Also adds 16 bit literal information.

Rationale:
When "run-away" instructions are disassembled, the literal
addresses may go out of range, causing oatdump to crash.
This CL guards memory access against the full memory range
allocated to assembly instructions and data (it is possible
but not really necessary to refine this a bit). Out of range
arguments are now displayed as (?) to denote the issue, which
is a lot nicer than crashing.

BUG=28670871

Change-Id: I51e9b6a6a99162546fe31059f14278e8980451c2

Don't expect permission to open foreign dex use marks.

sepolicy says we can create the mark files, but open requires
additional permissions we don't need. Don't print a warning when we
aren't given those additional permissions.

Bug: 28241500

(cherry picked from commit 12e4157666bfdc2ca38f459f6447385b80c7ddff)

Change-Id: I272befdb4fa58a0adf550fde52ddab5b5f5a9fe3

Merge "ART: Reference.getReferent intrinsic for x86 and x86_64"

Merge "Fix another case of live_in at irreducible loop entry."

Fix another case of live_in at irreducible loop entry.

GVN was implicitly extending the liveness of an instruction across
an irreducible loop.

Fix this problem by clearing the value set at loop entries that contain
an irreducible loop.

bug:28252896

(cherry picked from commit 77ce6430af2709432b22344ed656edd8ec80581b)

Change-Id: Ie0121e83b2dfe47bcd184b90a69c0194d13fce54

Use _exit instead of exit for the system exit.

We get DCHECK failures in ~Mutex() called from an atexit handler on
System.exit() in some tests. To avoid it, use _exit instead of exit.

Bug: 28106055
Change-Id: I2a6d2ad8170327553c9c9f0049829e3fd3937180

Merge "Add histogram for native allocations"

Add histogram for native allocations

Shows up in traces.txt, sample output:
Histogram of native allocation 0:4315,131072:33,1179648:3 bucket size 131072
Histogram of native free 0:995,131072:19,1179648:2 bucket size 131072

Bug: 28680116

(cherry picked from commit 0dce75dc6945c221a054eb9c479fb60efd193719)

Change-Id: Iaa07c6b8da0d6bab64a8fd4af8a02fc0b8c70e9e

Merge "Prevent ArtMethod clearing race"