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

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

Merge "ART: Add support for generic field signature"

Merge "ART: Fix Unstarted Runtime Class.forName"

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"

Merge "Add miscellaneous tests for class redefinition."

Merge "Use GetPeerFromOtherThread in debugger."

Use GetPeerFromOtherThread in debugger.

Test: run-jdwp-tests.sh --mode=host --variant=X64 --debug
Change-Id: I6b92153752becbad04563d96b824a06c002f8392

Testrunner: Handle case when test_info line exceeds console length

Case when invoked without verbose, testrunner will not print test_info
exceeding the width of the console. It will shrink down the test_name to
meet the requirement.

Test: Verified by locally running the test
Change-Id: Ibc8dff32e79818f7ac72322e5db01218343f3845

Merge "ART: Regenerate test data for 952."

Check for duplicate entries when using dexlayout

The previous approach always inserted in the CollectionMap. This did
not work correctly if two different objects had the same offset. Since
we constructed a unique_ptr of the object, it would simply get freed
if the insert failed because there was already an object with the same
offset.

This meant that there were some use after free errors for these
objects.

Test: dexlayout -a -b hulu.apk

Bug: 35305951
Change-Id: I217e6d83bf6e051b13b1a601ec25512f40593880

Merge "Testrunner: Accept test_name by run-test number"

Merge "Testrunner: Take default only when no other variant is selected"

Testrunner: Accept test_name by run-test number

The change allows the testrunner to accept test numbers for running
a particular test. It runs all the tests for which the prefix matches
with the passed test name.

For test name "001-H", it will run 001-HelloWorld
However, for test name "001", it will run 001-Main and 001-HelloWorld

If it doesn't find any test with matching prefix, it raises ValueError.

Test: Verified by running locally
Bug: 35260706
Change-Id: Ib132d6635b22ade1fd96728ba629fe7d580ba6cf

Clean up profman arg checking

Move argument consistency checking to the places the relevant verbs are
being processed.  This eliminates criss-crossing consistency checks that
are correct for one verb but not another.

Bug: none
Test: test-art-host
Change-Id: I520fc8c48675e44959c7de1967642a6ebe4100e4

Testrunner: Take default only when no other variant is selected

If a variant is selected either by setting the environment flag or
passing the variant as an arg to the python testrunner, the default
variant of that type will not be used unless it is explicitly set
by the above methods.

Test: Checked the behavior by running locally.
Change-Id: I6f480249408c0bf406b660e5098f298f21ff2db4

ART: Regenerate test data for 952.

Enable any internal users to run generator/build-test.sh.

Bug: 33191717,30550796
Test: art/test/run-test --host --gcstress 952
Change-Id: I2156c12cdf7484cfa296f53f4707e37102a1accb

Merge "ARM: Faster read barrier marking entrypoints."

Merge "Inline methods that throw."

Merge "Update verify-profile logic."

Merge "Use a synchronization barrier to fix flakes."

Remove use of npic image for gtests.

Test: art gtests on host and target
Bug: 33192586
Change-Id: Ic56e304eb539b4cf353281280a48ce23c74bf218

Update verify-profile logic.

To start measuring OTA time improvements correctly, return
verify-profile as non-profile filter of verify-profile.

bug:34715556
Test: device boots, interpret-only is not used
Change-Id: I098a23d6dc958530df4eb94209d07d735576e0eb

Inline methods that throw.

Forked from https://android-review.googlesource.com/214292

test: 637-checker-throw-inline
bug: 30933338
Change-Id: I184be82dfab0710af3f3796e9e486c7817fa9c60

Use a synchronization barrier to fix flakes.

test: test-art-host gcstress
Change-Id: I8ded9ec6efb48729bdfecaf9dc8daea164c6f519

Merge "Add a DCHECK in thread::GetPeer."

Merge "Change behavior of $noinline$ to now force not inlining."

Merge "Fix DEX zip open error message printing"

Merge "ART: Rewrite test 911"

Merge "ART: Change test 121-modifiers to use smali"

ART: Add currentThread cutout to Unstarted Runtime

Add currentThread() and getThreadState() cutouts to the unstarted
runtime to allow further compile-time initialization. The cutouts
are protected by call-stack checks.

Add tests.

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

Merge "Address some comments for aog/333864"

Merge "Revert "Revert "runtime: Mmap uncompressed dex files (in zip) as clean memory"""

Address some comments for aog/333864

Mostly comment changes.

Bug: 34621054

Test: test-art-host

Change-Id: I95f77651d47a8d0c71d580778d1126ab4aa1d751

ART: Rewrite test 911

Use explicit thread names to avoid output reliance on implicit
thread naming.

Bug: 31684812
Test: m test-art-host-run-test-911-get-stack-trace
Change-Id: I82004fea6f49831b5d378bef2c290b61c694a07a

Change behavior of $noinline$ to now force not inlining.

In order to start supporting more inlining, like inlining
methods that have a throwing branch, we need to change our
testing strategy.

test: test-art-host
Change-Id: I65354884898d86a18e78b96b7c21a8728c83f86d

Merge "ART: Add operator == and != with nullptr to Handle"

Merge "Fix test scripts to account for new linker behavior"

Revert "Revert "runtime: Mmap uncompressed dex files (in zip) as clean memory""

This reverts commit 960b2af8a05f0844e78004e2d0d3ae6ab058d430.

Fix failing 071-dex-file-map-clean on target
which was due to an IO race in the run-test script.

Test: make test-art-target
Bug: 27650033
Original-Change-Id: I18efbd392c5980ffe0d983833b6cc581e0237b92
Change-Id: I6f4ff1e85f8326916c2ae0842a32f53fb7901639

Remove unneeded build files

Also make run-test default to assuming java8 language source.

Test: mma -j40 test-art-host
Test: ./test/run-test --host --build-with-javac-dx 001-HelloWorld
Change-Id: Id10b0539d919802985a373b1720534735c32434b

Add miscellaneous tests for class redefinition.

There were a bunch of edge cases related to class redefinition and
obsolete methods that were never explicitly tested. This adds tests
for some of them.

Specifically we add the following tests:

1) Testing throwing exceptions that move through obsolete methods.
2) Testing Method.invoke of redefined methods.

Test: mma -j40 test-art-host

Change-Id: Ifa920e5d3b56206966acaa3ab2ffa901fd95fd5b

Merge "Update test 044 for java-8"

Merge "ART: Make dex2oat watchdog configurable"

ART: Add operator == and != with nullptr to Handle

Get it in line with ObjPtr and prettify our code.

Test: m
Change-Id: I1322e2a9bc7a85d7f2441034a19bf4d807b81a0e

Fix test scripts to account for new linker behavior

Remove /system/lib from the java.library.path to
avoid loading system libraries in classloader-namespace
by dependencies originated from libarttestd.so

Add additional libraries used by libarttestd.so to
the list of public libraries.

Bug: http://b/26833548
Test: m -j32 test-art-target
Change-Id: I8cfb1b1bec17ca32b2c691597a97b4e9d1667a2f

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