Merge "Create test for field access in obsolete methods."
am: bd1ec61522

Change-Id: I0c071547bd5070d88e1e91d6796365366617a012

Merge "Create test for field access in obsolete methods."

Merge changes Ie74f2dbe,I092d2e95,I5a75f5c4
am: 07001c8540

Change-Id: I7152ae8d2689fd2995d4c9c5443ff62a59734754

Fix 616-cha assumptions.
am: 6cb21552c9

Change-Id: If520e0315882f5d48814d84a584dd89634cc3f22

Fix JIT crash due to unverified dead code
am: 857f058d4b

Change-Id: I7d8c08c35ac56ab72aaae41663794c7f96b8b485

Use interpret-only instead of verify-at-runtime when testing JIT
am: 0966558b07

Change-Id: Ib0d76c219a887086626e6384ad19ec6b8c05eb9b

Merge changes I8cc87bac,If1a92e47
am: 1aa03b379f

Change-Id: I830f6fb2937d561a5d7762ac35b93ea6e6f72df7

Cleanup jvmti tests
am: ca5b9dccce

Change-Id: I6acbcddf3eb32749d5c7e15bbe7a27d301f830ee

Revert "Revert "Basic obsolete methods support""
am: ce77fc0e7f

Change-Id: I086f8b0b3c23ec64ffb577e9946c477dc5d82794

Merge "ARM: VIXL32: Use 16-bit encoding for B(cond,...) where possible."
am: 1ffb6fc921

Change-Id: I2c171a03e02067998e8cebc8aedeb6265d6db89d

Merge changes Ie74f2dbe,I092d2e95,I5a75f5c4

* changes:
  Fix 616-cha assumptions.
  Fix JIT crash due to unverified dead code
  Use interpret-only instead of verify-at-runtime when testing JIT

Create test for field access in obsolete methods.

Test: ./test/run-test --host 918
Change-Id: Idf27a0a713e12876e35de5c24bf5d12984881927

Merge changes I8cc87bac,If1a92e47

* changes:
  Cleanup jvmti tests
  Revert "Revert "Basic obsolete methods support""

Fix 616-cha assumptions.

616-cha relies that some classes are not loaded at startup. Disable the
app image to prevent that.

Test: ART_TEST_JIT=true m test-art-host-run-test-616-cha
Change-Id: Ie74f2dbeae0f898d42051f248492126615ffdad6

Fix JIT crash due to unverified dead code

The JIT compiler assumes that it only gets completely verified code. To
work around potential unverified dead code it uses kAccDontBotherCompile
flag set during runtime verification. However, if a class is verified
during a prior dex2oat the flag is not persisted and JIT happily things
that everything is ok.

The simplest fix is to mark classes with potential unverified dex code
as verify at runtime. We only do this for apps and assume that
everything in the boot image is well formed.

Test: m test-art-host
Bug: 31000839
Change-Id: I092d2e9553cd1c577036d78e8563a7a39d6cb7b9

Use interpret-only instead of verify-at-runtime when testing JIT

That's because we run most of our products with interpret-only and not
verify-at-runtime.

Test: ART_TEST_JIT=true m test-art-host
Change-Id: I5a75f5c47a9e811db3ee7c3475a03fb648dba80d

Merge "ARM: VIXL32: Use 16-bit encoding for B(cond,...) where possible."

Merge "MIPS64: Improve string and class loads."
am: 06ce6d4359

Change-Id: I6b35d073cb4088a65dcfe31f5403b1680aa9c53c

Merge "MIPS64: Improve string and class loads."

Merge "Sharpen HLoadClass from inliner."
am: eaaaaef76b

Change-Id: I951c3f54a90a20d39b3e92c53649de64019be5e0

Merge "Make 141-class-unload more robust under JIT."
am: ee3a0bde75

Change-Id: I4b31b791c4af840123fd1ae98d92de72bb4d1fe3

Merge "Sharpen HLoadClass from inliner."

Merge "Make 141-class-unload more robust under JIT."

Merge "CHA guard optimization (elimination/hoisting)."
am: fb5a6bb830

Change-Id: Ic40219b4344e34190c875ecd9e9f86fc484cb1a2

Cleanup jvmti tests

Deduplicate the 'run' scripts into the test/etc/run-test-jar file.

Test: mma -j40 test-art-host
Change-Id: I8cc87bacedb393bfa11cdf2345bfd7a4341caa37

Merge "CHA guard optimization (elimination/hoisting)."

MIPS64: Improve string and class loads.

This adds most kinds of string/class loads.
JIT string/class loads are TBD separately.

This also fixes Mips64Assembler::LoadLabelAddress()
(adding a constant to a 64-bit address must be done
using daddiu, not addiu).

Test: test-art-host-gtest
Test: booted MIPS64 (with 2nd arch MIPS32R6) in QEMU
Test: "make -j1 ART_TEST_DEFAULT_COMPILER=false ART_TEST_OPTIMIZING=true
       ART_TEST_INTERPRETER=false ART_TEST_JIT=false
       ART_TEST_PIC_TEST=true test-art-target-run-test64"

Change-Id: I1f94ece4cd202382c11167e1ed958e9d08d92822

CHA guard optimization (elimination/hoisting).

Test: manual by checking the dump-cfg output.

Change-Id: I254e168b9a85d2d3d23e02eea7e129c1bc9ab920

Revert "Revert "Basic obsolete methods support""

This reverts commit b81a9840b44480bfeacd74b8d9f51e06f295411d.

There were 2 issues with the original CL's test 916-obsolete-jit that
caused it to sporadically fail.

First, when checking if we had jitted the function under test in
916-obsolete-jit we failed to check to see if the function 'doCall',
which is used to work-around bugs in our deoptimization & compilation
systems, had also been jitted.

In the case where the 'sayHi' function had been jitted but the
'doCall' function had not we would (correctly) fail to redefine the
'Transform' class since we would not be able to deoptimize the 'sayHi'
function since it is under a quick_to_interpreter_bridge (runtime)
frame.

Secondly, the function Main.isInterpretedFunction was incorrect and
would always return false, regardless of the actual state of the
function. This would cause the test to fail as the
quick_to_interpreter_bridge frame prevented deoptimization of the
obsoleted function. Usually the warm-up period was enough to make sure
the methods were jitted anyway but this was not guaranteed.

Both of these problems become more likely to occur on systems with
more cpu contention such as the buildbots.

Test: stress --cpu 60 &; while ./test/run-test --host --jit 916; do ; done

Reason for revert: Fixed bug in test that was causing failures.

Original Tests:

Test: ./test/run-test --host 914
Test: ./test/run-test --host 915
Test: ./test/run-test --host 916
Test: mma -j40 test-art-host
Test: ART_TEST_JIT=true \
      ART_TEST_INTERPRETER=true mma -j40 test-art-host
Test: ./art/tools/run-jdwp-tests.sh --mode=host --variant=X64
Test: ./art/tools/run-jdwp-tests.sh --mode=host --variant=X64 --no-jit

Bug: 32369913
Bug: 33630159

Change-Id: If1a92e47b90965a7dc21c5826185debe62bd1554

Merge "Improved induction var and range analysis around types."
am: d54f43ca39

Change-Id: I580b92820d94de55bf8478d41ba8a2d5e3ab3832

Merge "Improved induction var and range analysis around types."

Merge "Revert "Revert "Revert "Use interpret-only instead of verify-at-runtime when testing JIT""""
am: 481593d874

Change-Id: Ie2517bba2c533d5a53113c0da326ca4c24000ff2

Merge "Revert "Revert "Revert "Use interpret-only instead of verify-at-runtime when testing JIT""""

Revert "Revert "Revert "Use interpret-only instead of verify-at-runtime when testing JIT"""

Reverting due to gtest crashes.

This reverts commit 3a3e4c10e4d08c9fa1fbeba6d74933964a3bb47f.

Change-Id: I7e1adf0d886a1cdaf3152469b3c11a33131ee498

Merge "Revert "Fix 616-cha assumptions.""
am: 761460ee54

Change-Id: I780638160fd9bb88c219f68052a5c31c03782ab4

Merge "Revert "Fix 616-cha assumptions.""

Revert "Fix 616-cha assumptions."

The change that this depends on has been reverted:
    https://android-review.googlesource.com/316224

This reverts commit 5c77917ca94b33576b4a6bb850bdb578df378292.

Change-Id: I590dabcc7d12e661536b4234e45b7e763c34c35d

Merge "Revert "Fix JIT crash due to unverified dead code""
am: 263cc99a5f

Change-Id: I7fb96da9847c216a5a8e06b6f8470fc09f01cd0e

Merge "Revert "Fix JIT crash due to unverified dead code""

Revert "Fix JIT crash due to unverified dead code"

Broke method_verifier_test and image_test.

Bug: 31000839

This reverts commit 132eb43396bdb0b9bdacf069289f019d85d358fc.

Change-Id: I85923e6bdb3e76f54978658aad816d4c79fc339e

Merge "Fix 616-cha assumptions."
am: 2d76677d7a

Change-Id: I60d4a592defe603c298ff96b2a3f7c0282243b83

Improved induction var and range analysis around types.

Rationale:
Lots of code should not depend on int only. This CL generalizes
the kinds of types that can be optimized after analysis. As part
of the CL, however, a minor cleanup regarding type safety of the
stored induction var analysis results is required. This further
improved our int benchmark, and brings the long benchmark up-to-par.

Test: m test-art-host-run-test
Change-Id: I5dfb623dabf9113de90c2f6da99328dda8f8b60b

Merge "Fix 616-cha assumptions."

Merge "Temporarily disable 153-reference-stress."
am: 9b8e6de85f

Change-Id: Id3a5b74c23eafd3f2a19fbf69cacca74cd921d26

Merge "Temporarily disable 153-reference-stress."

Merge "Fix arena allocation counting."
am: a7ad00aad1

Change-Id: I90d2755e6e12917afc94ef58e46adb0d7fb73b50

Merge "Fix arena allocation counting."

Merge "Fix JIT crash due to unverified dead code"
am: b44553224c

Change-Id: I59641114a6415f01abab3b50d11cc63986f7c7e6

Fix 616-cha assumptions.

616-cha relies that some classes are not loaded at startup. Disable the
app image to prevent that.

Test: ART_TEST_JIT=true m test-art-host-run-test-616-cha

Change-Id: I432c300a3cf3de7c0f2f9ab02d7576e92edac844

Merge "Fix JIT crash due to unverified dead code"

Sharpen HLoadClass from inliner.

Also cleanup HLoadClass constructor.

Test: ART_TEST_JIT=true m test-art-host-run-test

Change-Id: I8f803b05fb8a7267d1421ca9c032e624f27efed3

Fix arena allocation counting.

Add the "CHA" allocation kind description. Move the
statistics to the heap to avoid excessively large frames
triggering errors with -Werror, -Wframe-larger-than=.

Test: m test-art-host
Test: m test-art-host with arena allocation counting.
Change-Id: I7947a3a17b4c1ed773742516b1f7ab9a58c92150

Fix JIT crash due to unverified dead code

The JIT compiler assumes that it only gets completely verified code. To
work around potential unverified dead code it uses kAccDontBotherCompile
flag set during runtime verification. However, if a class is verified
during a prior dex2oat the flag is not persisted and JIT happily things
that everything is ok.

The simplest fix is to mark classes with potential unverified dex code
as verify at runtime. We only do this for apps and assume that
everything in the boot image is well formed.

Test: m test-art-host-706-jit-skip-compilation
Bug: 31000839

Change-Id: Ib73de1888581bb7202474cfd7aca70af4cc2cc00

Merge "Revert "Revert "Use interpret-only instead of verify-at-runtime when testing JIT"""
am: 71464a10a4

Change-Id: I10552f5fba712022ce6a21e5f7b3fbcfe7f71ee0

Merge "Revert "Revert "Use interpret-only instead of verify-at-runtime when testing JIT"""

Merge "Don't unquicken RETURN_VOID_NO_BARRIER with vdex."
am: c0e74baddb

Change-Id: I977bb2194de2ea070427663d72923646fc4bcea0

Merge "Don't unquicken RETURN_VOID_NO_BARRIER with vdex."

Make 141-class-unload more robust under JIT.

Make sure jit is stopped before counting maps.

bug: 29712509
test: 141-class-unload
Change-Id: I52c0fed74556fa228cd73c9c0bc6a1d019df0a1c

Merge "Add Thread entry to signal if the thread can call into java"
am: d96b96dd97

Change-Id: I70dd4e0fe4727daf13381a621042e9a55d80c862

Add Thread entry to signal if the thread can call into java
am: ccd56958eb

Change-Id: I6b5f99beea90fe4cdd1245a36e984adecc278530

Merge "Add Thread entry to signal if the thread can call into java"

Merge "ARM: Fix vixl related branch issue in JNI."
am: 685e4a3252

Change-Id: Idb507ed12f2af87b2f00210b3c1becfd9603fe19

Merge "ARM: VIXL32: Implement some dispatch optimizations."
am: 76873db28c

Change-Id: I8d2ba47fd208c0da2a5ca086a7c72bcacf3733e3

Merge "ARM: Fix vixl related branch issue in JNI."

ARM: Fix vixl related branch issue in JNI.

Fix VIXL asserts in EncodeLabelFor in JNI macro assembler when
label is out of range of 16-bit branch (now a veneer is correctly
used).

bug:32545704

Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-host
Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-target
Change-Id: Ie6401394cf364daeaaf107b42275997d2edf5b6d

Merge "ARM: VIXL32: Implement some dispatch optimizations."

Don't unquicken RETURN_VOID_NO_BARRIER with vdex.

The quickening of the RETURN_VOID instruction is based on
local information (no final fields) that doesn't get affected
by a boot image update.

Test: test-art-host, verifier_deps_test
bug:30937355

Change-Id: I12b22d7fcda6dc681a32ff752c3871f6e84f19a1

Temporarily disable 153-reference-stress.

This is intermittently failing. See 33389022 #13.

Note this could fail either in CMS or CC, gcstress or not.

Bug: 33389022
Test: test-art-host.
Change-Id: I257c2c771fcab9405c1a15fa6e93364908fd1294

Add Thread entry to signal if the thread can call into java

Compiler threads (AOT or JIT) should not call into Java as they have no
peers (which may lead to crashes, e.g. b/33067273)

Bug: 32602185
Bug: 33067273

Test: m test-art-host-run-test; m test-art-host-gtest
Change-Id: I97dda7a5444643db3c5d5318339a65a602f709e8

Merge "Allow redefined methods/classes to access fields"
am: 5b32b91d73

Change-Id: I75fff4a7353051adb0843bac579edf81247c1fb1

Merge "Allow redefined methods/classes to access fields"

Merge "Add HVariableInputSizeInstruction."
am: 89b31a3369

Change-Id: I4e3778a9ecf905e6a968e3fc6109209ad56087bb

Merge "Add HVariableInputSizeInstruction."

Allow redefined methods/classes to access fields

Previously we were not updating the ArtFields of redefined classes.
This meant that the fields of redefined classes could not be accessed.

Bug: 32369913

Test: ./test/run-test --host 917
Test: mma -j40 test-art-host

Change-Id: If039e883e9a57970a0cffbc7dcbbaaf2d44490aa

Merge "Clear exception if FindClass fails."
am: 20b66e727a

Change-Id: I2454d59bdfaa455462ac5035b7cd0118233a2b7e

Merge "Clear exception if FindClass fails."

Clear exception if FindClass fails.

Test: verifier_deps_test.
bug: 30937355
Change-Id: I9916ec00010de012939462d4f30147e8b64fe279

Merge "Do not resolve types with vdex."
am: 0f69f7425c

Change-Id: Id2eda0a19fd4d09092801b6ffde76b1239782987

Merge "Do not resolve types with vdex."

ARM: VIXL32: Use 16-bit encoding for B(cond,...) where possible.

If conditional branch's target is known to be not far from branch
(within the range of 254 bytes), 16-bit encoding can be used.
However, we can't assume this by default for branches to a basic
block or a slow path; if we do and fail the range requirement,
veneer pool will be emitted.

Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-host
Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-target
Change-Id: I2fbe6d1a43bc2d1b54472c2c3fe05a575e5634f2

ARM: VIXL32: Implement some dispatch optimizations.

Implement dispatch optimizations for LoadClass, LoadString and
HInvokeStaticOrDirect. Still need to implement JitTables.

Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-host
Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-target

Change-Id: I6e24bc11f3915aa92eede687a864b7ea3a6ab165

Do not resolve types with vdex.

We don't need to, and it's time expensive.

test: test-art-host, 2x faster vdex compilation time.
bug: 30937355

Change-Id: Ic04ebfee9172ed676fbb38e6901615ed3473352a

Merge "ARM: VIXL32: Do a code cleanup."
am: 7e9a562e42

Change-Id: I244776733d4b2944d0988120f04f7c6e8e0fe4f7

Merge "ARM: VIXL32: Do a code cleanup."

Merge "Remove dex cache string from mirror::Class."
am: 913e6e6f36

Change-Id: Ie5c693e2e6e7daa549abbf1152749af597d0faef

Merge "Remove dex cache string from mirror::Class."

Merge "Flush JIT data cache when committing code."
am: 4c1667fd52

Change-Id: If410759aad4f2aa3c442de7b36b48342d3f25347

Merge "Flush JIT data cache when committing code."

Merge "Fill in empty iftable early"
am: a3dccf967a

Change-Id: Iff2e45b20e8dba1c4cf41a5458a9d70d03c89189

Merge "Fill in empty iftable early"

Fill in empty iftable early

Previously it would not get filled in and remain null for some
erroneous classes. This caused crashes for things like
GetIfTableCount.

Added regression test (sample failure):
class_linker_test F 12-15 14:20:09 28771 28771 class-inl.h:532] Check failed: ret != nullptr java.lang.Class<Erroneous>

Bug: 33650297
Bug: 33642884

Test: test-art-host

Change-Id: I506210bad17d6d2c58ca7870165f8a4a514f64fe

Merge "Fixed signal 11 bug by not testing resolved method first."
am: 2c43590dc2

Change-Id: I1def3c48276633519d8d6871f41a58fe654f6109

Merge "Fixed signal 11 bug by not testing resolved method first."

Merge "Fine tuning the induction analysis."
am: e9f3269b54

Change-Id: Iaf633fbaa6626b60960f1c28d8f08295883c5272

Merge "Fine tuning the induction analysis."

Merge "Modify reporting when only true divergences are requested."
am: 45d75c6194

Change-Id: I14d99574fa0575dda01136ed7674ccc2f553ae88

Merge "Modify reporting when only true divergences are requested."

Add HVariableInputSizeInstruction.

Make HPhi and HInvoke subclasses of the new instruction.

Test: m test-art-host-run-test
Change-Id: I303c725876f1f4407b98702d92370be25193fc53

Merge "ART: Move to libbase StringPrintf"
am: 995a7db371

Change-Id: I1e08e8c07c20dbf69de2576d3aa1ffddcca9be82

Fine tuning the induction analysis.

Rationale:
Based on some self-imposed "blind" testing, improved
the induction variable analysis for typical cases
that provide a bit more elaborate HIR.

Test: test-art-host
Change-Id: I6e6bbf99928c29973178fa48f3942b14bf069944