am 16dd4dfd: Merge "ART: Relax GetInstructionSetFromString" into lmp-dev

* commit '16dd4dfd01c0ac432a32d081a15205222ffb1f1c':
  ART: Relax GetInstructionSetFromString

am 167350d9: Add null check to CheckVirtualMethod

* commit '167350d9c781c5e3893714bb26ab5cb1c4abf6b4':
  Add null check to CheckVirtualMethod

Add null check to CheckVirtualMethod

There was a runtime SIGSEGV that should have been a check jni
failure.

Also added regression test.

Bug: 16320699
Change-Id: If6c8e73959cefb24e4703f1562cdddb343d86630

am a846c1ac: Merge "X86: Fix alias info in GenInlinedIndexOf()." into lmp-dev

* commit 'a846c1ac65b197844380b8d28377a850dfa1fc94':
  X86: Fix alias info in GenInlinedIndexOf().

am c6a62592: Merge "Quick compiler: fix Arm64 inline locking code" into lmp-dev

* commit 'c6a62592836612b09ff1a8021e49b1328d85493b':
  Quick compiler: fix Arm64 inline locking code

Merge "Quick compiler: fix Arm64 inline locking code" into lmp-dev

Quick compiler: fix Arm64 inline locking code

The Arm64 monitor enter code was incorrectly comparing the
thread ID to zero rather than the lock value to determine if the
inline fast path was doable.  As a result, we ended up always
using the out-of-line case.

This fix delivers a 6% improvement to DeltaBlue.

b/16707253

Change-Id: I499530da84beecf1f3f5c5ebb666e3eb12599ac0

am d6ee5033: Merge "Make apps able to run with a failing patchoat" into lmp-dev

* commit 'd6ee5033433619a893a8f20319efcf8c8b1d1016':
  Make apps able to run with a failing patchoat

am 20e4afeb: Merge "Workaround problem reading main stack on intel devices." into lmp-dev

* commit '20e4afeb1ad9d142fe9e46efe812f0d562c28f1f':
  Workaround problem reading main stack on intel devices.

am 38bd1e1b: ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

* commit '38bd1e1b8aa39bdc6e67c5dc205cc8d294d985c5':
  ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

Merge "Workaround problem reading main stack on intel devices." into lmp-dev

ART: Fix ARM invoke stub, run Generic JNI in jni_compiler_test

On devices which support it, run all jni compiler tests also with
the Generic JNI infrastructure.

Fix art_quick_invoke_stub for ARM32 to enforce 16B stack alignment.
As the native convention does not require as much, the previous
code would keep it unaligned.

Bug: 17107929

(cherry picked from commit 6e498695b60f1532d2264ec6badb1cd6e10ecaa9)

Change-Id: I1481500ff7c627ab4e5bfbfd7245099a639aa049

Workaround problem reading main stack on intel devices.

This works around a problem where ART can't read the lowest page
of the main stack.  The workaround is to add 4K to the stack
start address on the main stack on Intel.

Please see https://b2.corp.google.com/issues/17111575 for
underlying cause.

Bug: 17031544
Change-Id: Ifc3216e10160bb1eec9d989fd57b10345bc89537

am e22c03ce: Merge "ART: Set default visibility to protected" into lmp-dev

* commit 'e22c03ceaa847e3b6c8fcb7ee9d838f61945cda7':
  ART: Set default visibility to protected

am 5afe466c: Check for constructor first when installing tracing stubs.

* commit '5afe466c027c6ac821bf7c9a88cd7de5260a639b':
  Check for constructor first when installing tracing stubs.

Check for constructor first when installing tracing stubs.

Addresses comments in
https://android-review.googlesource.com/#/c/104383/ made after the commit.

Bug: 16386215
Change-Id: I0e59b8507f601fd6508fbda8d24dd3c98ead1537

Merge "ART: Set default visibility to protected" into lmp-dev

am 214f4f15: Merge "Fix JDWP event races" into lmp-dev

* commit '214f4f15fe0137c285a05b3f70e52a211aecfdb5':
  Fix JDWP event races

Merge "ART: Relax GetInstructionSetFromString" into lmp-dev

ART: Relax GetInstructionSetFromString

Do not abort on an unknown instruction set string. Instead return
kNone and let the caller handle this.

Also simplify the patchoat tool to use this.

Bug: 17136416
Change-Id: I24131914bcf91c04ae93179bf809a2907f1f2b7a

Merge "Make apps able to run with a failing patchoat" into lmp-dev

Make apps able to run with a failing patchoat

Bug: 17000769

(cherry picked from commit 9dcc4572949f6a8231a1b4ed859676ba6f411726)

Change-Id: I0a1a4dc7f5d4bb268530840302ecfb1555231e05

am 5a001b3e: Merge "AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled." into lmp-dev

* commit '5a001b3e4a4b4d066fd987071eb38b2b97493807':
  AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled.

am 6724d864: Fix heap corruption and verification.

* commit '6724d8649ab73e4fb86c8014bda51b13bddf2f3f':
  Fix heap corruption and verification.

Fix heap corruption and verification.

There was a faulty assumption that space End() was always aligned to
card boundaries. This was true for all spaces other than the image
and resulted in heap corruption when the last object of the image
space contained a reference to an object in another space.

Also fixed an error where we called the pre GC verification post GC.

Bug: 17080623

Change-Id: I041ee564518f53b79c6e8dc2ad782a3152577a4e

am dfa7a6c8: Disable load/store elimination. DO NOT MERGE

* commit 'dfa7a6c8711623bf20aad58f57ae980b497ef4d7':
  Disable load/store elimination. DO NOT MERGE

Disable load/store elimination. DO NOT MERGE

The optimization has several issues and it has been
effectively disabled by
    https://android-review.googlesource.com/103511
Disable it completely to avoid wasting compilation time.

Bug: 16862804
Change-Id: I42f69ae64170dc44f5fd6455b6a51c9bd3f93672

am 5e8a63ae: Merge "ART: Do not recursively abort when visiting locks in a bad state" into lmp-dev

* commit '5e8a63ae39cb5db996229c6c4a1a21a4b8dec5a2':
  ART: Do not recursively abort when visiting locks in a bad state

am d84f916d: ART: Avoid expensive class lookup in compiler driver

* commit 'd84f916d35bb0dc74d7daa075b883e5ee1ce1109':
  ART: Avoid expensive class lookup in compiler driver

ART: Avoid expensive class lookup in compiler driver

When we are not compiling multidex, avoid an expensive by-descriptor
lookup for classes when deciding whether to skip compilation.

Bug: 17072698
Change-Id: I19bcb2a2928207fcae0badd466cdc52945bd37c4

Merge "X86: Fix alias info in GenInlinedIndexOf()." into lmp-dev

X86: Fix alias info in GenInlinedIndexOf().

For 32-bit X86, GenInlinedIndexOf() pushes and pops EDI. In
one branch it then calls Load32Disp() with adjusted stack
offset. That calculates wrong alias_info for the generated
insn. If left unfixed, this could confuse load hoisting.

Bug: 17128502

(cherry picked from commit 74de63bb1cc275b411cae28a96f9b3a78b939bc2)

Change-Id: I5dc82b7aae9e9655e75843a952b8ebb04269f46b

Merge "AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled." into lmp-dev

AttachCurrentThread should only warn on missing thread name if CheckJNI is enabled.

Bug: 17108693

(cherry picked from commit 3ea69c0abce1b81b3c45033867d49b00e6d6b709)

Change-Id: Ide8158cc3e12e604f4fa344cd881fe6f68476ab7

Merge "Fix JDWP event races" into lmp-dev

Fix JDWP event races

Fix a race where the JDWP connection could be closed before the VM_DEATH event
is sent during runtime shutdown.

Fix potential race where we could wait forever for the JDWP thread to establish
connection.

Bug: 16720689

Change-Id: I227e0a15a2fd17d7bfe2a66a35d719d558fcd32d

Merge "ART: Do not recursively abort when visiting locks in a bad state" into lmp-dev

ART: Do not recursively abort when visiting locks in a bad state

This avoids a nested abort in VisitLocks.

Bug: 17080621, 16382675
Change-Id: Id604976ac9dcac0e319fb25cab4d2cbc98d7ee24

am ebee8de7: Fix bugs in the x86 and arm versions of generic JNI.

* commit 'ebee8de725d5fa83483642786b19ea453d865762':
  Fix bugs in the x86 and arm versions of generic JNI.

am 9c290012: Execute an application even when dex2oat crashes.

* commit '9c290012b7f505ae1943ab87236f775b97a46e2d':
  Execute an application even when dex2oat crashes.

Fix bugs in the x86 and arm versions of generic JNI.

Also fix the run script of 116-nodex2oat to use the non-prebuild
script for target.

Bug: 17000769

(cherry-picked from commit 54accbca0b549b1b1ad3ef09655dad438bc1e104)

Change-Id: I439fcd710fb8bb408f3288ff8fb34fef23890adb

Execute an application even when dex2oat crashes.

Bug: 17000769

(cherry picked from commit 4fcdc94d22a4608e355aa8df36240181149d10e8)

Change-Id: Iccb1fec94fe64ce4c3097510952f275482b86aa9

am 378fe3d3: Merge "Make Monitor::Wait more robust to spurious Inflate failures." into lmp-dev

* commit '378fe3d39d3a55f61d82ea482b8ce3082aee0e21':
  Make Monitor::Wait more robust to spurious Inflate failures.

am bb6eccca: Merge "Use handle in one case that spans a gc-point." into lmp-dev

* commit 'bb6eccca7dcd673e24eed2fe95915780aa38a5d6':
  Use handle in one case that spans a gc-point.

am 95b4c65d: Prevent stubs from being installed in java.lang.reflect.Proxy.<init>.

* commit '95b4c65da93500cdbdcaa3e01010771ef3f466f1':
  Prevent stubs from being installed in java.lang.reflect.Proxy.<init>.

am 4bf8d11d: Fix proxy tracing and enable tests that now work with tracing.

* commit '4bf8d11df5dccc1b276cd9c40a98e8a14d79a9c8':
  Fix proxy tracing and enable tests that now work with tracing.

Prevent stubs from being installed in java.lang.reflect.Proxy.<init>.

This CL is a better fix for proxy tracing and undoes the changes in
https://android-review.googlesource.com/#/c/103025/

Bug: 16386215

(cherry picked from commit db8a664e0b68c7c4d36270cd21dce8de1912d7f9)

Change-Id: Ic9e0ea2af7cb2da5d90c56aa009de92dba14cc47

Fix proxy tracing and enable tests that now work with tracing.

Also updates proxy_test to generate an image for GetQuickOatCodeFor.

Bug: 16386215

(cherry picked from commit f0a3f09c3d54646166a55c05a6b39c7dd504129c)

Change-Id: I138edbad9e1646db8590f2b1b73f2788d9710e68

am 93de4273: [ART] Move to system/core native bridge library

* commit '93de4273d72a2558a7b3423547b5074cd76c5796':
  [ART] Move to system/core native bridge library

[ART] Move to system/core native bridge library

Some cleanups on the way:
- move NativeBridgeRuntimeCallbacks from a global to a field
- rename native_bridge_library_string to the more suggestive
navtive_bridge_library_path
- clean up imports

Bug: 16884833

(cherry picked from commit I73aab8e212860ba5aee9444d801806d3da326a41)

Change-Id: I67f037ea81d2a4ede4294afd8b84b7640e534a13

am 4f1d05f0: Merge "More efficient stack walk in exception throwing." into lmp-dev

* commit '4f1d05f0531b980b16dbff18161f02160919ec31':
  More efficient stack walk in exception throwing.

Merge "Make Monitor::Wait more robust to spurious Inflate failures." into lmp-dev

Make Monitor::Wait more robust to spurious Inflate failures.

Bug: 17062710
Change-Id: Ife5f6b335caacc70cab543cd568676d277d3beb6

Merge "More efficient stack walk in exception throwing." into lmp-dev

Merge "Use handle in one case that spans a gc-point." into lmp-dev

Use handle in one case that spans a gc-point.

Bug: 16689428

(cherry picked from commit 38eecb0f4288a374c9b0b4b4df8793eb5fc6697c)

Change-Id: Id4a1bde9012e4dd333493e46997c200537c9cf55

am b84d7a22: Revert "Revert "ART: Fix mac build""

* commit 'b84d7a226399dcc60bc0a421715ffc90ce9213f7':
  Revert "Revert "ART: Fix mac build""

Revert "Revert "ART: Fix mac build""

This reverts commit 350b03bca65f87895275d1ed4bb895d5afeeeff1.

Also fixes a fault_handler symbol.

Change-Id: I1315527400bffc8ded9f2a7e7fdb1c8a6c104b96

am 03c9785a: Revert "Revert "Reduce stack usage for overflow checks""

* commit '03c9785a8a6d712775cf406c4371d0227c44148f':
  Revert "Revert "Reduce stack usage for overflow checks""

Revert "Revert "Reduce stack usage for overflow checks""

Fixes stack protection issue.
Fixes mac build issue.

This reverts commit 83b1940e6482b9d8feba5c492507735686650ea5.

Change-Id: I7ba17252882b23a740bcda2ea94aacf398255406

More efficient stack walk in exception throwing.

In the exception handling code, we currently walk down the stack
twice, once to get the stack height which we use to compute frame IDs
(the bottom frame is zero), and once more to find the catch block to
jump to.

For a deep stack, this could result in very slow exception
handling. That is, if have a lot of finally or catch blocks that we
end up jumping to in a deep stack, we need to do a lot of
catch/rethrow chains. Since we'd need to walk down to the bottom each
time to compute frames IDs in each catch/rethrow, we'd need to walk
down O(N^2) frames at the worst case.

Instead of frames IDs ((the bottom frame is zero), we will use the
frame depth (the top frame is zero) and no longer need to walk down
the stack just to get the stack height. We walk down O(N) frames.

This was what was happening with
code.google.gson.functional.CircularReferenceTest. With this change,
the test run time went from ~120s down to ~3s on N5 and it no longer
crashes due to the thread suspension timeout.

(cherry pick commit 649278cec7119cdd1bea3d0b710dbb2aa7c650b6)

Bug: 16800209
Change-Id: Ie815df1e3e8fb9d82e40685d4cc2b8838fd8aa07

am b162bf5a: Use homogeneous space compaction if proper flag is set.

* commit 'b162bf5af5c2e508c6947471ceffaa98991794f4':
  Use homogeneous space compaction if proper flag is set.

Use homogeneous space compaction if proper flag is set.

If the flag ART_USE_HSPACE_COMPACT is set then we use
hspace compaction.

Bug: 16401001

Change-Id: I74e1db764f9ff16c007fa3bd16cb2c9b468ec83c

am 1d778570: Merge "ART: Fix run-test in case of relative TMPDIR" into lmp-dev

* commit '1d7785704cafca20629c4bfb87c60d99dd69f72a':
  ART: Fix run-test in case of relative TMPDIR

am 0b6952ca: Merge "Revert "ART: Fix mac build"" into lmp-dev

* commit '0b6952cac86de294fb01898694b6a55f0a162a43':
  Revert "ART: Fix mac build"

am e470a027: Merge "Fix intrinsic Math.abs(double) for ARM." into lmp-dev

* commit 'e470a02792ce584fe0ff1b9a7dd6aebe62ca9be5':
  Fix intrinsic Math.abs(double) for ARM.

am b0bf089c: Merge "Revert "Reduce stack usage for overflow checks"" into lmp-dev

* commit 'b0bf089c2d80f861e4d8798e97828452b2e0b99b':
  Revert "Reduce stack usage for overflow checks"

am a73a8b55: Move explicit GC after we are done initalizing all classes.

* commit 'a73a8b55415b544355438522524a640378698d85':
  Move explicit GC after we are done initalizing all classes.

Move explicit GC after we are done initalizing all classes.

Previously we had the GC happen for each dex file, this added a
few seconds of time on boot.oat creation.

Bug: 16853450

(cherry picked from commit 093ef212b7db4c17ed62df5cac0c1ac3a24e691d)

Change-Id: Ia469838eff1550c175c3465a478d8081023601bd

am 0c2a6d76: Merge "ART: Fix another potential N-th visitor issue" into lmp-dev

* commit '0c2a6d76ccedb77e79adc68c723cd4ed8278fee9':
  ART: Fix another potential N-th visitor issue

Merge "Revert "Reduce stack usage for overflow checks"" into lmp-dev

Revert "Reduce stack usage for overflow checks"

This reverts commit 63c051a540e6dfc806f656b88ac3a63e99395429.

Change-Id: I282a048994fcd130fe73842b16c21680053c592f

Merge "ART: Fix another potential N-th visitor issue" into lmp-dev

am 564f5830: ART: Fix mac build

* commit '564f58305961986591ccb2fff04b9ccdd430a6db':
  ART: Fix mac build

ART: Fix mac build

is_main_thread is unused in the case of a mac.

Bug: 15435566

(cherry picked from commit 7b1bf42dbc765524d54d45a70cb7eb1cccf0c617)

Change-Id: I9848d0156a4903ce73d20b5ce282b61ceb3039ff

am c483a73c: Merge "ART: Fix class-linker handling" into lmp-dev

* commit 'c483a73cc2f9d63c1b6bccd1007117cf0d6dcd56':
  ART: Fix class-linker handling

am f8c4ed08: ART: Fix wrong command line parameter

* commit 'f8c4ed08832ec5302c9ea8441ec8cb39b87075ab':
  ART: Fix wrong command line parameter

am 4c37afb7: Get the native bridge library from the framework.

* commit '4c37afb7b207424ff788ab8d8778ea4e8d4ac324':
  Get the native bridge library from the framework.

am 20ae1d77: Clean up native bridge

* commit '20ae1d77463d1c3f206eac8d541124b4555464de':
  Clean up native bridge

ART: Fix wrong command line parameter

Bug: 16843953

(cherry picked from commit c4a7acf505944e7d937f0670b322e12302dd38dc)

Change-Id: Ib59e10a5d97a20ea1a013587645e0eeb204eda8d

Get the native bridge library from the framework.

Framework now passes the name of the native bridge to the runtime so
there's no need to do the reading in libart.

Bug: 16843953

(cherry picked from commit 421b6466977d4b1ccd453f23b7b492f219099702)

Change-Id: I9a0a8b8c64949e05c6f90bc846d68f0797640f6d

Clean up native bridge

(partially cherry picked from commit
68d8b42ddec39ec0174162d90d4abaa004d1983e)

Change-Id: I85813e97d73b1b331646dd0e880108dfbfe80f69

ART: Fix another potential N-th visitor issue

Bug: 16867274
Change-Id: I5466af563032041ef4a547894f40f303871302c6

am 7e9cdbbc: Merge "ART: Tolerate shallow call stack in VMStack_getCallingClassLoader" into lmp-dev

* commit '7e9cdbbc14c015028d4b239a09c5af33ffc2087c':
  ART: Tolerate shallow call stack in VMStack_getCallingClassLoader

am c85a899b: Revert "Revert "Remove GCC atomic workarounds.""

* commit 'c85a899b23327c3d5693c8dabf5b482c3fb5ded8':
  Revert "Revert "Remove GCC atomic workarounds.""

am f0a37663: Initialize art::Atomic\'s value to 0.

* commit 'f0a37663309e468f11858bffa3221e7223cf577d':
  Initialize art::Atomic's value to 0.

Revert "Revert "Remove GCC atomic workarounds.""

This reverts commit 626a2468e4e4f39db7b0f35a4fee87293e360e92.

(cherry picked from commit aab0f86e3b079598d41c3a00bfa765a7589c5110)

Change-Id: I2333b54dcfd5ee8227a6479d2e3814b25377bdfd

Initialize art::Atomic's value to 0.

art::Mutex and art::ConditionVariable were DCHECKing that the value was
zero on entering the constructor without ever initializing the value.
Since 0 is a sensible default, make it the default for art::Atomic
rather than initializing for each use.

Bug: 16301104

(cherry picked from commit 6a3f8d93ddf09b5f6667820089e488958cba8361)

Change-Id: Iab69bccac9f904111a72eff8162c7f0ca979a08a

ART: Set default visibility to protected

Set default visibility of symbols to protected. This allows the
linker to optimize internal calls and helps avoid plt calls.

Make almost all assembly stubs hidden, as -fvisibility does not
seem to apply to them. Change the assembly tests accordingly. Also
allows to clean up previous hacks to avoid plt calls.

Bug: 16974467
Change-Id: I9030dcf6116251f434f94a2b08e56e12085af652

am 63c051a5: Reduce stack usage for overflow checks

* commit '63c051a540e6dfc806f656b88ac3a63e99395429':
  Reduce stack usage for overflow checks

Reduce stack usage for overflow checks

This reduces the stack space reserved for overflow checks to 12K, split
into an 8K gap and a 4K protected region.  GC needs over 8K when running
in a stack overflow situation.

Also prevents signal runaway by detecting a signal inside code that
resulted from a signal handler invokation.  And adds a max signal count to
the SignalTest to prevent it running forever.

Also reduces the number of iterations for the InterfaceTest as this was
taking (almost) forever with the --trace option on run-test.

Bug: 15435566

Change-Id: Id4fd46f22d52d42a9eb431ca07948673e8fda694

Conflicts:
compiler/optimizing/code_generator_x86_64.cc
runtime/arch/x86/fault_handler_x86.cc
runtime/arch/x86_64/quick_entrypoints_x86_64.S

am c739de01: Merge "Avoid use of std::string where we have const char*." into lmp-dev

* commit 'c739de01b905729b8587003a52a55b62358e979d':
  Avoid use of std::string where we have const char*.

am 83715081: Add allocation rate tracking to systrace.

* commit '837150818af0eee993f7d93c5a50c3c7b46f1dab':
  Add allocation rate tracking to systrace.

Add allocation rate tracking to systrace.

Bug: 16956980

Change-Id: I7074f4ae0f75d1bb58e9860ded4e17ce6978d74e

am b0f05b96: Add implicit checks for x86_64 architecture.

* commit 'b0f05b9654eb005bc8c8e15f615a7f5a312f640c':
  Add implicit checks for x86_64 architecture.

Merge "ART: Fix class-linker handling" into lmp-dev

ART: Fix class-linker handling

ResolveMethod did not account correctly for the mutual exclusivity
of direct and static methods. In such a case we threw a NoSuchMethodError,
while the correct behavior is to throw an IncompatibleClassChangeError.

Bug: 16956477
Change-Id: Id014affe0b8a43dbd75570b123b921d5853ab135

Merge "ART: Tolerate shallow call stack in VMStack_getCallingClassLoader" into lmp-dev

ART: Tolerate shallow call stack in VMStack_getCallingClassLoader

When the call stack does not have the three methods we expect,
the visitor will return a nullptr value.

Add a test to JniTest (and refactor the test a little for code reuse).

Bug: 16867274
Change-Id: I5fb8f91f372a41e0bc11ef9f70640834591afa53

Add implicit checks for x86_64 architecture.

This combines the x86 and x86_64 fault handlers into one.  It also
merges in the change to the entrypoints for X86_64.

Replaces generic instruction length calculator with one that only
works with the specific instructions we use.

Bug: 16256184

Change-Id: I1e8ab5ad43f46060de9597615b423c89a836035c
Signed-off-by: Chao-ying Fu <chao-ying.fu@intel.com>