Merge "MethodHandles: Remove experimental flag."
am: 49a694b84b

Change-Id: I944ec7dec6b4924aa2122ba26f2e671a1afb792e

Merge "MethodHandles: Remove experimental flag."

MethodHandles: Remove experimental flag.

Runtime::IsMethodHandlesEnabled now always returns true.
This change also removes the (now unnecessary) run scripts
for method handle tests.

Note that the --experimental=method-handles flag is left in
place for the test build steps (etc/default-build) because it's still a
convenient way to supply arguments to jack / smali.

Test: make test-art-host

Change-Id: I8475c7b8ac2615ac382a06ce54be39dce3cc7713

Merge "Acquire the mutator lock before comparing classes/strings."
am: 0a384f5beb

Change-Id: I69a0d6cab86d22aa6a2181cb4c8db7a4041adc29

Merge "Acquire the mutator lock before comparing classes/strings."

Merge "Move some fields in art::Thread to reduce maintenance burden."
am: 041043021e

Change-Id: I74f2ea210589a882a04303cf8cee73b0445410be

Merge "Move some fields in art::Thread to reduce maintenance burden."

Acquire the mutator lock before comparing classes/strings.

Scratch my initial thought we woudldn't need it because the
handlescope is visited during the pause: as the compiler thread
is in state native, the GC can concurrently update the handlescope,
leading to false negatives when doing class/string equality.

bug:34240874
test: test-art-host gcstress
Change-Id: Icda0722fb49300a7de57e1c5d1efaa9e8dbda83f

Merge "Remove unused code for patching oat files."
am: 8bd59a0fd4

Change-Id: I431625fa1e278b37395db462ef7b5744fdb40f48

Move some fields in art::Thread to reduce maintenance burden.

Move fields `thread_local_start`, `thread_local_pos`,
`thread_local_end` and `thread_local_objects` before fields
`jni_entrypoints` and `quick_entrypoints` within
art::Thread, to avoid repetitive art::Thread field moves in
future CLs caused by the addition or deletion of entry
points.

Test: m test-art-host
test: m test-art-target (on ARM)
Change-Id: Ib67842e44a7f21a871ca4d1bb95dc6f7cfedc829

Merge "Remove unused code for patching oat files."

Merge "Put the resolved class in HLoadClass."
am: dcc7ab628c

Change-Id: Iabc1bd05ff2bab7977f5290e202f3d09050a80a5

Merge "MethodHandles: Implement invoker / exactInvoker."
am: 9748d3d209

Change-Id: I23b81dee0a15515bd43e4276e8d831967f560753

Merge "Handle verify-profile and bootclasspath classes in vdex."
am: e9e43e93a7

Change-Id: Iaee159194336648d2c72539e722dd2e737a96515

Merge "Put the resolved class in HLoadClass."

Merge "MethodHandles: Implement invoker / exactInvoker."

Merge "Handle verify-profile and bootclasspath classes in vdex."

Merge "Fix missing OOME check in reflection."
am: c1b02c5ea7

Change-Id: Ibc239be00a04dac3d10d1827c1462fb22029196b

Merge "Fix missing OOME check in reflection."

Remove unused code for patching oat files.

patchoat is now only used for patching images. It no longer needs to
support patching of non-PIC oat files. This change removes code from
patchoat that was specific to patching of non-PIC oat files.

Bug: 33192586
Test: m test-art-host
Test: device boots
Change-Id: I8918a708e441493a21f25d537ed0b74748bb9641

Merge "MethodHandles: Add tests for MethodHandles.Lookup.bind"
am: 64c1f53f93

Change-Id: Ifdee9bb486697af7ba87997d9b13aa97cb827a55

Merge "Put inlined ArtMethod pointer in stack maps."
am: e037a736be

Change-Id: I30cfb7e71cfd58827e1445f5fad296c6620230e0

Merge "MethodHandles: Add tests for MethodHandles.Lookup.bind"

Merge "Fix braino in RTP."
am: db54cc4285

Change-Id: I6e428e1e3a7c5792cc8d340775db1ebd521c5554

Merge "Put inlined ArtMethod pointer in stack maps."

Merge "Fix braino in RTP."

Put the resolved class in HLoadClass.

To avoid repeated lookups in sharpening/rtp/inlining.

Test: test-art-host test-art-target
Change-Id: I08d0da36a4bb061cdaa490ea2af3a3217a875bbe

Fix missing OOME check in reflection.

Test: 080-oom-throw
Change-Id: I5f18ea756e815978bf8682555470505959a8fad5

Put inlined ArtMethod pointer in stack maps.

Currently done for JIT. Can be extended for AOT and inlined boot
image methods.

Also refactor the lookup of a inlined method at runtime to not
rely on the dex cache, but look at the class loader tables.

bug: 30933338
test: test-art-host, test-art-target
Change-Id: I58bd4d763b82ab8ca3023742835ac388671d1794

Handle verify-profile and bootclasspath classes in vdex.

Two problems:
1) An apk might define a class twice, or define a class that
   is already in the bootclasspath, or define a class that
   in the future happens to be in the bootclasspath.
2) verify-profile does not make classes that were not verified
   as verify-at-runtime for vdex.

Fixes:
1) Check that the resolved class is part of the dex file that
   we are currently looking into. If not, don't update its
   verification status.
2) Make unverified classes as such when they are not in the profile.

bug:34108532
Test: 634-vdex-duplicate
Change-Id: I77c5e417c16c91af257b88b6456d07c0e4c2ca93

Merge "ART: Add Thread TLS support"
am: 0d478f289f

Change-Id: If75e5a108a224b64d501795ee00a5250a2edbfbb

Merge "ART: Add Thread TLS support"

MethodHandles: Implement invoker / exactInvoker.

Accompanies libcore commit 0a72753297b45d08ca33cf7e87888f5fee87f69e.

- Adds callsiteType field to mirror::EmulatedStackFrame.
- Adds unit tests for MethodHandles.invoker /
  MethodHandles.exactInvoker.

Test: art/test/run-test --host 957
Change-Id: Ia27742c80de560a5b38f792bfa97664678cec8ae

Fix braino in RTP.

We should use the `compare` instruction, and not the `check`
instruction.

Test: test-art-host, 633-checker-rtp-getclass
bug: 34273618
Change-Id: I388ec92974202da3a318921097a14462f3f3f8b6

ART: Add Thread TLS support

Add support for GetThreadLocalStorage and SetThreadLocalStorage.
Add test.

Bug: 31684593
Test: m test-art-host-run-test-924-threads
Change-Id: Ia2e567a832716b076a61664809ab29290fffcb70

Merge "ART: Add GetAllThreads"
am: 4ca818c31c

Change-Id: I2e104979d32ac059a178a765ea582c49a85b4f38

Merge "ART: Add GetAllThreads"

Merge changes I168f24de,I8ec60a98,I3c426ed7,Ic809b0f3
am: 5e01df491f

Change-Id: Ie48b865f4be84851153d4b67a055a164f65cdeaa

Implement HLoadClass/kBssEntry for boot image.
am: 1998cd0260

Change-Id: Iaacccb37f0c0b7272acb42f604446de9ddd39494

Store resolved types for AOT code in .bss.
am: 6bec91c7d4

Change-Id: Ibe8454204b13103131655f9de2fa266ea1b0d97f

Make runtime call on main for HLoadClass/kDexCacheViaMethod.
am: 4155998a2f

Change-Id: Ib41477d435236a47897a58531385840087fcef8e

Remove HLoadClass::LoadKind::kDexCachePcRelative.
am: 48886c2ee6

Change-Id: I72b52604f4ba786d857ab6f534356fd081220af2

Merge changes I168f24de,I8ec60a98,I3c426ed7,Ic809b0f3

* changes:
  Implement HLoadClass/kBssEntry for boot image.
  Store resolved types for AOT code in .bss.
  Make runtime call on main for HLoadClass/kDexCacheViaMethod.
  Remove HLoadClass::LoadKind::kDexCachePcRelative.

MethodHandles: Add tests for MethodHandles.Lookup.bind

Test: art/test/run-test --host 956
Change-Id: I76f1fb69e7213dfcfadcc64641fe2afce46ed6cb

Merge "ART: Fix artInvokePolymorphic."
am: d9a9d44ae0

Change-Id: Ia885f1127d4f9b52f4b5a10e7d33d9affd3a4091

Merge "MIPS64: Make GetFeatureString() returns default instead of empty string"
am: a11ad8a46d

Change-Id: I8054b6a949ffa0750274c97809d29461891669df

Merge "ART: Fix artInvokePolymorphic."

Merge "MIPS64: Make GetFeatureString() returns default instead of empty string"

ART: Fix artInvokePolymorphic.

artInvokePolymorphic called the wrong visit method on
RememberForGcArgumentVisitor. It should have called VisitArguments()
rather than Visit().

Bug: 30550796,33191393
Test: art/test/run-test 953
Test: m test-art-run-test
Change-Id: I46442203e8dc7000efd2cd64b2dcd5ed83163be6

Implement HLoadClass/kBssEntry for boot image.

Test: m test-art-host
Test: m test-art-host with CC
Test: m test-art-target on Nexus 9
Test: Nexus 9 boots.
Test: Build aosp_mips64-eng
Bug: 30627598
Change-Id: I168f24dedd5fb54a1e4215ecafb947ffb0dc3280

Merge "Revert "ART: Add ThreadGroup API support""
am: a48d74a59a

Change-Id: Ib6662233e71249b92a3b3bc730e09eaa01dd9cc2

Merge "Revert "ART: Add ThreadGroup API support""

Merge "Revert "Revert "ART: Compiler support for invoke-polymorphic."""
am: 58207cfd22

Change-Id: I298e0f0cb1522481666ef325df8abd0a46057b52

Store resolved types for AOT code in .bss.

Test: m test-art-host
Test: m test-art-target on Nexus 9.
Test: Nexus 9 boots.
Test: Build aosp_mips64-eng.
Bug: 30627598
Bug: 34193123
Change-Id: I8ec60a98eb488cb46ae3ea56341f5709dad4f623

Make runtime call on main for HLoadClass/kDexCacheViaMethod.

Remove dependency of the compiled code on types dex cache
array in preparation for changing to a hash-based array.

Test: m test-art-host
Test: m test-art-target on Nexus 9
Bug: 30627598
Change-Id: I3c426ed762c12eb9eb4bb61ea9a23a0659abf0a2

Remove HLoadClass::LoadKind::kDexCachePcRelative.

Test: m test-art-host
Test: m test-art-target-run-test-552-checker-sharpening
Bug: 30627598
Change-Id: Ic809b0f3a8ed0bd4dc7ab67aa64866f9cdff9bdb

Merge "Revert "Revert "ART: Compiler support for invoke-polymorphic."""

Revert "ART: Add ThreadGroup API support"

Test fails

Bug: 31455788

This reverts commit 02afcde4132070d925838bc09c0f24d1bd45273d.

Change-Id: I470becb29fda54db0db6f764a83efc926badb555

MIPS64: Make GetFeatureString() returns default instead of empty string

Since there isn't any feature left for MIPS64 (smp was recently
removed), GetFeatureString() returns an empty string. This causes
failures when running run-tests. dex2oat fails with following message:

Error parsing '--instruction-set-features=': No instruction set
features specified

Also change expectations in instruction_set_features_mips64_test.

Test: mma test-art-host-gtest
Test: mma test-art-target-gtest-instruction_set_features_mips64_test

Change-Id: Icc60a676a1b2fb10f89c888fa8a0d21fd0a105b5

Revert "Revert "ART: Compiler support for invoke-polymorphic.""

This reverts commit 0fb5af1c8287b1ec85c55c306a1c43820c38a337.

This takes us back to the original change and attempts to fix the
issues encountered:

- Adds transition record push/pop around artInvokePolymorphic.
- Changes X86/X64 relocations for MacSDK.
- Implements MIPS entrypoint for art_quick_invoke_polymorphic.
- Corrects size of returned reference in art_quick_invoke_polymorphic
  on ARM.

Bug: 30550796,33191393
Test: art/test/run-test 953
Test: m test-art-run-test

Change-Id: Ib6b93e00b37b9d4ab743a3470ab3d77fe857cda8

Merge "Revert "Try really hard for JIT in test 916""
am: 4cddd03950

Change-Id: Ie8c700b10c1de1ef51661b4e290123c1d3dbe837

Merge "Implement RedefineClasses, also redefine multiple classes atomically."
am: dd1dd32c87

Change-Id: I64f2b6b0bd144f59e71ebf6cdc668d266b894d3a

Merge "Revert "Try really hard for JIT in test 916""

Merge "Implement RedefineClasses, also redefine multiple classes atomically."

Merge "ART: Add GetFrameCount and GetFrameLocation"
am: cca87081f8

Change-Id: I135d73458e3f34c3632f8b2262d9633742eb8267

Merge "ART: Add GetFrameCount and GetFrameLocation"

ART: Add GetAllThreads

Add support for GetAllThreads. Add a test.

Bug: 31684593
Test: m test-art-host-run-test-924-threads
Change-Id: I7068dd4d3700a32a87a44d38590e53df0bd238c7

ART: Add GetFrameCount and GetFrameLocation

Add support for GetFrameCount and GetFrameLocation. Add tests.

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

Revert "Try really hard for JIT in test 916"

This reverts commit a64fa43345124eb7891b9d6a6258f6ed0cbd1504.

Reason for revert:  makes test-art-host-run-test-debug-prebuild-interpreter-relocate-trace-cms-checkjni-picimage-npictest-ndebuggable-916-obsolete-jit fail

Test: Treehugger
Change-Id: Idce6fc9310fe85ae62dd19da1e978299951c19ee

Implement RedefineClasses, also redefine multiple classes atomically.

We need to be able to redefine multiple classes atomically for JVMTI.
This implements that behavior. It also implements RedefineClasses
since until we have class transformation it is trivial.

Test: mma -j40 test-art-host

Change-Id: I80784f919a4366c465b93fede94f4bf763c0ee70

Merge "ART: Remove suspension from AttachAgent"
am: 2db7a1d7d6

Change-Id: I01a9d56ad14007582543a782f9d6379321b5a844

Merge "ART: Remove suspension from AttachAgent"

Merge "Make our jvmti run-tests work with the RI again."
am: 942ea7af2b

Change-Id: I623600325f6b67517887b1ba52e2902717759384

Merge "Make our jvmti run-tests work with the RI again."

Merge "ART: Add ThreadGroup API support"
am: 53dd074910

Change-Id: Ifce6fb458468e6b8fb832b414ff0802cd36b7731

Merge "Try really hard for JIT in test 916"
am: 8799ea0a82

Change-Id: I09818bede81a922fda0516b364ee943a6c77c25c

Merge "ART: Add ThreadGroup API support"

Merge "Try really hard for JIT in test 916"

Merge "Complete unrolling of loops with small body and trip count one."
am: 2f12c6581b

Change-Id: If41805ba3d8a0552ac84b3fa5ce040fd0d845853

Merge "Complete unrolling of loops with small body and trip count one."

ART: Remove suspension from AttachAgent

It conflicts with calls later. Every plugin needs to do local
protections for itself.

Follow-up to commit ad2dc875ce29c3fafc1129a2a25184df28b66a10.

Bug: 31682382
Bug: 34275017
Test: m test-art-host-run-test-909-attach-agent
Change-Id: Ib698d880feb774a00c819ef14c27a48501ca1f13

Complete unrolling of loops with small body and trip count one.

Rationale:
Avoids the unnecessary loop control overhead, suspend check,
and exposes more opportunities for constant folding in the
resulting loop body. Fully unrolls loop in execute() of
the Dhrystone benchmark (3% to 8% improvements).

Test: test-art-host

Change-Id: If30f38caea9e9f87a929df041dfb7ed1c227aba3

Merge "ART: Add GetThreadListStackTraces"
am: c00735af8a

Change-Id: I024b1de5490a1614e702a9449970d4252339c4fe

Merge "ART: Refactor test 911"
am: cd203212f6

Change-Id: I7dc79bde2c93655d2e18bf5c91819deeac9c80ec

Merge "ART: Add GetThreadListStackTraces"

Merge "ART: Refactor test 911"

ART: Add ThreadGroup API support

Add support for GetThreadGroupInfo, GetThreadGroupChildren and
GetTopThreadGroups. Add tests.

Bug: 31455788
Test: m test-art-host-run-test-925-threadgroups
Change-Id: I56809c95dfd2666c2e18769a8960d6b1604274b4

Try really hard for JIT in test 916

Sometimes we would fail to jit stuff due to high load. This makes us
try even harder to JIT so we won't fail in these scenarios.

Test: mma -j40 test-art-host
Test: stress --cpu 60 &; while ./test/run-test --host 916; do ; done

Change-Id: Ic944582bf021f119b8bc3f135af508ed8a8586c4

Merge "Make 129-ThreadGetId more robust."
am: a4fc28c3b9

Change-Id: I69993799d2dd9f25c876489b5a48478f30b71c4d

Merge "Make 129-ThreadGetId more robust."

Merge "ART tests: do not retrieve OatQuickMethodHeader for bridge methods"
am: 7e25123127

Change-Id: Id84f8b1a56f74192612a4293fb8b2dfb2db75ac4

Merge "ART tests: do not retrieve OatQuickMethodHeader for bridge methods"

Merge "MIPS32: Refined comment in java.lang.Math.abs(float/double) intrinsics"
am: 6f4b55bd48

Change-Id: I9553a126acad354457d304de2ac19b0f286259f2

Merge "MIPS32: Refined comment in java.lang.Math.abs(float/double) intrinsics"

Merge "Revert "Update ART run-tests for new Jack optimizations""
am: c0dae61ff5

Change-Id: I0e28a554ec35acc3c5e4ad82fc3d3cc2121e1be2

Merge "Revert "Update ART run-tests for new Jack optimizations""

Revert "Update ART run-tests for new Jack optimizations"

This reverts commit 8960b0ba78a70948810f0e178f51b0a6d201985d.

Bug: 34066768
Test: make -j32 test-art-host
Change-Id: Ifeb22d1e909d31ac6dd31bc0a66bab41ccb53425

Make 129-ThreadGetId more robust.

Make sure we get the heap task daemon, to avoid a race with the runtime
creating the stack trace, and then updating the peer.

test: 129-ThreadGetId
bug: 28261069
Change-Id: I739ab6cd0180e2be07b7cecac6ad8a905a7c9cd0

MIPS32: Refined comment in java.lang.Math.abs(float/double) intrinsics

Bug: 30758343
Test: ./run-test 631-checker-fp-abs on CI20

Change-Id: Id7270c40887a0b904af59d6034b6867033c7dba3

Merge "Update ART run-tests for new Jack optimizations"
am: 5c9f90c5ec

Change-Id: If5eaff50bfdd71f1fc538a9b8cc966839e62ef30