am d55e9b1a: Merge "Refactor exception handling for deoptimization"

* commit 'd55e9b1a9f45c19cd7b376a8839ce37f86c66a64':
  Refactor exception handling for deoptimization

Merge "Refactor exception handling for deoptimization"

am da6e4feb: Merge "Resolve method\'s argument reference types"

* commit 'da6e4feb6a02fed7f307e1ef3b90e716a99ba24c':
  Resolve method's argument reference types

Refactor exception handling for deoptimization

This CL refactors the exception handling (on the quick side) by isolating the
search of catch handler and the preparation of deoptimization.

We rename the CatchFinder class to QuickExceptionHandler so it's less specific
to catch handler search.

Finding catch handler happens in QuickExceptionHandler::FindCatch. Since the
CatchBlockStackVisitor resolves exception types, it may cause thread suspension
and breaks the assertion current thread can't be suspended. Therefore, we place
the exception in a SirtRef (while it is detached from the current thread) and
remove the thread suspension assertion.

Deoptimization now happens in QuickExceptionHandler::DeoptimizeStack. It uses
the new DeoptimizeStackVisitor class to create shadow frames.

We also add the Thread::GetDeoptimizationException method to get the definition
of the fake exception in only one place.

Change-Id: I01b19fa72af64329b5c3b6c7f0c3339d2d724978

Merge "Resolve method's argument reference types"

Resolve method's argument reference types

Using ResolveClassAndCheckAccess allows to resolve reference types and check
we have access to these types too. This also prevents from getting unresolved
reference types, which has been observed during deoptimization.

Bug: 13464407
Change-Id: I4c02fdff191c195f34502a4d4a20357419982eb1

am 942c8e37: Merge "Support field watchpoint in interpreter"

* commit '942c8e3723ad1e12039deef5754440dd8e1c546d':
  Support field watchpoint in interpreter

Merge "Support field watchpoint in interpreter"

Support field watchpoint in interpreter

We report field read/write events to instrumentation from the interpreter. This
allows it to send JDWP field access and field modification events to debugger.
This completes CL https://android-review.googlesource.com/90390.

We also fix the JDWP FieldOnly modifier by introducing ModBasket.fieldTypeID.
We incorrectly used ModBasket.classId which is actually dedicated to ClassOnly
modifier based on thread's location's class id.

Finally, we now enable canWatchFieldModification and canWatchFieldAccess JDWP
capabilities so a debugger can request these events to be reported.

Bug: 8267708
Change-Id: I987852ad47abb27b2f7e78544a8189c7a4e2f462

am 01100510: Merge "Use __FUNCTION__ in CHECK_NON_NULL_*ARGUMENT."

* commit '011005100214eb40c2aaad3f8fe9c3dc9465bcec':
  Use __FUNCTION__ in CHECK_NON_NULL_*ARGUMENT.

Merge "Use __FUNCTION__ in CHECK_NON_NULL_*ARGUMENT."

Use __FUNCTION__ in CHECK_NON_NULL_*ARGUMENT.

Prevents naming errors and reduces code maintenance requirements.

Change-Id: I14aba05f0d5dab8009f472b3af0b698551ac2a7e

am 81d79dc3: Merge "Make unresolved exception types cause soft verification errors."

* commit '81d79dc3e2a5e92c65bad955f2bea7ee161f66aa':
  Make unresolved exception types cause soft verification errors.

am aa5d55fb: Merge "Clean up ScopedThreadStateChange + Get/SetPrimtiveArrayRegion"

* commit 'aa5d55fb3259525d8d2e930f4ec7cb193646fb57':
  Clean up ScopedThreadStateChange + Get/SetPrimtiveArrayRegion

Merge "Make unresolved exception types cause soft verification errors."

Make unresolved exception types cause soft verification errors.

Before, the verifier would allow the first exception it finds
for a catch block to be unresolved, but this causes problems if
it merges other exceptions later. The verifier should soft fail
for any unresolved exception type it finds.

Bug: 14256107
Change-Id: I22563ebfe8c9680cc676b73516d5b48bc9c4ecf3

Merge "Clean up ScopedThreadStateChange + Get/SetPrimtiveArrayRegion"

Clean up ScopedThreadStateChange + Get/SetPrimtiveArrayRegion

Simplified code in ScopedThreadStateChange and fixed an incorrect
name in Get/SetPrimitiveArrayRegion.

Change-Id: Id71affec1d64911449d792911cd52104dd179840

am 17c50db4: Merge "More code for the read barrier support."

* commit '17c50db442d2791d9c4d7d1e98060556c323ce9b':
  More code for the read barrier support.

Merge "More code for the read barrier support."

am 1b4e3d5b: Merge "Revert "ART: Extracts an utility function of the duplicated code""

* commit '1b4e3d5bc1e2d604adeda4ebbd137f20ee041138':
  Revert "ART: Extracts an utility function of the duplicated code"

Merge "Revert "ART: Extracts an utility function of the duplicated code""

Revert "ART: Extracts an utility function of the duplicated code"

This reverts commit 973cc95da6fb617bab133bd7a693c1cb7eafd393.

Change-Id: I3883c74ba06116e89d89d9cf085f20cff5d15f77

am a062d8d0: Merge "Add test that Class.newInstance does not wrap its exceptions."

* commit 'a062d8d0791967e8c4018be5aeb501b9e7a3460c':
  Add test that Class.newInstance does not wrap its exceptions.

Merge "Add test that Class.newInstance does not wrap its exceptions."

am 1768775e: Merge "ART: Extracts an utility function of the duplicated code"

* commit '1768775ec15443a8ff326e63c0b3f8deb17699e6':
  ART: Extracts an utility function of the duplicated code

Merge "ART: Extracts an utility function of the duplicated code"

ART: Extracts an utility function of the duplicated code

This patch introduces an utility function 'DataOffsetOfType' in
'mirror::Array' class that calculates the data offset at given index
in an array of given type.

Change-Id: Idb19558653c70a129245f220f0fbb553f898865b
Signed-off-by: Dmitry Petrochenko <dmitry.petrochenko@intel.com>

am 96a4f293: Merge "ART: Fixes an issue with REX prefix for instructions with no ModRM byte"

* commit '96a4f29350bf279d48bff70e21e3264cce216683':
  ART: Fixes an issue with REX prefix for instructions with no ModRM byte

am ae55e6c0: Merge "Don\'t allow heap transitions if the runtime is shutting down."

* commit 'ae55e6c0c6a6fa393742a00ac5b7748f9ce45344':
  Don't allow heap transitions if the runtime is shutting down.

Merge "ART: Fixes an issue with REX prefix for instructions with no ModRM byte"

Add test that Class.newInstance does not wrap its exceptions.

Unlike Constructor.newInstance, Class.newInstance should not wrap
exceptions it throws.

Bug: https://code.google.com/p/android/issues/detail?id=68620

Change-Id: I47d78904e72f299284d1382bc715ab120d3b1a7f

am 1c17753c: Merge "art: fix dlopen error message"

* commit '1c17753ca9cf125ffd1fa47f34650fb6f4005409':
  art: fix dlopen error message

am 125011d7: Merge "Delete throw launchpads."

* commit '125011d70aa84b3fd9052f1c90101401b0851928':
  Delete throw launchpads.

Merge "Don't allow heap transitions if the runtime is shutting down."

Merge "art: fix dlopen error message"

art: fix dlopen error message

detail is a std::string *, dereference it when logging the error
message.

Change-Id: If73a5806445df17d91ff1a7f4f3a6d31f0e9f54e

Merge "Delete throw launchpads."

Don't allow heap transitions if the runtime is shutting down.

Bug: 14254222
Change-Id: I97ac988a96b56a8dc1eec018bf3ef7a691f04745

am de981b0c: Merge "Separate maps from code in oat file."

* commit 'de981b0c9124f0b1f29e18ea6b8cb8328f561aab':
  Separate maps from code in oat file.

Merge "Separate maps from code in oat file."

am a08ec9b3: Merge "Fix FindCatchBlock to work in -Xverify:none mode."

* commit 'a08ec9b372d4f5e918b3d68499fbd1731180cd98':
  Fix FindCatchBlock to work in -Xverify:none mode.

Merge "Fix FindCatchBlock to work in -Xverify:none mode."

Fix FindCatchBlock to work in -Xverify:none mode.

FindCatchBlock now uses ResolveType to get the exception type,
since it might not be able to find it in the dex cache.

Bug: 13948502
Change-Id: Ia6f1c7dc743206ae1c8551bf6239f48ee4d3a784

am 4af159bf: Merge "Remove support for app JNI workarounds."

* commit '4af159bfcc66c6e90a2aee4a2035a996a399e7ed':
  Remove support for app JNI workarounds.

Merge "Remove support for app JNI workarounds."

am d299898a: Merge "Warn when thread state lookup fails."

* commit 'd299898a55f41ccf78b865e153ae91321229353b':
  Warn when thread state lookup fails.

Merge "Warn when thread state lookup fails."

Remove support for app JNI workarounds.

Change-Id: I4396df7e93fcace4b5b19c2c387e5c30089182a6

More code for the read barrier support.

Make it possible to disable the RB in Object::SizeOf() (and the
functions it calls transitively) which the collector will need to call
to get the size of an object when copying.

Add Object::AtomicSetReadBarrierPointer() for atomic write of a RB
pointer.

Bug: 12687968
Change-Id: Ibedd252860ac7ccd17e4e7d71b377a8892b48ff0

am 0ea4bf7e: Merge "Update load/store utilities for 64-bit backends"

* commit '0ea4bf7edb20be30f63566bce2d9db23f0b1c87f':
  Update load/store utilities for 64-bit backends

am 39595830: Merge "Return bytes freed from RosAlloc."

* commit '39595830dbfefa47bbde20490b4dda3385de33a4':
  Return bytes freed from RosAlloc.

Merge "Update load/store utilities for 64-bit backends"

Update load/store utilities for 64-bit backends

This CL replaces the typical use of LoadWord/StoreWord
utilities (which, in practice, were 32-bit load/store) in
favor of a new set that make the size explicit.  We now have:

   LoadWordDisp/StoreWordDisp:
    32 or 64 depending on target.  Load or store the natural
    word size.  Expect this to be used infrequently - generally
    when we know we're dealing with a native pointer or flushed
    register not holding a Dalvik value (Dalvik values will flush
    to home location sizes based on Dalvik, rather than the target).

   Load32Disp/Store32Disp:
     Load or store 32 bits, regardless of target.

   Load64Disp/Store64Disp:
     Load or store 64 bits, regardless of target.

   LoadRefDisp:
     Load a 32-bit compressed reference, and expand it to the
     natural word size in the target register.

   StoreRefDisp:
     Compress a reference held in a register of the natural word
     size and store it as a 32-bit compressed reference.

Change-Id: I50fcbc8684476abd9527777ee7c152c61ba41c6f

Merge "Return bytes freed from RosAlloc."

Return bytes freed from RosAlloc.

There was a problem with how RosAlloc space sweeping worked caused by
using the object size in the FreeList call, this won't work well with
class unloading since the object's class may be freed before the
object.

Bug: 13989231
Change-Id: I3df439c312310720fd34249334dec85030166fe9

Warn when thread state lookup fails.

Change-Id: I06781caddcade26148921bc225f28db8b3a63a35

Separate maps from code in oat file.

Write all GC maps first, then all mapping tables and then
all vmap tables and only then align the offset to page size
and write all method code chunks with headers.

Bug: 11767815
Change-Id: Ic83555c8303c5be119afc43e95e58c0a32ff2a4f

am 82b1a818: Merge "Avoid allocating OatFile::OatClass on the heap."

* commit '82b1a81890970a8b07f9132aeae537a6c43df6b0':
  Avoid allocating OatFile::OatClass on the heap.

am 618a8700: Merge "Fix crash when debugging exception"

* commit '618a87009202dc959c935ed8f237ae32bdec57d0':
  Fix crash when debugging exception

am 00256f3e: Merge "Revert "64bit changes to the stack walker for the Quick ABI.""

* commit '00256f3e03adaf11b8fbb8bb8b7a11a9c316127d':
  Revert "64bit changes to the stack walker for the Quick ABI."

am e08920e3: Merge "64bit changes to the stack walker for the Quick ABI."

* commit 'e08920e3b15d463cd70c51276677291d1af7bce8':
  64bit changes to the stack walker for the Quick ABI.

am ea810632: Merge "Speed up single-stepping"

* commit 'ea8106321c7c05f8b135d7b4cdb34f5d1e6ed0e3':
  Speed up single-stepping

am 6123d94c: Merge "GenArithOpInt should ensure that reg is in core"

* commit '6123d94cddf34f3fdb8ece55476e414e34662394':
  GenArithOpInt should ensure that reg is in core

Merge "Avoid allocating OatFile::OatClass on the heap."

Merge "Fix crash when debugging exception"

Merge "Revert "64bit changes to the stack walker for the Quick ABI.""

Revert "64bit changes to the stack walker for the Quick ABI."

This reverts commit 5cb328362a633302ca0fcdbaa0da7d94069df051.

Change-Id: Icc0ec1a9f15c2e4e9103e732d7ba75a4feb853b1

Merge "64bit changes to the stack walker for the Quick ABI."

Merge "Speed up single-stepping"

Merge "GenArithOpInt should ensure that reg is in core"

Delete throw launchpads.

Bug: 13170824

Change-Id: I9d5834f5a66f5eb00f2ac80774e8c27dea99949e

am 4e0d5ee5: Merge "Revert "Revert "Use LIRSlowPath for throwing ArrayOutOfBoundsException."""

* commit '4e0d5ee57ab5f76a761cf0d7ebb50b782e3eacdc':
  Revert "Revert "Use LIRSlowPath for throwing ArrayOutOfBoundsException.""

Merge "Revert "Revert "Use LIRSlowPath for throwing ArrayOutOfBoundsException."""

am 63da330f: Merge "Add an option to specify a list of methods for LIR dumping."

* commit '63da330fa097aedc97e40ded054106a7168ea116':
  Add an option to specify a list of methods for LIR dumping.

Merge "Add an option to specify a list of methods for LIR dumping."

am b0618ae0: Merge "ART: GTest target tests missing "PASSED""

* commit 'b0618ae005f2ae5bbdd5bcf4279ede2deb6d62ef':
  ART: GTest target tests missing "PASSED"

Merge "ART: GTest target tests missing "PASSED""

ART: GTest target tests missing "PASSED"

This seems to have gotten lost a while ago.

Change-Id: Ia0114d4fc0dfcf6e53339307cd556e168312fc53

am 72f184d0: Merge "Fix dependencies in the test suite for multi-target tests"

* commit '72f184d0751dcd6f4d03451b047943ca1da47339':
  Fix dependencies in the test suite for multi-target tests

Merge "Fix dependencies in the test suite for multi-target tests"

Fix dependencies in the test suite for multi-target tests

Change-Id: I94dd767490a7cd8cceddc158d90f56f5bd3fef26

Add an option to specify a list of methods for LIR dumping.

An example would be:
--runtime-arg -verbose-methods:Array.checkRange32,Array.checkRange64
or
"-verbose-methods:void Array.checkRange32,void Array.checkRange64"

Change-Id: I61c1eb3b2eb4b24126a9264261c27889d53cc6bd

am 86e1b5e7: Merge "Get instruction-set/features from build/core/config.mk"

* commit '86e1b5e7e2bca99dd2092eab8ced977d97830873':
  Get instruction-set/features from build/core/config.mk

Merge "Get instruction-set/features from build/core/config.mk"

am 9b3335e3: Merge "Modify unwind to comply with stack parser tools."

* commit '9b3335e3503724dbcb492e159789adf98ae3958b':
  Modify unwind to comply with stack parser tools.

Get instruction-set/features from build/core/config.mk

Depends upon https://android-review.googlesource.com/#/c/91524

Change-Id: Iad1a1b0499aac7acf7c4e42b62bb3fe4474e2f47

Merge "Modify unwind to comply with stack parser tools."

Modify unwind to comply with stack parser tools.

Bug: 14081592

Change-Id: I6906b2575c74d64f1c3ba7602779b3a789de1c69

am ed0bc0bb: Merge "Fixing missing newlines in Usage calls"

* commit 'ed0bc0bbf26f6751f31213d24ccdac49766e93ff':
  Fixing missing newlines in Usage calls

Merge "Fixing missing newlines in Usage calls"

Revert "Revert "Use LIRSlowPath for throwing ArrayOutOfBoundsException.""

This adds back using LIRSlowPath for ArrayIndexOutOfBoundsException.
And fix the host test crash.

Change-Id: Idbb602f4bb2c5ce59233feb480a0ff1b216e4887

am b21f7c9f: Merge "Fix ForegroundHeapGrowthMultiplier and ParseDouble."

* commit 'b21f7c9f9729be60438c6c3778662d81a8c2f2d6':
  Fix ForegroundHeapGrowthMultiplier and ParseDouble.

Fixing missing newlines in Usage calls

Change-Id: Ie5f1098dc7b3c485732bff929d36dbcc4b69511f

am fa25af3e: Merge "Replace ObjectSet with LargeObjectBitmap."

* commit 'fa25af3e3410537e86b485682d519dafbad01753':
  Replace ObjectSet with LargeObjectBitmap.

Merge "Fix ForegroundHeapGrowthMultiplier and ParseDouble."

Fix ForegroundHeapGrowthMultiplier and ParseDouble.

ForegroundHeapGrowthMultiplier had a bad range of allowed values.
The new range is 0.1 - 10.0 instead of 0.1 to 0.9. ParseDouble usage
had incorrect printing var args and caused segfaults.

Change-Id: I4573005ee9f888cc8140200543176d6a2e17fccc

Merge "Replace ObjectSet with LargeObjectBitmap."

am 44b0053f: Merge "Fix elf_writer_test for 64b target"

* commit '44b0053fdb7ad8a30138d29f714172a7dc69efb8':
  Fix elf_writer_test for 64b target

Merge "Fix elf_writer_test for 64b target"