Revert "Revert "ARM: VIXL32: Use VIXL backend by default.""

bug:35977033

This reverts commit 25275bef429dc6a48b79411e0d0b32207294523b.

Change-Id: I440bf8415e2bf550607595499701fb3e7c33b37e

Revert "ARM: VIXL32: Use VIXL backend by default."

Revert while investigating.

bug:35977033

This reverts commit e6316892821287b1d1906b9962eae129fbdc37be.

Change-Id: I51e24a6e539072a6d0d470dfe41855a4847f3e96

ARM: VIXL32: Use VIXL backend by default.

export ART_USE_OLD_ARM_BACKEND=true to use the previous backend.

Test: mma test-art-host && mma test-art-target

Change-Id: I4024a4ea15fa8ce1269c0837f6ea001b6c809df5

Merge "Emit the dex_pc if the line number is absent."

Merge "Remove testing of prebuilt npic boot images."

Merge "Add inline caches to offline profiles"

Merge "Added a few more integral SIMD extensions for x86/x86_64 (SSE)."

Merge "Revert^4 "Hash-based dex cache type array.""

Merge "Revert "run-test: Ignore OUT_DIR flag""

Merge "Revert "Temporary fix for build bots: Set verbose always true""

Revert^4 "Hash-based dex cache type array."

Added extra output to the abort message to collect more data
when we hit the crash. Added extra check when loading an app
image to verify that the class table isn't already broken.

Test: testrunner.py --host
Bug: 34839984
Bug: 30627598
Bug: 34659969

This reverts commit 5812e20ff7cbc8efa0b8d7486ada2f58840a6ad5.

Change-Id: I9bb442a184c236dcb75b3e42a095f39cd6bee59d

Merge "Use stderr to avoid mangled print statements."

Use stderr to avoid mangled print statements.

test: 909-attach-agent
Change-Id: I1037edca0cc07b0635057ac50ae5c6557e054d78

Merge "Use "false" instead of "/system/bin/false"."

Merge "Clean up tracking new roots, do not track them for CC."

Use "false" instead of "/system/bin/false".

The android root might be different than /system

test: test-art-target with ANDROID_ROOT=/data/local/tmp/system
Change-Id: I7b0a6b5444442661daaf5c53032d93263d388f99

Merge "Make dex2oat support profiles passed to non profile filters"

Revert "Temporary fix for build bots: Set verbose always true"

This reverts commit 8858eac1d5869d95b1234c9bb5e32f25c25418a4.

Reason for revert: It was a temporary fix to get the build bots(go/ab).

Change-Id: I27321b69cfcf6f39c01276639c2bb5711469ed5a

Revert "run-test: Ignore OUT_DIR flag"

This reverts commit ad9f7238268c0db4934b5f1bba9814bc002368a0.

Reason for revert: It was a temporary fix to get the build bots green.

Change-Id: I9a99fb50145e7b68e52c558520c703853ff57d7f

Make dex2oat support profiles passed to non profile filters

Using a non profile filter with an input profile should compile
everything but generate an app image and optimize layout based on
the profile.

Change UseProfileGuidedCompilation to use the profile arguments.
Using profile arguments instead of being based on the compiler
filter lets us do full speed compile and layout based on profile.

Fix ShouldCompileBasedOnProfile to use the compiler filter instead
of the existence of a profile.

Fixed gtests.

Test: test-art-host

Bug: 34927277

Change-Id: I325a10d2072ed427bb32f96e4efa54cf81e94ad3

Merge "Move thread_running_gc_ logic to VisitObjectsInternalRegionSpace"

Merge "Skip loop optimization if there is no loop in the graph."

Merge "Add nullptr check to CreateAnnotationSetItem."

Merge "ART: Add constructor support to unstarted runtime"

Move thread_running_gc_ logic to VisitObjectsInternalRegionSpace

Cleaner to have the logic in the caller.

Test: test-art-host CC
Change-Id: I93a16f8baf327b6692cce5c6141d1c361ce53f16

Merge "ART: Fix oatdump dex export"

Merge "Move InMemoryDexClassLoader to be a subtype of BaseDexClassLoader"

Skip loop optimization if there is no loop in the graph.

LinearizeGraph() does quite some allocations.
Also add some comments on the possible false positives of
some flags.

Test: m test-art-host
Change-Id: I80ef89a2dc031d601e7621d0b22060cd8c17fae3

Added a few more integral SIMD extensions for x86/x86_64 (SSE).

Rationale:
ART vectorizer needs a couple of extra SIMD operations before
sending out the larger general CL.

Test: assembler_x86[_64]_test
Bug: 34083438
Change-Id: Id156283424ad311e6109b360efcd409c671cd5b7

ART: Add constructor support to unstarted runtime

Add support for Constructor.newInstance0.

Bug: 34956610
Test: m
Test: m test-art-host-gtest-unstarted_runtime_test
Test: Device boots
Change-Id: Ifcea33359c30b70262cd2f4f56f59515b06532ce

ART: Fix oatdump dex export

Check for start index.

Bug: 35459241
Test: m
Test: m test-art-host-gtest-oatdump_test
Change-Id: I655d7be93427fd219ec445187c1d49076883bd3e

Merge "Use the alternate 32-bit base address for asan build only."

Merge "ART: fix method handle invocation of abstract & interface methods"

Move InMemoryDexClassLoader to be a subtype of BaseDexClassLoader

Also write a test that class transformation works with this type of
class loader.

Test: mma -j40 test-art-host

Change-Id: Ibc2214191e04876ff7bbea010be6ec03b6f41904

Merge "ART: Add support for generic method signature."

Clean up tracking new roots, do not track them for CC.

This addresses comments from
    https://android-review.googlesource.com/321552

Test: ART_USE_READ_BARRIER=false testrunner.py -b --host
Test: ART_USE_READ_BARRIER=true testrunner.py -b --host
Bug: 30627598
Change-Id: I9740e599fe8170201a3b5f10113bbeb0dee500fe

ART: fix method handle invocation of abstract & interface methods

Bug: 35422408
Test: m test-art-host
Change-Id: I1f60501a7d2e9d8c8b935709ae2d624af9f882ef

Merge "x86/string compression: Use TESTB instead of TESTL in String.charAt()."

Merge "Tests for String.setCharAt() breaking string compression."

x86/string compression: Use TESTB instead of TESTL in String.charAt().

And fix disassembly of the now unused TESTL.

Test: testrunner.py --host with string compression enabled.
Test: Manual inspection of dump-oat output.
Bug: 35433135
Bug: 31040547
Change-Id: I36c955bc1f2243954ecc315266a2f3fce5d87693

Tests for String.setCharAt() breaking string compression.

With string compression, all compressible strings must be
compressed. The internal API String.setCharAt() can break
that invariant when overwriting a non-ASCII character with
an ASCII character, turning an uncompressible string into
a compressible one. It can also truncate a non-ASCII
character written to a compressed string. These regression
tests check the public API that exposes the problem.
Submitting these tests (without a fix) shall prevent us
from enabling string compression before it's ready.

Test: testrunner.py --host -t 021-string2
Test: Manually check that new asserts fail with string compression.
Bug: 31040547
Change-Id: I66f27a73f273f7648acbdf1b601345711f37c85e

Remove testing of prebuilt npic boot images.

Because we always compile prebuilt images pic.

Test: test-art-host, test-art-target.
Bug: 33192586
Change-Id: I7cd3d3bb7b6cfbf79c2e442f3793a4967f37d84c

Merge "Remove passing tests from libcore_failures.txt."

Merge "Change IsMovingGCDisabled to take into account GC critical sections"

Change IsMovingGCDisabled to take into account GC critical sections

Fixes CC test 130.

Test: test-art-host

Change-Id: I39a0d6b88a7e651a5fb2320fd34c5a8bdc615345

Merge "ART: Add GetSignatureAnnotation to unstarted runtime"

Merge changes I4ae3eac3,Ibeddd7a5

* changes:
  run-test: Ignore OUT_DIR flag
  Testrunner: Release semphores/mutex when exception is caught.

run-test: Ignore OUT_DIR flag

This is a temporary fix to keep the build bots green.

Test: Run the test locally.
Change-Id: I4ae3eac3b3e460a094981f4229aa16e15c0f1750

Testrunner: Release semphores/mutex when exception is caught.

Threads should release semaphore and mutex even when the exceptions are
caught.

Test: Manually running the testrunner
Change-Id: Ibeddd7a5167752be5f056728edcf6e117d947ba1

Merge "Temporary fix for build bots: Set verbose always true"

Temporary fix for build bots: Set verbose always true

Test: Manual run
Change-Id: I7228c8f64a9a13de4ef3aa73e0e6cd4950ac9646

ART: Add GetSignatureAnnotation to unstarted runtime

Add support for Class.getSignatureAnnotation to the unstarted
runtime. Refactor dex file annotation code to support transactions.

Bug: 34956610
Test: m
Test: m test-art-host-gtest-unstarted_runtime_test
Test: Device boots
Change-Id: I9f4a93e05ad6999c46c3bb1dbabe48a1ad6225b2

Merge "Revert "Revert "Avoid adding region space bitmap to heap bitmap"""

ART: Add support for generic method signature.

Add support for generic_ptr to GetMethodName.

Bug: 34615460
Test: m test-art-host-run-test-910-methods
Change-Id: Ia48d3c09e49f33a6a8e51d17f3070ea4cd82744c

Merge "ART: Add support for generic field signature"

Merge "ART: Fix Unstarted Runtime Class.forName"

Add nullptr check to CreateAnnotationSetItem.

Unit test to follow.

Test: mm test-art-host
Bug: 35408418
Change-Id: I7157fe44b36babdec9a269e3726b393b0ff40dec

Add inline caches to offline profiles

Add support for inline caches in profiles:
- extract inline caches from the jit cache when the profile saver
queries the hot methods
- bump profile version to support the new data
- add new tests
- inline caches are only supported for same-apk calls (including
multidex)

Test: m art-test-host-gtest-profile_compilation_info_test
Bug: 32434870
Change-Id: I38b4ca0a54568d2224765ff76023baef1b8fd1a2

Use the alternate 32-bit base address for asan build only.

Following up
https://android-review.googlesource.com/#/c/331545/.

Bug: 34606909
Test: marlin-userdebug asan coverage build boots
Test: test-art-host

Change-Id: I3fbf64a822520992670e91cc857f7279bd288546

Merge "Revert "Enable String compression.""

Revert "Enable String compression."

The internal API String.setCharAt() is broken
with string compression.

Bug: 31040547

This reverts commit 24bbf98cec9f39a2592892adb245ce15d58cab19.

Change-Id: If2c2fa0eb42a8613994dfa5299febf7a5de2bf31

Merge "ART: Add support for generic class signature"

Merge "Testrunner: Return exit code 1 when test fails."

Testrunner: Return exit code 1 when test fails.

Test: manual
Change-Id: I08a3cd4464469303ce00b4830fefba33dcdd5389

Merge "Fix image dependencies for test-art-host-run-test-dependencies"

Fix image dependencies for test-art-host-run-test-dependencies

when ART_TEST_INTERPRETER and other flags are not set.

Test: manual
Change-Id: Ia49e41444aab068642d2ea63e98c970206be19e0

Revert "Revert "Avoid adding region space bitmap to heap bitmap""

The issue was that hprof dumping could run in the middle of non CC
GC. This meant the allocation stack and live bitmap could both visit
the same object. The solution is to use a GC critical section.

Bug: 34967844

Test: test-art-host

This reverts commit 215835daf496f44b94b37eb89dd659f84e3ae44d.

Change-Id: I99e65ff31ece74aa94fc55cc7480e27c1e39661b

ART: Fix Unstarted Runtime Class.forName

We really only support the boot classloader, and only in the null
denotation. Otherwise the class-linker would want to install a
class table into the fake BootClassLoader.

Add tests.

Bug: 34956610
Test: m test-art-host-gtest-unstarted_runtime_test
Change-Id: I47e284fbd17eb8d33665d2788afcbcc3d09d3d2e

ART: Add support for generic field signature

Add support for generic_ptr to GetFieldName.

Bug: 34615460
Test: m test-art-host-run-test-918-fields
Change-Id: If754461e1dbbfd775746d0883d0cb7e613b494e4

ART: Add support for generic class signature

Add support for generic_ptr to GetClassSignature.

Bug: 34615460
Test: m test-art-host-run-test-912-classes
Change-Id: Ie123879ec90b116d25c522035f8806f05253a0ec

Merge "Revert "Revert "Add test for changing annotations."""

Merge "Fix 624-checker-stringops for PIC."

Merge "Disable the dex2oat swap test for x86 altogether."

Fix 624-checker-stringops for PIC.

Do not rely on the const-string "x" to be non-throwing;
just pull the "x" out of the loop to make it irrelevant.
(The non-PIC test was "lucky" because "x" is a boot image
string and HLoadString/kBootImageAddress in non-throwing.
In PIC mode, the HLoadString/kBssEntry is throwing; the
"is in boot image" optimization for HLoadClass has not been
implemented for HLoadString.)

Test: testrunner.py --host -t 624
Test: testrunner.py --host --pictest -t 624
Change-Id: Iff5cfb1276af0e4896707f19a18e6053afd87a77

Merge "Enable String compression."

Remove passing tests from libcore_failures.txt.

Passing tests that are suppressed show up as warnings in vogar.  I ran
a script to look at the build logs for the buildbot and find the warnings
that are common to all recent runs, meaning the test passes in every
configuration we use, so it should be safe to enable.

Bug: 21722374
Test: looked at build logs of buildbots
Change-Id: Ied157778152f08e194ab8ffc8d7a86b133dccb99

Merge "Revert "Avoid adding region space bitmap to heap bitmap""

Revert "Avoid adding region space bitmap to heap bitmap"

This reverts commit fdb7d5d882a5f18326746cd92e83473be48231eb.

Test failures due to failing the added DCHECK.

Bug: 34967844
Change-Id: If4a6ec70330c7f773b550481c3dd2e7a2e5d4f9a

Disable the dex2oat swap test for x86 altogether.

On x86, this test has been seen failing on the CMS
configuration, not just the CC configuration.

Test: m ART_USE_READ_BARRIER=true test-art-host-gtest
Test: m ART_USE_READ_BARRIER=false test-art-host-gtest
Bug: 29259363
Change-Id: I3690aaec0aedd0020660aff42b408487e042cfd4

Merge "Remove use of npic image for gtests."

Merge "Avoid adding region space bitmap to heap bitmap"

Avoid adding region space bitmap to heap bitmap

Adding the bitmap here really served no purpose and caused
VisitObjects to visit region space bitmap objects twice. This had
unwanted results like adding objects to hprof dumps twice.

Regression test is covered by DCHECK in hprof and test 130.

Test: N6P boot and test-art-host

Bug: 34967844
Change-Id: I53a6075d577430089bb42f84a3c80c1cc5302206

Merge "Add invoke infos to stack maps"

Merge "Remove unneeded build files"

Revert "Revert "Add test for changing annotations.""

This reverts commit b2793dd8d9c517b899c0c63dd219dee8f6391c94.

Reason for revert: Sorted annotations before printing.

Test: mma -j40 test-art-host
Test: ART_TEST_FULL=true \
      ./test/testrunner/testrunner.py \
         -t test-art-host-run-test-948-change-annotations

Change-Id: Ie63e1a046baf3c69542d00dbad8ef5f6f29099de

Merge "Revert "Add test for changing annotations.""

Revert "Add test for changing annotations."

This reverts commit 701fc35b59d7d3eb33fb5d3cc90b14d29726ca66.

Reason for revert: annotation order is affected by configuration for some reason!

Change-Id: Ice1406ff41b704915f981d8d379ced9d0793a822
Test: revert

Merge "ART: Remove unintended debug logging"

Merge "Add jni_libraries_lock_ to expected_mutexes_on_weak_ref_access_."

Merge "Clean up profman arg checking"

Merge "cpp-define-generator: presubmit hook prints better error message"

Merge "Add test for changing annotations."

Add invoke infos to stack maps

Invoke info records the invoke type and dex method index for invokes
that may reach artQuickResolutionTrampoline. Having this information
recorded allows the runtime to avoid reading the dex code and pulling
in extra pages.

Code size increase for a large app:
93886360 -> 95811480 (2.05% increase)

1/2 of the code size increase is from making less stack maps deduped.
I suspect there is less deduping because of the invoke info method
index.

Merged disabled until we measure the RAM savings.

Test: test-art-host, N6P boots

Bug: 34109702

Change-Id: I6c5e4a60675a1d7c76dee0561a12909e4ab6d5d9

Add jni_libraries_lock_ to expected_mutexes_on_weak_ref_access_.

Fix a DCHECK failure where jni_libraries_lock_ is unexpectedly held
during a weak ref access.

Bug: 35360959
Test: test-art-host
Change-Id: Ief9e97c03cdff7f350e739f1004925a3689ecaf2

ART: Remove unintended debug logging

Test: m
Change-Id: Id6085215d8a9e2d22454894adc3c1e9923e54611

cpp-define-generator: presubmit hook prints better error message

Test: cd art && ../tools/repohooks/pre-upload.py
Bug: 35381554
Change-Id: Ia8536631c8c3117aeb0bf79862757717842b3a2b

Merge "ART: Add currentThread cutout to Unstarted Runtime"

Add test for changing annotations.

Also remove the LOG(WARNING) about not checking for annotations
changes since it doesn't seem the RI checks for this either.

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

Merge "Check for duplicate entries when using dexlayout"

Merge "Testrunner: Handle case when test_info line exceeds console length"