Merge "ART: Refactor JVMTI run tests"

Merge "Change thread suspend timeout to be fatal for non-debug"

Merge "Re-enable test 983 with speed-profile"

ART: Refactor JVMTI run tests

In preparation for code deduplication with CTS.

Bug: 32072923
Test: m
Change-Id: Ibbe005c94252bd29eae7f88aad301b0b20ddb80d

Re-enable test 983 with speed-profile

We make sure to fixup dex files in all situations before passing it to
JVMTI agents. This means that dexlayout won't break this test anymore.

Test: stress --cpu 60; \
      while ./test/run-test --host --random-profile 983; do; done

Change-Id: I047ffcbbba51db4c7bf5c22e482bd5864bc8b6d1

Merge "(Re)enable the use of inline caches in AOT"

Merge "Update profile version"

Merge "ART: Fix VISIT_OBJECTS and ABORT for FollowReferences"

Update profile version

I forgot to do it in cea9e9d26c32372ec8c75e6c89b1bfc08c5e1911. This will
force regeneration of possibly faulty profiles.

Test: m test-art-host
Bug: 36371709
Change-Id: I476c97005b6c17a22c57117c06028a2371276326

ART: Fix VISIT_OBJECTS and ABORT for FollowReferences

If a root report does not set VISIT_OBJECTS, the referree should
not be added to the worklist.

Whenever an ABORT is requested, do not report the remaining roots.

Bug: 36727422
Test: art/test.py --host -t 913
Change-Id: I942559f6ab356fe2573a3a9bb88b2662cd5b23d0

Merge "Dedup original dex file for classes transformed on first load."

Merge "Revert^2 "Improve user-friendliness of art script.""

Revert^2 "Improve user-friendliness of art script."

Fixes quoting for mksh for arguments with whitespace.

Test: art/tools/run-jdwp-tests.sh --mode=device
Change-Id: I0b8983a0c0ebcc1b64ad943859fbb85d3774ab75

Merge "Do more GCs for test 141"

Merge "Refactor image writer"

Merge "Update deopt statements in 638-checker-inline-caches"

Dedup original dex file for classes transformed on first load.

We set the originalDexFile field to a native pointer to the
art::DexFile (as a boxed Long). In the case where there were no
non-can_retransform_classes agents this DexFile will be owned by the
transformed classes ClassLoader and is guaranteed to be valid for as
long as the Class is. This means we can simply keep track of that
pointer to hold onto the original Dex file.

Test: ./test.py --host -j40
Bug: 31455788
Change-Id: I0e5a74f20f86d32dc73babe64b81469b3155c4dd

Do more GCs for test 141

Try to prevent rare race conditions that could cause the class loader
to not be unloaded.

Bug: 36377932
Test: test/testrunner/testrunner.py --host -t 141

Change-Id: I9429350b73c241478d474af37262576a512cc6a6

Merge "Downgrade another duplicate method DCHECK to WARNING"

Refactor image writer

Aim to have common functions for copying references and pointers.
Required for adding support for faster image fixups.

Test: test-art-host -j32

Bug: 34927277
Bug: 34928633
Change-Id: Ia654efc483b332eea3535570496bfeccd7c635ee

(Re)enable the use of inline caches in AOT

Test: m test-art-host (638-checker-inline-caches, 644-checker-deopt)
Bug: 36371709
Change-Id: Ic6e1b58845e7f7107be5a7041c47c44702af7b02

Update deopt statements in 638-checker-inline-caches

Test: m test-art-host-638-checker-inline-caches after inline
      caches have been activated
Bug: 36371709
Change-Id: I8378b0027a94bda89097fb0d3081e9184069643d

Merge "Undo dex2dex compilation before invoking LoadHook"

Merge "Layout string data"

Merge "Change Ti::Agent collection from vector to List"

Downgrade another duplicate method DCHECK to WARNING

This DCHECK got missed when writing change
'Ib26f30718726a9064056855192cef302f3df1eeb'. This is also caused by
duplicate methods in dex files causing some ArtMethod's to appear in
vtables more than once.

Bug: 32549051
Bug: 32546154
Bug: 36446088
Bug: 36399552

Test: ./test/testrunner/testrunner.py --host -j40
Change-Id: Id8ef07d7185958a91f008c2297db96c090dbecd7

Undo dex2dex compilation before invoking LoadHook

We need to undo any dex_to_dex compilation that might have taken place
before passing a dex file to any registered ClassFileLoadHooks to
ensure that no internal opcodes are present in any methods.

Test: ./test.py --host -j40
Bug: 36653594

Change-Id: Ia42c77312e685d69f6b3ea764fad01710b10ab45

Merge "Gracefully handle bogus profiling info for inline caches."

Merge "Fix profile merges in profman"

Layout string data

Inspect dex code in class initializers and hot methods to find out
what const strings are likely to be resolved. Along with these,
look at static field accesses and group the field name and type with
other hot strings.

Group method shorties together.

Maps vdex PSS: 7478k -> 6900k
No change in odex PSS.

Changed dexlayout_test to use -v instead of comparing to a hex
output.

Bug: 36457259
Bug: 35800981

Test: test-art-host

Change-Id: Iaa5de649c5b2b1d9f178525d2f42fe00e3fb879f

Change Ti::Agent collection from vector to List

We want these to be in a list to avoid the need for moving them
around, which would introduce the need to perform annoying accounting.

Test: ./test.py --host -j40
Bug: 36716572
Bug: 31455788
Change-Id: Ie16523f4e7d79934fa7844fb9b9a26e125cfa9eb

Merge "ART: Fix ObjectFree reporting"

Merge "ART: Fix GetThreadInfo"

Merge "Performance improvements in invoke-polymorphic runtime support"

Performance improvements in invoke-polymorphic runtime support

Fineses the runtime support for invoke-polymorphic to improve
MethodHandle benchmark measurements by an average of 14%.

- Faster conversion checks for non-exact invokes.
- Avoid conversion checks for CallerTransformers and Transforms.
- Fast path for exact invokes avoiding conversion checks.
- Use cheaper constructs where possible.

Most of these changes are driven by simulation in callgrind.

Bug: 30550796
Test: m test-art-host
Change-Id: I38722b0d41db8e528f2730a826b1710b9fd1f6a9

Merge "MIPS: Implement read barriers."

MIPS: Implement read barriers.

This is the core functionality. Further improvements
will be done separately.

This also adds/moves memory barriers where they belong and
removes the UnsafeGetLongVolatile and UnsafePutLongVolatile
MIPS32 intrinsics as they need to load/store a pair of
registers atomically, which is not supported directly by
the CPU.

Test: booted MIPS32R2 in QEMU
Test: test-art-target-run-test
Test: booted MIPS64 (with 2nd arch MIPS32R6) in QEMU
Test: "testrunner.py --target --optimizing -j1"
Test: same MIPS64 boot/test with ART_READ_BARRIER_TYPE=TABLELOOKUP
Test: "testrunner.py --target --optimizing --32 -j2" on CI20
Test: same CI20 test with ART_READ_BARRIER_TYPE=TABLELOOKUP

Change-Id: I0ff91525fefba3ec1cc019f50316478a888acced

ART: Fix ObjectFree reporting

Free events should not be dispatched to all registered envs. They
are specific to the env that the tag came from.

Add a DispatchEvent that takes an env, add an env field to the
tagging table, and connect the two.

Update the stress test. Make it actually work with a tagging
function for the second env.

Bug: 36648696
Test: ./test.py --host -r -t 905
Change-Id: I485ef1a6a57e233a2c2128b30cae93532676b3bf

Merge "Incorporate random profile tests into run-test."

ART: Fix GetThreadInfo

Fix the retrieval of the context classloader. Make sure to have
the field from the Thread class, don't look into a subclass.

Add caching for the field. Also fix the usage restriction of the
function, which is only valid during the Live phase.

Bug: 36654185
Test: ./test.py --host -r -t 924
Change-Id: I5cce41b31c32e59c80bb7c3afa03e8b0975ec54e

Fix profile merges in profman

profman used ProfileCompilationInfo::Load() which was not preserving the
correct order of the dex files (in a multidex profile).

The CL fixes profman to use ProfileCompilationInfo::MergeWith which
guarantees the right dex order and redesigns profile storage to avoid
such mistakes in the future. Instead of keeping data in a map indexed by
the profile key, store it in a vector whose index match profile_index.
This way, any iteration over profile info becomes deterministic with
respect to the profile index of the dex files.

Test: m test-art-host-gtest-profile_assistant_test
      m test-art-host-gtest-profile_compilation_info_test
      profile YouTube.apk and compile it based on the reference profile
(failing before)

Bug: 36371709
Change-Id: Ideda1336e5aff59a7c5560429da645fe02c804c9

Merge "Fix lock order for jvmti kTaggingLockLevel"

Gracefully handle bogus profiling info for inline caches.

Test: 643-checker-bogus-ic
Bug: 36371709
Change-Id: I2a52ac67a45a6bb5ddefb053a35e6edf5bc2da2f

Merge "Add --profile mode to the art script"

Fix lock order for jvmti kTaggingLockLevel

Add test for duplicate env free order.

Bug: 36648696

Test: ./test.py --host

Change-Id: I76fc8187b29f5e66cc29674320e887dbc508fe19

Incorporate random profile tests into run-test.

Uses profman to generate random profiles to pass into run-test.
Adds a new COMPILER_TYPE "speed-profile" to the build.

Bug: 36107940
Test: mm test-art-host
Change-Id: I00463f59ccbd9ffa4ddff097314374aede0d47c7

Merge "Add profman option to generate a random profile based on dex files."

Add --profile mode to the art script

--profile will run dalvikmvm twice. The first run records the profile by
replacing the compiler filter with interpret-only and enabling profile
saving. The second run will pass the recorded profile to dex2oat using
'-Xcompiler-option --profile-file=...'.

Test: ./out/host/linux-x86/bin/art --profile --64 -Ximage:$PWD/out/host/linux-x86/framework/core.art -Xusejit:true -Xcompiler-option --compiler-filter=speed -Xcompiler-option --compiler-backend=Optimizing -verbose:oat -Djava.library.path=$PWD/out/x86_64/host/linux-x86/lib64 -cp $PWD/generated/benchmarks.dex benchmarks.ExoPlayerBench.java.ExoPlayerBench

Bug: 36032648
Change-Id: I2dbcae43dd2972273511df9e01ebcbdfb1d84372

Merge "Add test to verify dex file passed to agent"

Add profman option to generate a random profile based on dex files.

Extends --generate-test-profile to make a randomized profile based on a
set of dex files. Will randomly incorporate classes and methods from all
the dex files given. Can be given a random seed for reproducibility.

Bug: 36107940
Test: mm test-art-host-gtest-profile_assistant_test
Change-Id: Ib0e54bde5317439516a39afa41c3c9980834fa54

Merge "Close the right fd in native_stack_dump"

Merge "Fix return value of AtomicMethodRefMap::Get()"

Merge "Fix arm64 and x86_64 code generator assertion"

Merge "MIPS64: Add vector registers to Mips64ManagedRegister class"

Add test to verify dex file passed to agent

Bug: 31455788
Test: ./test.py --host -j40
Change-Id: Ibd75e21d6d9f19d9bb125a43a0f54262724e894c

Merge "Math.round float intrinsic for ARM."

Math.round float intrinsic for ARM.

Test: m test-art-target
Test: m 082-inline-execute
Change-Id: I1b384e388b9959eca662922bffa8675cdbfe6e6b

Merge "Make data dependency around HDeoptimize correct."

Merge "Remove unreachable code in DexLocationToOdexFilename."

Merge "ART: Fix some tidy warnings"

Close the right fd in native_stack_dump

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

Fix return value of AtomicMethodRefMap::Get()

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

Fix arm64 and x86_64 code generator assertion

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

Merge "ART: Clean up field initialization"

Merge "ART: Remove unused DexToDex level."

Merge changes I45940d2d,I100b6246,Ie0ba58b3,I7943e7a6,I172cef38, ...

* changes:
  Remove signal mask checks from native bridge tests.
  sigchain: fix sigaction(signal, nullptr, &sa).
  sigchain: make SIG_DFL handling less broken.
  sigchain: allow use of sigprocmask inside handlers.
  sigchainlib: hopefully fix darwin build breakage.
  Revert^2: "Make sigchain actually own the signal chain.""

Merge "Testrunner: Fix output parsing issue"

ART: Fix some tidy warnings

Fix some tidy warnings for macros and references.

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

ART: Remove unused DexToDex level.

There is only "skip" and "optimize." We no longer have a required
tier, as constructor fencing is now elision-based.

Bug: 36653594
Test: m test-art-host
Change-Id: Ide874a581fc5f887d020b3d32507e3973e7fd9a4

Remove signal mask checks from native bridge tests.

Bug: http://b/36205469
Bug: http://b/36262089
Change-Id: I45940d2d189b6e144eb2f6ebb6d24fd70e3bd206

ART: Clean up field initialization

Add explicit field initialization to default value where necessary.
Also clean up interpreter intrinsics header.

Test: m
Change-Id: I7a850ac30dcccfb523a5569fb8400b9ac892c8e5

Merge "One more test to disable for CHA."

Merge "Ensure 'modified' field of ArtClassDefinition is always initialized."

One more test to disable for CHA.

bug:36344221
Change-Id: I03f6a3d9c6da9c3cd7e838a0c34a2cca0ab29f26

Merge "Testrunner: Temporarily fix long Dex File name issue"

Merge "Ensure jvmtiCapabilities structs are initialized."

Testrunner: Fix output parsing issue

Android build system determines failure info in the build log
by looking for a line starting with "FAILED: ".

The new format of the testrunner makes sure that the build system
is able to parse the error from the log.

Test: ./art/test.py -r -t 00
Change-Id: Iab29f254f600d4c3ee74cae2675da3a624e724ef

Ensure 'modified' field of ArtClassDefinition is always initialized.

This field was not always being initialized due to the use of a
default constructor. This meant we would go down unintended code paths
in some circumstances.

Test: ./test.py --host -j40
Bug: 36602266

Change-Id: Iba3521d4a2fa752a7ba59dcbcbbd30e0c09869ff

Ensure jvmtiCapabilities structs are initialized.

We were not zero-ing a handful of jvmtiCapabilities structs.

Bug: 31455788
Test: ./test.py --host -j40
Change-Id: Ia8c67a10e3b9068bb198392d48831b36290953bf

Merge "Check for unattached threads in JVMTI functions"

Remove unreachable code in DexLocationToOdexFilename.

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

Merge "Revert "Improve user-friendliness of art script.""

Revert "Improve user-friendliness of art script."

Breaks jdwp test on device.

This reverts commit 35710a024c487c9e12a9a3ad72f22b10056e1a82.

Change-Id: Ib693e8588eef297495c1d55e4f55cd2ec8ef2bf3

Make data dependency around HDeoptimize correct.

We use HDeoptimize in a few places, but when it comes to data
dependency we either:
- don't have any (BCE, CHA), in which case we should make sure no
  code that the deoptimzation guards moves before the HDeoptimize
- have one on the receiver (inline cache), in which case we can
  update the dominated users with the HDeoptimize to get the data
  dependency correct.

bug:35661819
bug:36371709
test: 644-checker-deopt
Change-Id: I4820c6710b06939e7f5a59606971693e995fb958

Merge "Initialize art_method_ in HGraph."

Initialize art_method_ in HGraph.

Spotted by Ivan Maidanski!

Benign as HGraph is allocated on the arena, and arenas
are always zero initialized.

test: test-art-host
Change-Id: Id8abe421e732dcf7a760f118b16b85fe1fac7c78

Merge "MIPS64: Improve method entry/exit code"

Merge ""Revert^6 "CHA for interface method.""""

Merge "Revert "Revert "Make original dex file be more deduplicated"""

Merge "Add 'dist' to gtest targets to avoid jack issues."

Add 'dist' to gtest targets to avoid jack issues.

Change-Id: Ibde8ba3e64817859b2bd61e575ea45057c96cd8e

Merge "ARM: Reduce the number of branches generated for HCondition and HSelect"

Merge "ARM: Improve the code generated for HInstanceOf"

Merge "Improve user-friendliness of art script."

"Revert^6 "CHA for interface method."""

Update test expectations for CHA tests flaking on no-dex2oat.

bug:36344221

This reverts commit 27ef25f084017421ca05508208f436b5fc11df73.

Change-Id: Ie92adc7a2ec3b3081a1c57d71f8c89247e58cd46

Merge "Improvements in the Inliner."

Merge "Makefile cleanup/fix after refactoring."

Makefile cleanup/fix after refactoring.

1) Put hprof-conf in ART_TEST_HOST_RUN_TEST_DEPENDENCIES
2) Remove duplicated code.

Change-Id: I11d141a13e94292f5a2bfefe1016be39aaece019

Merge "Fix issue with Android.run-test.mk"

Testrunner: Temporarily fix long Dex File name issue

Instead of passing long test names, it passes a directory with shorter name.

Bug: 36039166
Test: TMPDIR=/usr/local/google/home/shubhamajmera/aosp/tmp1 \
      TEMP=/usr/local/google/home/shubhamajmera/aosp/tmp1 \
      TMP=/usr/local/google/home/shubhamajmera/aosp/tmp1 \
      ./art/test.py \
      -t test-art-host-run-test-debug-prebuild-interpreter-relocate-npatchoat-ntrace-cms-checkjni-picimage-npictest-ndebuggable-201-built-in-except-detail-messages64 \
      -r

Change-Id: Id361e130301a23cd0c0d10a16cdd0dc48882c22b