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""

test-art-host-run-test-debug-prebuild-jit-relocate-ntrace-gcstress-checkjni-picimage-npictest-ndebuggable-974-verify-interface-super64 fails on the build servers.

It's unclear if it's because of this CL but revert again to check if it fixes the problem.

Locally everything is green...
This reverts commit 0b64a5195b38773084ff94618ac22284bc855b92.

Change-Id: I4668b8d8f8c9672fd942a693d98a20113fbb6b9e

Merge "Add dequickening support."

Merge "ARM: VIXL32: Fix PackedSwitch."

Merge "Fix valgrind issue with thread_pool_test."

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

Merge "Make ImageWriter and PatchOat helper classes nested."

Add dequickening support.

Both vdex and BCI require full dequickening support. This
change implements dequickening by using the existing encoded
quickening, and adds the checkcast quickening to that encoding.

bug: 30937355
bug: 32369913
Test: dex_to_dex_decompiler_test.cc test-art-host

Change-Id: Ie95f46946d59b28157d6e47dcf4a859be032d1c3

Fix valgrind issue with thread_pool_test.

Make sure the task list is empty at the end of the test.

Test: thread_pool_test

Change-Id: I155a8eb90dc83a0dd9128f0c0bc64deeba7a92e9

Merge "ART: Make libopenjdkjvmti a run-test dependency"

ART: Make libopenjdkjvmti a run-test dependency

Add libopenjdkjvmti as a host dependency for running tests, as
required by the tests 900-913.

Bug: 31455788
Test: m test-art-host
Change-Id: I4910f587e35615a10168a39ef2c60d3d80981fc4

Merge "Fix false positive stack ASan errors on x86."

Merge "ART: Remove ti library dependency for ti tests"

Fix false positive stack ASan errors on x86.

Call a special ASan function in x86 DoLongJump to let the tool know
that strange things are going to happen with the stack pointer. Other
architectures do not need this because they do the stack pointer
manipulation in a separate ((noreturn)) function, and noreturn
attribute triggers the same conservative behaviour in ASan.

Bug: 24199050
Bug: 32849425
Test: SANITIZE_HOST=address stub_test32
Change-Id: Ic639d2d84474d875487e66369c034db9907a7e8f

ART: Remove ti library dependency for ti tests

The agent so should not depend on libopenjdkjvmti.so.

Bug: 31455788
Test: m test-art-host
Change-Id: I7825d2a475a9651483b9c7cd11016d4f2dba23f9

Merge "ART: Rewrite test 913 printing"

Merge "Change offline profile to store type_idx of classes."

Merge changes I710ca0ed,I576e7926,Id0144b68

* changes:
  ART: Stop JIT in 913-heaps
  ART: Add ScopedJitSuspend
  ART: Change ThreadPool::Wait behavior

ART: Rewrite test 913 printing

Detailed printing has to be postponed, as many required functions
are not callback-safe. Refactor the printing logic.

Bug: 31385354
Test: m test-art-host-run-test-913-heaps
Change-Id: I690752bffe4b27292eb2f2e89b1b6281a5836bb2

Change offline profile to store type_idx of classes.

Changed from storing class_def_idx to type_idx. Allows dexlayout to move
class defs without affecting the profile.

Test: mm test-art-host

Change-Id: I8a040adf0f5f47a1592e18aeb55edde01cb3e1ed

Merge "[MIPS64] JNI Compiler: compiler independent test"

Make ImageWriter and PatchOat helper classes nested.

Nested classes are better for encapsulation. For example,
we've had an art::FixupRootVisitor in both image_writer.cc
and patchoat.cc and we didn't get an ODR violation only
because these two files are never used in the same process.

Test: Rely on the TreeHugger.
Change-Id: I39a86994923a9c0faa5f05d8c9ede982ce9c0c4c

ART: Stop JIT in 913-heaps

To avoid any JIT interaction, stop the JIT thread pool in
test 913-heaps.

Bug: 31385354
Test: m ART_TEST_JIT=true test-art-host-run-test-913-heaps
Change-Id: I710ca0ed3736b5394863aa76aff9ea2ae7d50cb1

ART: Add ScopedJitSuspend

Add a helper to suspend the JIT in a scope. This will
wait for the JIT to quiesce, finishing already running
compile jobs. Note that the queue will not be drained,
jobs not picked up by the pool, yet, will remain in
the queue.

Bug: 31385354
Test: m test-art-host
Change-Id: I576e7926423f19a8f382be1263838cd924955f1c

ART: Change ThreadPool::Wait behavior

When a pool is in the stopped state, Wait() will not wait for all
tasks to complete.

Bug: 31385354
Test: m test-art-host-gtest-thread_pool_test
Change-Id: Id0144b685ee2fddf1a1c2c2ca334251130121033

ARM: VIXL32: Fix PackedSwitch.

Fix the case when the jumptable is very far and offset doesn't
fit ADR offset field.

Test: export ART_USE_VIXL_ARM_BACKEND=true && \
      mma test-art-host dist && mma test-art-target dist

Change-Id: I2b8ac04eb343947f9a9301f3ce6285794ddad4b8

Merge "Revert "jni: Punt to generic JNI for mips""

Merge "jni: Add @CriticalNative support for MIPS32/MIPS64"

Merge "MIPS32: Assign missing entrypoints in InitEntryPoints"

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

This reverts commit f1d926592ca7030ad12abc75f2b56643a9e23645.

Change-Id: I161f2e0381a31cad8b014d7384eac2e776ae5411

Merge "Add const-class benchmark."

Add const-class benchmark.

Also fix indentation in const-string benchmark.

Test: vogar --benchmark art/benchmark/const-class/src/ConstClassBenchmark.java
Test: vogar --benchmark art/benchmark/const-string/src/ConstStringBenchmark.java
Bug: 30627598
Change-Id: I04ca9514f6e647640e2c0d9d65fdc4ace652927a

[MIPS64] JNI Compiler: compiler independent test

Fixed MIPS64 stackArgsSignExtendedTest that did not work properly
when code was built with clang. The the test is now written in an
asm stub and should be completely compiler indepedent.

Test: mma test-art-target-gtest-jni_compiler_test

Change-Id: Idefef3a8414dd8b88a28d2d8862a724bc72402a7

Merge "MethodHandles: Check return types when resolving handles."

Merge "ARM: VIXL32: Arch specific operations."

Merge "Address comments from aog/303658."

Merge "Ensure CC GC is used for the read barrier config."

Merge "Change iftable to never be null"

Address comments from aog/303658.

Bug: 12687968
Test: test-art-host with CC.
Change-Id: I87a3be336cb1c6b17acc515415c325464ffddc01

Ensure CC GC is used for the read barrier config.

To prevent the -Xgc option from causing unsupported GC types to run.

Bug: 12687968
Test: test-art-host with CC.
Change-Id: I10b42190c0888342e9127c52e863ddae82e4d7a1

Change iftable to never be null

Simplifies code generation by removing a null check. The null case
is rare.

Ritzperf code size: 13107624 -> 13095336

Also addressed comments from previous CL.

Bug: 32577579

Test: test-art-host, run ritzperf both with CC

Change-Id: I2b31e800867112869d7f0643e16c08826296979e

ARM: VIXL32: Arch specific operations.

Tests which now start to pass:
  550-checker-multiply-accumulate
  564-checker-negbitwise

Test: export ART_USE_VIXL_ARM_BACKEND=true && \
      mma test-art-host dist && mma test-art-target dist

Change-Id: I0b057486a8e0e48afbc52dd18b61ea16c671bec2

Merge "A few more tests for LICM."

A few more tests for LICM.

Rationale:
These tests ensure that some possible improvements in LICM
(e.g. hoisting add in environment out of the loops) will
be done right in the future.

Test: test-art-host

Change-Id: Ic00775ccf03408273d427ee9399df55b091c9fa4

MethodHandles: Check return types when resolving handles.

Test: m test-art-host-run-test-956-methodhandles
Bug: 30550796
Change-Id: Idff2a3649d812b2901794594661327154299cc05

Merge "ARM: VIXL32: Fix VisitTypeConversion."

Merge "Use OUT_DIR_COMMON_BASE if OUT_DIR is unset."

Merge "ART: Prioritize reference table dump"

Merge "ART: Print debug info in test 913"

Merge "Pass object instead of class to instanceof entrypoint"

Pass object instead of class to instanceof entrypoint

Reduces code size. Also avoid read barrier for kArrayCheck case.

Bug: 32577579

Test: test-art-host, test-art-target CC

Change-Id: Ia890f656fe166b2d39c522b63a8a6469404134ae

ART: Print debug info in test 913

In case we find an unexpected thread root, dump all stacks.

Bug: 31385354
Test: m test-art-host-run-test-913-heaps
Change-Id: I62635950d5afe639d0f86f2b436809d75375956f

Merge "Add option to force all code to be deoptimizable"

Merge "ART: Name dex2oat main thread "dex2oat""

Add option to force all code to be deoptimizable

Currently this can only be passed during runtime startup. We will add
support for doing it from the zygote in the future.

Bug: 28769520
Bug: 32369913

Test: m test-art-host

Change-Id: I5eb16887ea14b140217c123b3effd47c5c8b2768

Merge "Fix use of std::stoi without error handling."

ART: Prioritize reference table dump

Sort the reference table summary dump by highest count, prioritizing
repeated types and instances. This will help with triaging leaks.

Add a test.

Bug: 32857749
Test: m test-art-host-gtest-reference_table_test
Change-Id: I7e61881b5badf9ac2b6b72333f31437ab498caee

Merge "Optimize non-intrinsic object array copy for Baker."

Fix use of std::stoi without error handling.

Changed to use strtoul, with a break if it fails to read the checksum.

Test: mm test-art-host
Bug: 32066687
Change-Id: Ic1299c23aae15a8329ad57b4eca5c0857f6eefb6

Merge "Change 64 bit atomic counter to size_t"

Optimize non-intrinsic object array copy for Baker.

Check the gray bit per array rather than per element.

Bug: 12687968
Test: test-art-host with CC.
Change-Id: I20b44e522e66f0b156644b93a2885a8e7f40f8db

Merge "Clean up interface check cast"

Clean up interface check cast

Changed arm, arm64 to use less labels and removed forward branch
in the success case.

Cleaned up X86, X86_64 to remove the is_null label.

Bug: 12687968
Bug: 32577579

Test: test-art-host, test-art-target CC
Change-Id: Iba426dff548b2ef42198fad13efeb075f7c724a7

ART: Name dex2oat main thread "dex2oat"

For better representation in tools like top and ps.

Test: m
Change-Id: Ia93271a850d5b0dfa79d09ea4f6b02b5230727d0

Merge "ART: Make test 030 slightly more robust"

ARM: VIXL32: Fix VisitTypeConversion.

Pass 422-type-conversion and some other tests.

Test: export ART_USE_VIXL_ARM_BACKEND=true && \
      mma test-art-host dist && mma test-art-target dist

Change-Id: I89a491a7724ae5286b717133eec16433b7066845

Merge "Revert "Revert "Revert "JIT root tables.""""

Revert "Revert "Revert "JIT root tables."""

libcore failures:
dalvikvm32 F 11-14 03:04:06 14870 14870 jit_code_cache.cc:310] Check failed: new_string != nullptr

This reverts commit 75afcdd3503a8a8518e5b23d21b6e73306ce39ce.

Change-Id: I5a6b6b48aa79a763d1ff1ba4d85d63811254787d

Merge "Update test expectation."

Update test expectation.

test: 902
Change-Id: I0d9082a34a66c5f2aadec90eda0c5c2a0feef2e4

Merge "Revert "Revert "Rename vdex-fd to output-vdex-fd, add input-vdex-fd."""

Merge "Revert "Revert "JIT root tables."""

Merge "Update 902 test to highlight JIT issue."

Update 902 test to highlight JIT issue.

Test: 902-hello-transformation
bug: 32821077
Change-Id: Ic7d267a5e62d4e001d1b85f1cc3d10546814bd18

Merge "ARM: VIXL32: Implement more codegen visitors."

Merge "Add visiting for class loaders in StickyMarkSweep"

Use OUT_DIR_COMMON_BASE if OUT_DIR is unset.

In build/make/core/envsetup.mk the logic for setting OUT_DIR is this:

ifeq (,$(strip $(OUT_DIR)))
ifeq (,$(strip $(OUT_DIR_COMMON_BASE)))
OUT_DIR := $(TOPDIR)out
else
OUT_DIR := $(OUT_DIR_COMMON_BASE)/$(notdir $(PWD))
endif
endif

To get the correct value for its out_dir variable, buildbot-build.sh
needs to do the same, otherwise it'll generate incorrect target names
in environments where OUT_DIR_COMMON_BASE is set but OUT_DIR unset.

Test: art/tools/buildbot-build.sh --host

Change-Id: I558949655a97b47ce3c560ed70310c9768e1da5c

Merge "ART: Fix alloc-dealloc mismatch"

ART: Fix alloc-dealloc mismatch

Mark type as array.

Bug: 31098551
Test: m test-art-host
Change-Id: Ia27a997e838a408cc68f0d9fe58734155e709e85

Add visiting for class loaders in StickyMarkSweep

StickyMarkSweep clear the mark stack,
Because all reachable objects must be referenced by a root or a dirty card.
But, there are some marking hole for class object.

If some object is marked and the object and it's class object is not dirty,
Then class object cannot be marking by card table.

In previous OS including mashmellow,
Class table was maintaned by class linker
and all class object was marked with kVisitRootFlagAllRoots flag.

In N OS,
Class object is not marked with kVisitRootFlagAllRoots.
So, I added new flag to mark class object and using it StickyMarkSweep.

Added regression test in 141-class-unload.

Test: test-art-host

Change-Id: I57599e6db53b260f4c5ef466b63962141b8da5c3
Signed-off-by: Hyangseok Chae <neo.chae@lge.com>

Merge "Fixed bug in LICM"

Merge "Make empty checkpoint work while weak ref access is disabled."

Fixed bug in LICM

Rationale:
We should stop hoisting anything that can throw
as soon as something else that can do something
visible (either throw or write something) is *not*
hoisted (used to be just throw test on second part).

Bug: 32810295
Test: test-art-host
Change-Id: Id88b712a5d9e37598d0bebbd4ecf4b1d8ee787b5

Merge "ART: Properly synchronize test 039"

Merge "ART: Expose vreg and visitor in RootInfo"

Merge "ART: Rewrite test 913 verification"

Merge "MethodHandles: Fix MethodType::IsConvertible() error."

MethodHandles: Fix MethodType::IsConvertible() error.

Conversions between non-numeric primitives and their boxed equivalents
were being rejected.

Test: m test-art-host-run-test-956-methodhandles
Bug: 30550796
Change-Id: I4ee255be3a4549246548185e362789561382ba1b

Revert "Revert "JIT root tables.""

Also  contains Revert "Support kJitTableAddress in x86/arm/arm64."

This reverts commit 4acd03638fcdb4e5d1666f8eec7eb3bf6d6be035.
This reverts commit 997d1217830c0a18b70faeabd53c04700a87d7d9.

Test: ART_USE_READ_BARRIER=true/false test-art-host test-art-target

Change-Id: I77cb1e9bf8f1b4c58b72d3cf5ca31ced2aaa1ea3

Merge "Dump c1visualizer output before and after each pass."

Merge "ARM: VIXL32: Implement InstanceOf, InvokeInterface and others."

Merge "MethodHandles: Enable return value conversions."

Merge "Fix mips build"

Fix mips build

Test: untested
Change-Id: Ied193834bf2a187b0cb8ffdbdece72023698b1f6

Merge "Use entrypoint switching to reduce code size of GcRoot read barrier"

ART: Rewrite test 913 verification

Make the test more robust wrt/ the liberty the specification
gives for iteration order. Also make it robust wrt/ JIT by
waiting for any compilation to finish before following
references.

This reverts commit a35c584d7eb0ae02c9511f5f891b284362b12e28.
This reverts commit 8c433b9e45a26e3bde123a5b2697ca801ae8b039.

Test: m ART_TEST_INTERPRETER=true ART_TEST_JIT=true test-art-host-run-test-913-heaps
Change-Id: Ia94c597f4f7b66ab1014efb483b3aed2e859099a

Use entrypoint switching to reduce code size of GcRoot read barrier

Set the read barrier mark register entrypoints to null when the GC
is not marking. The compiler uses this to avoid needing to load the
is_gc_marking boolean.

Code size results on ritzperf CC:
arm32: 13439400 -> 13242792 (-1.5%)
arm64: 16380544 -> 16208512 (-1.05%)

Implemented for arm32 and arm64. TODO: Consider implementing on x86.

Bug: 32638713
Bug: 29516974

Test: test-art-host + run ritzperf
Change-Id: I527ca5dc4cd43950ba43b872d0ac81e1eb5791eb

Merge "Use enum for read barrier options in compiler"

ART: Make test 030 slightly more robust

Run a serious of immediate GCs, synchronizer with the finalizer
starintg  and then let the main thread sleep for a minute, in an
effort to make the test deterministic and get the finalizer
daemon scheduled.

Test: m test-art-host-run-test-030-bad-finalizer
Change-Id: Id379d8147ecc8f6aa0a7d8d5058030a4c2d4de46

ART: Properly synchronize test 039

Properly synchronize the test so that the output is deterministic.

Test: m test-art-host-run-test-039-join-main
Change-Id: Ia2212c7f77c3bf016d588014b22fa5abaef076e3