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

Temporary fix for build bots: Set verbose always true

Test: Manual run
Change-Id: I7228c8f64a9a13de4ef3aa73e0e6cd4950ac9646

Testrunner: Return exit code 1 when test fails.

Test: manual
Change-Id: I08a3cd4464469303ce00b4830fefba33dcdd5389

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

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

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

Merge "Revert^6 "Make class redefinition work with native methods on stack.""

Fix DEX zip open error message printing

We were printing the pointer to the error message string rather than the
string itself.

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

ARM: Faster read barrier marking entrypoints.

Avoid MVN for forwarding address check and another one
undoing the first when we hit a forwarding address.

Test: m test-art-target
Bug: 29516974
Change-Id: I9c0ca70a6fe98f0200df27bf7ed776aee039e64d

Revert^6 "Make class redefinition work with native methods on stack."

We were incorrectly trying to obtain the profiling information of a
native method.

This reverts commit 02b2349b1cd2a78e86b1a7542f8330e6c3aaeb35.

Reason for revert: Fixed test failure with jit configurations

Test: ART_TEST_JIT=true mma -j40 test-art-host

Change-Id: Ic7112104aec64e597d2df80db5dc2a505d2cc2dd

Merge "Revert "Enable dex2oat swap test for CC on x86.""

Revert "Enable dex2oat swap test for CC on x86."

Is't still failing and the difference can be
rather huge:
    Expected: (native_alloc_2_) < (native_alloc_1_),
    actual: 379712 vs 248600

Bug: 29259363

This reverts commit 38f5663a970b633f08e144b4dff33633c51352ee.

Change-Id: Ie1d62fbba688aeadffb9d5aafd5b9cf632742ed6

Merge "ART: invoke-custom support"

Merge "ARM64: Improve String.equals() intrinsic for const strings."

Add a DCHECK in thread::GetPeer.

Motivated  by https://android-review.googlesource.com/#/c/333205/.

Test: test-art-host run-jdwp-test.sh
Change-Id: I173c060324aa0dc39144db55e3a97e672c012ba8

Merge "Revert "Disable libcore tests for now.""

Revert "Disable libcore tests for now."

Vogar has been fixed.

bug:35308152

This reverts commit b1cde6a6d0101328e683fe07f81d40a1368eaae9.

Change-Id: I2440da0a97561f6794f8827b536481e1a6536c91

ART: invoke-custom support

Adds invoke-custom instruction to the interpreter.

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

ARM64: Improve String.equals() intrinsic for const strings.

And add additional tests to 021-string2.

aosp_angler-userdebug:
  before:
    arm64 boot*.oat: 43324664
    arm64 boot*.oat/string compression: 43411112
  after:
    arm64 boot*.oat: 43300136 (-24528)
    arm64 boot*.oat/string compression: 43345464 (-65648)

The string compression code size difference drops from
86448 to 45328.

Test: m test-art-target on Nexus 6P
Test: m test-art-target on Nexus 6P with string compression enabled.
Bug: 31040547
Change-Id: I99a3777b91b248da2b0ac25abd260f9e5abb2c09

Merge "Bump oat version."

Bump oat version.

Two recent changes should have bumped the oat version but
they did not:
    https://android-review.googlesource.com/336161
    https://android-review.googlesource.com/338409

Test: Rely on Treehugger.
Change-Id: I105f2f2d54953ed808ded92234c06efcc155d313

Merge changes I1042e87f,I58b17ecf

* changes:
  Remove remaining uses of DexFile::GetChecksum.
  Improve checking of multidex dex checksums.

Merge "Revert "Revert^4 "Make class redefinition work with native methods on stack."""

Revert "Revert^4 "Make class redefinition work with native methods on stack.""

ART run-test 945-obsolete-native is failing in automated
testing.

This reverts commit 724f77e2fed038d57a3d08fdcf656d703e3473ea.

Change-Id: I4eceb45765826ca442ee8f4800b2bed51f98e662

Remove remaining uses of DexFile::GetChecksum.

Remove or replace with DexFile::GetMultiDexChecksums as appropriate.

Bug: 34604632
Test: m test-art-host, added image_space_test to test ValidateOatFile.
Change-Id: I1042e87f29a242046d055f0be374ad9fbce4671a

Improve checking of multidex dex checksums.

* Fall back to odex file for all multidex entries if the apk is stripped,
  not just for the main multidex entry.
* Verify the number of multidex entries has not changed.
* Improve performance by getting all checksums from the apk in one go
  and cache the results instead of repeatedly opening the apk.
* Stop referring to non-main multidex entries as "secondary" dex files.

Bug: 34604632
Test: added tests to dex_file_test and oat_file_assistant_test
Test: m test-art-host

Change-Id: I58b17ecfbc9165a5bfeffd5281ee21d108f64479

Merge "ART: Add DEX support for MethodHandle and CallSite info."

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

Merge "ART: Correctly handle temporary classes in class-load events (4/3)"

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

Assert failing for "earchbox:search":
F zygote64: class_linker.cc:4612] Check failed: handle_scope_iface.Get() != nullptr

Test: m test-art-host
Bug: 34839984
Bug: 30627598
Bug: 34659969

This reverts commit 85c0f2ac03417f5125bc2ff1dab8109859c67d5c.

Change-Id: I39846c20295af5875b0f945be7035c73ded23135

Merge "Revert^4 "Make class redefinition work with native methods on stack.""

ART: Correctly handle temporary classes in class-load events (4/3)

When a temporary class is given out in a ClassLoad event, all stored
references need to be fixed up before publishing a ClassPrepare event.

This CL handles objects stored in the heap as referents.

Bug: 31684920
Test: m test-art-host-run-test-912-classes
Change-Id: If140ecae675cd7bc648f622eaf200f8ad8b15438