Merge "ART: Simplify patchoat" into nyc-dev
am: df3a54bf63

* commit 'df3a54bf63597095f2fac3d13b22a112eb6dcd6b':
  ART: Simplify patchoat

Merge "ART: Simplify patchoat" into nyc-dev

ART: Simplify patchoat

Removed unused configurations. Cut out cruft that isn't used anymore
since multi-image. Make it smaller to not require new selinux
permissions for OTAs.

Split up the main function, to make the flow clearer for image vs
app/oat.

Bug: 25612095
Bug: 26317072
Change-Id: Ieb55bd1a13cbb44fd0e2979a8a61a509df9c1e7e

Merge "Generalized "dom-based" dynamic BCE to symbolic base + offset."
am: 0665090c88

* commit '0665090c880ec22655ff8b82084746c4407ea792':
  Generalized "dom-based" dynamic BCE to symbolic base + offset.

Merge "Generalized "dom-based" dynamic BCE to symbolic base + offset."

Generalized "dom-based" dynamic BCE to symbolic base + offset.

Rationale:
So far, if all others failed, BCE would use a dominator-based
dynamic deoptimization to eliminate bounds checks in e.g. a[0],
a[1], a[2], etc. This CL generalizes this to any symbolic base
with offset in e.g. a[base], a[base+1], etc. The runtime tests
(two for symbolic, one for constant) carefully account for
arithmetic wrap-around.

bug=26680114

Change-Id: I7432a200fd69791914ed776c77fa62567b5863c0

Fix app image memory leak
am: 6cc91e4d8c

* commit '6cc91e4d8c5956d3c4323b416f2a344bf848f741':
  Fix app image memory leak

Fix app image memory leak

There was a memory leak if we failed to add an image space, this
was caused by releasing ownership without adding it to the heap.

Bug: 22858531

(cherry picked from commit bd064ea2269b23360e32e8139c22d5993ddc385b)

Change-Id: Ib45c0140b82cee5da04ed4a9f101f7c554cb25da

Merge "Fix app image memory leak"
am: b9adbf63f8

* commit 'b9adbf63f880f246d83b3af4ca03aca07711f857':
  Fix app image memory leak

Merge "Fix app image memory leak"

Fix app image memory leak

There was a memory leak if we failed to add an image space, this
was caused by releasing ownership without adding it to the heap.

Bug: 22858531

Change-Id: Ia3c97ec822ca5362f52008e813fa434107198bc9

Merge "Fix run-test"
am: 51333f93a8

* commit '51333f93a827a36ed179d44f076d36dd629a7fe8':
  Fix run-test

Merge "Fix run-test"

Fix run-test

Change-Id: I69b7d9af864e7a34c039630b3c3c8929ce04b658

Add MODULE_LICENSE_GPL.. for art/runtime/openjdkjvm.
am: 9a3a10ac81

* commit '9a3a10ac819f456f7bcc626046342a845902fe96':
  Add MODULE_LICENSE_GPL.. for art/runtime/openjdkjvm.

Merge "Be less extreme on the iteration in 570-checker-osr."
am: 1b07552999

* commit '1b0755299928221a1dcc5db383bf8fc378d29b60':
  Be less extreme on the iteration in 570-checker-osr.

Merge "Be less extreme on the iteration in 570-checker-osr."

Be less extreme on the iteration in 570-checker-osr.

Looks like otherwise it timeouts on buildbots.

Change-Id: I26f36e9dcd3368bdf326756fdd428e641db06f3b

Merge "ART: Remove HTemporary"
am: 0d15f267eb

* commit '0d15f267eb9d629e26df0c603c39f9665951e970':
  ART: Remove HTemporary

Merge "ART: Remove HTemporary"

Add MODULE_LICENSE_GPL.. for art/runtime/openjdkjvm.

This needs a companion makefile that declares a no-op module
so that the build system can leach of its LOCAL_PATH to collect
all subdirectory sources.

bug: 27160386
Change-Id: Iac5321304ae68ac203eda8e4f7d5d81339dab851

Merge "ART: Treat throw with non-reference type as hard failure in verifier."
am: 6c5907ad7a

* commit '6c5907ad7ae1d242b90d692c478ba68b5cdcb44a':
  ART: Treat throw with non-reference type as hard failure in verifier.

Merge "ART: Treat throw with non-reference type as hard failure in verifier."

ART: Treat throw with non-reference type as hard failure in verifier.

The code
   const v0, 0xbad
   throw v0
crashes dex2oatd/Quick by DCHECK in art::Mir2Lir::LoadValueDirect.
dex2oat works fine producing VerifyError later in runtime.
Optimizing also pass as it rejects methods with soft failures.

Fix this by rejecting such methods in Verifier.

Bug: 27148248
Change-Id: Ib783f60a210362654d40e84172e7bd579913a4d4
Signed-off-by: Pavel Vyssotski <pavel.n.vyssotski@intel.com>

ART: Remove HTemporary

Change-Id: I21b984224370a9ce7a4a13a9652503cfb03c5f03

Merge "Re-enable OSR."
am: 5960a3f430

* commit '5960a3f430d1e7f47ea3db9a8364dc82e3f8add9':
  Re-enable OSR.

Merge "Re-enable OSR."

Merge "Fix compiler driver gtest."
am: b20d0f6327

* commit 'b20d0f6327acbc225eba517d220832ab9a1cd599':
  Fix compiler driver gtest.

Merge "Fix compiler driver gtest."

Fix compiler driver gtest.

(cherry picked from commit 35c4e0b2ad573e820d6e9d461a571af300611d36)

Change-Id: Ia59f4463a6158f7a949debd7a93f35fa633cd36a

Merge "Store relative paths in the profile file"
am: 1660d214de  -s ours

* commit '1660d214de55678c97defef10395f6f9d184e1fa':
  Store relative paths in the profile file

Merge "Store relative paths in the profile file"

Store relative paths in the profile file

Bug: 26409579
Bug: 26080105

(cherry picked from commit f06d6bc0d083d383371aeb1f0d97249f3e20fc25)

Change-Id: I7eeea06e7df49a7f8af4c4cb71da58c21f1c185b

Re-enable OSR.

Fixes two bugs:
- Dealing with proxy methods, which the compiler and code cache
  does not handle.
- Dealing with phi types, that may have been speculatively optimized
  but do not hold once jumping to the compiled code.

Change-Id: I7dcd9976ef7b12128fff95d2b7ed3e69cc42e90a

Merge "Fix x86-64 Baker\'s read barrier fast path for CheckCast."
am: d3caabd4f8

* commit 'd3caabd4f85f86dd744da432993e12935d843a83':
  Fix x86-64 Baker's read barrier fast path for CheckCast.

Merge "Fix x86-64 Baker's read barrier fast path for CheckCast."

Merge "Some clean-up for the handling of HSelect in LSE"
am: d3df33e6c2

* commit 'd3df33e6c24e3cd62991b2a65833f16dc05a17b8':
  Some clean-up for the handling of HSelect in LSE

Merge "Some clean-up for the handling of HSelect in LSE"

Fix x86-64 Baker's read barrier fast path for CheckCast.

Use an art::x86_64::Label instead of an
art::x86_64::NearLabel as end label when emitting code for a
HCheckCast instruction, as the range of the latter may
sometimes be too short when Baker's read barriers are
enabled.

Bug: 12687968
Change-Id: Ia9742dce65be7d4fb104688f3c4717b65df1fb54

Merge "Make run test 572-checker-array-get-regression gcstress friendly."
am: e52bf00fed

* commit 'e52bf00fed14563d23e38a74908180e152b26629':
  Make run test 572-checker-array-get-regression gcstress friendly.

Merge "Make run test 572-checker-array-get-regression gcstress friendly."

Make run test 572-checker-array-get-regression gcstress friendly.

The large array allocated in
test/572-checker-array-get-regression/src/Main.java used to
be too big for the small heap used in gcstress mode.  Use a
smaller array size that however still exercises the initial
issue checked by this regression test.

Bug: 26817006
Bug: 12687968
Change-Id: Id7080a18007cad9b5d4ac31b45f7df46213a908e

Merge "Fix AllocateBlockedReg."
am: 9b262128d7

* commit '9b262128d7dc2b8caa5265e1b3c967465e58ef77':
  Fix AllocateBlockedReg.

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

* commit '953e4c990456a71d351e17aee62e80e8133a186e':
  Revert "Revert "Revert "ART: Enable Jit Profiling in Mterp for arm/arm64"""

Merge "Fix AllocateBlockedReg."

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

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

Unfortunately, run-test interpreter on arm32 are still timing out, and the following jdwp tests on armv8 are failing:

  org.apache.harmony.jpda.tests.jdwp.Events.EventWithExceptionTest#testBreakpoint_BeforeException (no test history available)
  org.apache.harmony.jpda.tests.jdwp.Events.EventWithExceptionTest#testFieldAccess (no test history available)
  org.apache.harmony.jpda.tests.jdwp.Events.EventWithExceptionTest#testFieldModification (no test history available)
  org.apache.harmony.jpda.tests.jdwp.Events.EventWithExceptionTest#testMethodExit (no test history available)
  org.apache.harmony.jpda.tests.jdwp.Events.EventWithExceptionTest#testMethodExitWithReturnValue (no test history available)
  org.apache.harmony.jpda.tests.jdwp.Events.FieldAccessTest#testFieldAccessEvent (no test history available)
  org.apache.harmony.jpda.tests.jdwp.Events.FieldModification002Test#testFieldModifyEvent (no test history available)
  org.apache.harmony.jpda.tests.jdwp.Events.FieldModificationTest#testFieldModifyEvent (no test history available)
  org.apache.harmony.jpda.tests.jdwp.Events.MethodExitWithReturnValueTest#testMethodExitWithReturnValueException (no test history available)

This reverts commit 9687f244bdb5dd0b4d9dd804a7c8c7b4a911d364.

Change-Id: Iadac4902ab8d7eb574cc4abeba5f93388d59dcb4

Merge "Always visit ProfilingInfo, the counter could be reset."
am: 34937e2ed4

* commit '34937e2ed46fa9f56d99e9f32e0bfad050e5e798':
  Always visit ProfilingInfo, the counter could be reset.

Always visit ProfilingInfo, the counter could be reset.
am: 87170e7641

* commit '87170e764119d23caf5f0b319f8bf2ab6b4e974b':
  Always visit ProfilingInfo, the counter could be reset.

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

* commit '797f70cb2eacbd1b6c1db8bff0375abbcc27d3b9':
  Revert "Revert "ART: Enable Jit Profiling in Mterp for arm/arm64""

Always visit ProfilingInfo, the counter could be reset.

The counter could be reset or overflow, but we may have
inline caches populated.

bug:26775558

(cherry picked from commit ef404468d727332b088e6d461d0cf2f0ce0a897a)

Change-Id: Ib0cc8da05aa75d50c0474a1ddcb88cd60ae34707

Merge "Always visit ProfilingInfo, the counter could be reset."

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

Merge "ART: Fix arm mterp shadow handling"
am: dd08dd7109

* commit 'dd08dd7109f09dea53e64685f7db1439a0b07b2b':
  ART: Fix arm mterp shadow handling

Merge "ART: Fix arm mterp shadow handling"

ART: Fix arm mterp shadow handling

The shadow frame conventions for the interpreter require that any
time a virtual register is set, a corresponding shadow register
must also be set (cleared if non-reference).  The arm Mterp
implementation was failing to do this for some 64-bit operations.

Change-Id: Iee174d54318bf6ebb677966ad020d8cbe578986c

Try to substitute constructor chains for IPUTs.
am: f8b3b8bc37

* commit 'f8b3b8bc37fb04d8ae113ae6bfcf4de2f5a700d4':
  Try to substitute constructor chains for IPUTs.

Merge "ART/Thumb2: Disassemble SBFX/UBFX."
am: be89a6f1d9

* commit 'be89a6f1d9e41b154be3fe1da97cedb6964fbd35':
  ART/Thumb2: Disassemble SBFX/UBFX.

Merge "ART/Thumb2: Disassemble SBFX/UBFX."

ART/Thumb2: Disassemble SBFX/UBFX.

Change-Id: I856206de81f41959f68de0653db021903dd1a210

Merge "Fix run test 572-checker-array-get-regression on 32-bit archs."
am: fc06b93dee

* commit 'fc06b93dee031ec16272ec64fca92a0e639ae73e':
  Fix run test 572-checker-array-get-regression on 32-bit archs.

Merge "Fix run test 572-checker-array-get-regression on 32-bit archs."

Fix run test 572-checker-array-get-regression on 32-bit archs.

HCurrentMethod instructions are typed as long on 64-bit
architectures, but as integer on 32-bit ones.

Bug: 26817006
Bug: 12687968
Change-Id: Icec2f0b0629c0af7a7afeabb0fd2881babb86368

Merge "Fix ARM64 Baker\'s read barrier fast path for ArraySet."
am: 0c82e1afa3

* commit '0c82e1afa325cff3c409c8e35184f139f5c06f68':
  Fix ARM64 Baker's read barrier fast path for ArraySet.

Merge "Fix ARM64 Baker's read barrier fast path for ArraySet."

Try to substitute constructor chains for IPUTs.

Match a constructor chain where each constructor either
forwards some or all of its arguments to the next (i.e.
superclass constructor or a constructor in the same class)
and may pass extra zeros (of any type, including null),
followed by any number of IPUTs on "this", storing either
arguments or zeros, until we reach the contructor of
java.lang.Object.

When collecting IPUTs from the constructor chain, remove
any IPUTs that store the same field as an IPUT that comes
later. This is safe in this case even if those IPUTs store
volatile fields because the uninitialized object reference
wasn't allowed to escape yet. Also remove any IPUTs that
store zero values as the allocated object is already zero
initialized.

(cherry picked from commit 354efa6cdf558b2331e8fec539893fa51763806e)

Change-Id: I691e3b82e550e7a3272ce6a81647c7fcd02c01b1

Merge "Try to substitute constructor chains for IPUTs."
am: 927f40d925

* commit '927f40d925e2106598d413ac02e8d80463357258':
  Try to substitute constructor chains for IPUTs.

Merge "Try to substitute constructor chains for IPUTs."

Merge "ART: Update dalvik-cache for zygote after OTA" into nyc-dev
am: 8b0e9b87ab

* commit '8b0e9b87ab6e54237b2d1e101b8cce7ea6443238':
  ART: Update dalvik-cache for zygote after OTA

ART: Give better error messages in dex-file verifier
am: a5c4493c37

* commit 'a5c4493c37806f1b027c90f5ec3fe808adfdf3fe':
  ART: Give better error messages in dex-file verifier

Merge "ART: Update dalvik-cache for zygote after OTA" into nyc-dev

ART: Update dalvik-cache for zygote after OTA

Detect OTA artifacts and move them over to the regular dalvik-cache
on boot.

Bug: 25612095
Change-Id: I9a1a22641cc7833435098627a5cc6d954521c4ea

Merge "Mterp: Fix and restore mac host build"
am: dfcd2f43a6

* commit 'dfcd2f43a6694482a4e691261905508f65d8dc9e':
  Mterp: Fix and restore mac host build

Merge "Mterp: Fix and restore mac host build"

Try to substitute constructor chains for IPUTs.

Match a constructor chain where each constructor either
forwards some or all of its arguments to the next (i.e.
superclass constructor or a constructor in the same class)
and may pass extra zeros (of any type, including null),
followed by any number of IPUTs on "this", storing either
arguments or zeros, until we reach the contructor of
java.lang.Object.

When collecting IPUTs from the constructor chain, remove
any IPUTs that store the same field as an IPUT that comes
later. This is safe in this case even if those IPUTs store
volatile fields because the uninitialized object reference
wasn't allowed to escape yet. Also remove any IPUTs that
store zero values as the allocated object is already zero
initialized.

Change-Id: If93022310bf04fe38ee741665ac4a65d4c2bb25f

Merge "ART: Give better error messages in dex-file verifier"
am: 4c3127698b

* commit '4c3127698b90e0475f233433eda936424b8e945a':
  ART: Give better error messages in dex-file verifier

ART: Give better error messages in dex-file verifier

Try to decode field and method names when an access-flag violation
has been found. This is not guaranteed to work, if the file is
broken enough.

Bug: 27064244
Bug: 27070841

(cherry picked from commit c9f0ba1a4a2c9366ffc2dc2f9c2d8d3f09bb2112)

Change-Id: Ie913076462e958d4f21b481631bc874cf6f67c0d

Merge "ARM: Add direct calls to math intrinsics"
am: c70646e3c7

* commit 'c70646e3c75ec97648bfa7c295e4b1685895afcc':
  ARM: Add direct calls to math intrinsics

Merge "ART: Give better error messages in dex-file verifier"

Merge "ARM: Add direct calls to math intrinsics"

ART: Give better error messages in dex-file verifier

Try to decode field and method names when an access-flag violation
has been found. This is not guaranteed to work, if the file is
broken enough.

Bug: 27064244
Bug: 27070841
Change-Id: Ie913076462e958d4f21b481631bc874cf6f67c0d

Default methods should not be found by getDeclaredMethod
am: 3b7027e261

* commit '3b7027e26106e7bfaa2bfae0f24489372f7ddbf0':
  Default methods should not be found by getDeclaredMethod

Default methods should not be found by getDeclaredMethod

(cherrypick commit 64fdc804f6b4d8a6f7d86bd9672e3bb1cde7056f)

Bug: 27060609

Change-Id: I77337f5d4c6bb9dd14f57012a9daa1e56a760f1a

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

Fixes: missing sign extension in iget template
       Call to wrong branch profiling helper in arm/goto_16 and arm/goto_32
       Missing export PCs
Reworks: Branch handlers to reduce cost of branch profiling.

Re-enables Jit profiling for both Arm and Arm64.

Performance note:
Branch profiling is relatively expensive, though the real
cost will depend on branch frequency.  Taking a very
branch intensive benchmark, CaffeineMark's logic test, we
see the following scores (higher is better):

Mterp  (profiling off) 6187
Mterp  (profiling on)  4305

Switch (profiling off) 3931
Switch (profiling on)  2032

This reverts commit 95717f0010e7a9445450f4d39babfaf3a83e29b5.

Change-Id: Ia2ef8b54ce95bfa86178b89c43f8a703316b2944

Merge "openjdkjvm: Don\'t leak the results of JVM_RawMonitorCreate."
am: 96bbc8fcac

* commit '96bbc8fcac2d408e0afdcae933653198fb8bbe92':
  openjdkjvm: Don't leak the results of JVM_RawMonitorCreate.

Merge "openjdkjvm: Don't leak the results of JVM_RawMonitorCreate."

openjdkjvm: Don't leak the results of JVM_RawMonitorCreate.

bug: 27050501

Change-Id: Ib0db3e5a1ec412de07455f09fe88b3102b347e46

Fix ARM64 Baker's read barrier fast path for ArraySet.

Do not exhaust the pool of scratch (temporary) registers
gratuitously when emitting an instrumented array load with a
large constant index.

Bug: 26817006
Bug: 12687968
Change-Id: I65a4fe676aa3c9e2c8d7e26195d9af6432c83ff9

Merge "openjdkjvm: clean up verbose logging."
am: 17077d9090

* commit '17077d90907120d75cc4f534ce4de741da3bff2d':
  openjdkjvm: clean up verbose logging.

Merge "openjdkjvm: clean up verbose logging."

openjdkjvm: clean up verbose logging.

bug: 26994173
Change-Id: I4b55d68fa89737e25e9d7b10b335423b3ea4bef2

Some clean-up for the handling of HSelect in LSE

Use $noinline and throwing an Exception for doing no-inlining in
the test.
Also create reference info early for HSelect if it has reference
type, which may help the pre-existence-based alias analysis.

Change-Id: Ia825058a457b049f87a13087bb78f5233513e0a8

Merge "Check pages are readable after mprotect for SEGV_ACCERR diagnosis"
am: 5b00f4169f

* commit '5b00f4169f1a153f85216f8624cfbd691458060f':
  Check pages are readable after mprotect for SEGV_ACCERR diagnosis

Merge "Check pages are readable after mprotect for SEGV_ACCERR diagnosis"

Check pages are readable after mprotect for SEGV_ACCERR diagnosis

We have a suspected mprotect issue where we see seg faults after pages
are mprotected with PROT_READ|PROT_WRITE with the SS collector on
host. This change attempts to see if earlier reads would cause faults
similarly.

Bug: 19894268

Change-Id: I041a663c6b55b747120915f73a0db5f566744ed8

Merge "Default methods should not be found by getDeclaredMethod"
am: e4ddacdb1f

* commit 'e4ddacdb1f0798e084b9ce2de7acac797f2c0a75':
  Default methods should not be found by getDeclaredMethod

Merge "Default methods should not be found by getDeclaredMethod"

Merge "Revert "Make it possible to enable native debugging through debug flags""
am: 9fab717bb6

* commit '9fab717bb67e6d141ca7a6d6c33adbf5b9d3dba7':
  Revert "Make it possible to enable native debugging through debug flags"

Merge "Revert "Make it possible to enable native debugging through debug flags""

Revert "Make it possible to enable native debugging through debug flags"

The change causes issues in test-art-target-gtest-jni_internal_test32

This reverts commit c94a61f06ffc13288c67891048128c987b29bf33.

Change-Id: Iecfe3c6874d7b0dd59f10156fe2eb743ab7221dc