Merge "Additional debug logging for bug 33231647."
am: 6afaa42f50

Change-Id: Ia828c033c0dc304ed95f27caf233a494fbfa92b4

Merge "Additional debug logging for bug 33231647."

Additional debug logging for bug 33231647.

Bug: 33231647
Test: m test-art-host
Change-Id: I5847ce92be5f588852e04ff44fe6eca19f363b93

Merge "Mterp/arm64: Add CFI directives."
am: 4e1a19b2e2

Change-Id: Ic153b79632b1a7304a220b9a248f1def324a9d16

Merge "Mterp/arm64: Add CFI directives."

Mterp/arm64: Add CFI directives.

Also add two bug 31975598 workarounds to arm64 entrypoints.

Test: m ART_TEST_INTERPRETER=true test-art-target
Bug: 31456348
Bug: 31975598
Change-Id: Ibf64160cf3b3f1ef644ff8f051ab7dc89643acf3

Merge "Make sure that const-class linkage is preserved, try again."
am: 9036914553

Change-Id: I28498fa06cfcb2a75ce7f87fc60d5ea0f63ef193

Merge "Make sure that const-class linkage is preserved, try again."

Make sure that const-class linkage is preserved, try again.

This CL causes occasional test failures on the build servers
which we were not able to reproduce locally. So we add some
some additional debug output to help pinpoint the cause.

Bug: 30627598
Bug: 33231647
Test: m test-art-host

This reverts commit 171cf811a1cdf8b1cbc5151505d8630741ce4cf3.

Change-Id: Id56a3f0e86e8212fd547e09c61794401bff47fb0

Merge "ARM: VIXL32: A couple of fixes to have more tests passing."
am: 982e0bf9ed

Change-Id: I57c2634b6508ced3b095aad2a60ca7db0fac9162

Merge "Remove the fast path in art_quick_resolve_string."
am: 85b38bd1fd

Change-Id: I16a4aadc2f34516580d018b0fef8fa5b12a100d5

Merge "ARM: VIXL32: A couple of fixes to have more tests passing."

Merge "Remove the fast path in art_quick_resolve_string."

Merge "Move compiled methods to use AtomicMethodRefMap"
am: 2ba0c377ec

Change-Id: Ifeb213ca7ca1e27fda641e0b0e537e3d3076a4ef

Merge "Move compiled methods to use AtomicMethodRefMap"

Merge "Optimizations around escape analysis. With tests."
am: 679ba8d5d5

Change-Id: I78b8d4d3da19997158f45d9f2fd75578de87842f

Merge "Optimizations around escape analysis. With tests."

Merge "Infrastructure for obsolete methods"
am: 0fd9a7d678

Change-Id: I1cf9630b5f5363334bc37e6f81b1dc733efa1c66

Merge "Infrastructure for obsolete methods"

Optimizations around escape analysis. With tests.

Details:
(1) added new intrinsics
(2) implemented optimizations
      more !can be null information
      more null check removals
      replace return-this uses with incoming parameter
      remove dead StringBuffer/Builder calls (with escape analysis)
(3) Fixed exposed bug in CanBeMoved()

Performance gain:
This improves CafeineString by about 360%
(removes null check from first loop, eliminates second loop completely)

Test: test-art-host

Change-Id: Iaf16a1b9cab6a7386f43d71c6b51dd59600e81c1

Move compiled methods to use AtomicMethodRefMap

Use AtomicMethodRefMap to avoid locking and map costs.

Native RAM usage for interpret-only compile FB:
71270192B -> 64117168B

Perf:
CompilerDriver::GetCompiledMethod: 0.88% -> 0.04%
Mutex::ExclusiveLock: 1.55% -> 1.05%

Bug: 32641252

Test: test-art-host-run-test

Change-Id: I146e7bf2f3328b4fa84f2cc19d591f4da82dccfc

Merge "Revert "Revert CC related changes.""
am: 07ab8831ff

Change-Id: Ibd61ac9cbce9269f084a60eed6b2f71a74c8320f

Merge "Revert "Revert CC related changes.""

Merge "Add release fence after copying object"
am: 7874c0c926

Change-Id: Ibf553d90eac400406cb32ba66f6e5be94fb6bf73

Merge "Add release fence after copying object"

Merge "Add atomic method reference map"
am: 392fb010a7

Change-Id: I29b36b6868d52fb197cfcde7804fd417251a3238

Merge "Add atomic method reference map"

Add atomic method reference map

Less RAM usage and faster than using a normal map with
MethodReference. Speed is faster by avoiding locking and tree
traversal. RAM usage is lower since the map usually had a value
for most method references.

Plan on using for marking methods for dex2dex, storing compiled
methods. Also use the new map for VerifiedMethods (refactoring).

Added test.

Bug: 32641252

Test: test-art-host-run-test

Change-Id: I46268031b8e0daf9be3597145cf6ecf579a039e2

Revert "Revert CC related changes."

Disable entrypoint switching in ResetQuickAllocEntryPointsForThread
instead of callers. Fixes bug where instrumentation would switch
to non CC entrypoints for non X86_64 architectures causing aborts.

Bug: 31018974

Test: test-art-host
Test: test/run-test 099

This reverts commit 96172e0172c5fca6e9a5ad4b857a24d8c7b064e5.

Change-Id: If206694ae35ff4446c6a8a97bfbcbf2dac35e3f9

Merge "ART: Add dex::StringIndex"
am: 211fd80290

Change-Id: Ic693797b08729ba2e400657a8106bec1f8c22190

Infrastructure for obsolete methods

This adds some of the structures and functions needed to eventually
support obsolete methods. Specifically this adds the code to create
and iterate through the obsolete dex-cache maps of classes and
re-organizes the redefinition code to be more sensible and extensible.

Bug: 32369913
Test: ./test/run-test --host 902
Test: mma -j40 test-art-host

Change-Id: I93d60fa66e7512e8b059cdf42af8a56e93ef4bd7

Merge "ART: Add dex::StringIndex"

Add release fence after copying object

Try to ensure that the field CAS in ConcurrentCopying::Process will
never reorder before the object copy.

May fix a bug where null classes are seen.

Test: test-art-host CC

Bug: 33210571
Change-Id: I71ab937b7ca60c88c5f69698731edac6a8f2e91a

ARM: VIXL32: A couple of fixes to have more tests passing.

Fix 103-string-append, CheckCast visitor, stop interpretting
boot.oat, port Floating point compare optimization.

Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-host
Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-target
Change-Id: I7e06f9bc1ec60a850a8fc4ef7a344b9cfc736033

Merge "Do not call IsMarked on null."
am: 304f2d3a1d

Change-Id: Ic4e6c1fe8f924f82c10c344babdda4330188b81b

Merge "ART: Remove duplicate StackVisitor method."
am: 846ddc7b9f

Change-Id: I7c568e581c87573922bbf30151aa4db1eb7428b1

Merge "Do not call IsMarked on null."

Merge "ART: Remove duplicate StackVisitor method."

Do not call IsMarked on null.

The concurrent collector checks it.

Test: ART_TEST_JIT=true test-art-host-run-test-jit
Change-Id: I02c2462ce1d8f3092f0b2f52f8b6dede8ff4a339

Merge "ARM: VIXL32: Implement HClassTableGet Visitor."
am: 725466c020

Change-Id: I15c8535549261b379e331493c44a0c1e06eb72b3

Merge "ARM: VIXL32: Implement HClassTableGet Visitor."

Merge "ARM: VIXL32: Fix some test failures."
am: 07e6a7947f

Change-Id: I58b8126bcb7fa128edc4a726f955157d4c6c6036

Merge "ARM: VIXL32: Fix some test failures."

Merge "Don't call realpath on a file that may not exist."
am: 73209a4a64

Change-Id: I3eb3bc9e876f3efdafd738da2d43fa9ad5e8d0be

Merge "Don't call realpath on a file that may not exist."

Merge "Move inline caches GC handling in JitCodeCache."
am: c9e2fc27c5

Change-Id: I9c05d759e5b14bd6ec26870f3fff7ecd4817bedf

Merge "Move inline caches GC handling in JitCodeCache."

Merge "Revert CC related changes."
am: 9ebca690fd

Change-Id: I56445fbe00521ea7c47432d1b56f0955de804e41

Merge "Revert CC related changes."

Don't call realpath on a file that may not exist.

Otherwise realpath prints an error message.

bug: 25658944
Change-Id: I63145c3d955e088e53708d8c6add945e6eab6654

Revert CC related changes.

Revert: "X86_64: Add allocation entrypoint switching for CC is_marking"
Revert: "Fix mips build in InitEntryPoints"
Revert: "Fix mac build in ResetQuickAllocEntryPoints"

Test: test-art-target-run-test
Change-Id: If38d44edf8c5def5c4d8c9419e4af0cd8d3be724

Merge "Don't use ashmen when kIsTargetLinux is set."
am: 626b839f88

Change-Id: I8ccc7b148e49ef51e5424d70749d05a21d90ec57

Merge "Don't use ashmen when kIsTargetLinux is set."

Merge "Improve invalid object logging"
am: f5ba1152d7

Change-Id: Icb2effb6f24f918b1f4b3b7139a18fc05e5133b3

Merge "Improve invalid object logging"

ART: Remove duplicate StackVisitor method.

Remove method that is the same as the older IsShadowFrame and only
used in tests.

Test: m ART_TEST_JIT=true ART_TEST_INTERPRETER=true test-art-host-run-test-454-get-vreg
Test: m ART_TEST_JIT=true ART_TEST_INTERPRETER=true test-art-host-run-test-457-regs
Test: m ART_TEST_JIT=true ART_TEST_INTERPRETER=true test-art-host-run-test-570-checker-osr
Change-Id: Id222d25d5345790b6c500ec8120880a3b9d79986

Merge "Added support for android root in dexfuzz."
am: 10096becf6

Change-Id: I4e27363162486aa63b18b74aca8efe73fb14a52c

Merge "Added support for android root in dexfuzz."

Merge "Fix mips build in InitEntryPoints"
am: d7a7c7f3e9

Change-Id: I9de67848af40584fca3af778a0ae92d263b8e191

Merge "Fix mips build in InitEntryPoints"

Fix mips build in InitEntryPoints

Added missing arg.

Test: Untested
Change-Id: I7bd25203eee92d0424f03ee9fdb9e7079d05d23c

Don't use ashmen when kIsTargetLinux is set.

Test: get JIT enabled when running ART.
Change-Id: Ic1c351601d8e85ba6597c3b400fa0094e070a56a

Merge "Fix mac build in ResetQuickAllocEntryPoints"
am: f9c2f88d13

Change-Id: Ieec9eed95858167e263fd2f045385f951e3d9ff9

Merge "Fix mac build in ResetQuickAllocEntryPoints"

Fix mac build in ResetQuickAllocEntryPoints

Unused arg warning.

Test: None

Change-Id: I5b147b1a584b96ad5457c9fee36f3edb0498cecf

Merge "ARM: Fix JNI MacroAssembler after VIXL update."
am: ae7f6b965e

Change-Id: Ia433a1b5eda0b825aecc992103823a44ca9a713c

Merge "ARM: Fix JNI MacroAssembler after VIXL update."

Merge "X86_64: Add allocation entrypoint switching for CC is_marking"
am: ab191538a1

Change-Id: I568ce012c3e585fade7bb919182cdaa4ae2be086

Merge "X86_64: Add allocation entrypoint switching for CC is_marking"

Merge "Revert "Revert "Revert "Revert "Revert "Make sure that const-class linkage is preserved.""""""
am: 8fb28dcf0e

Change-Id: Icf47b0bb6c44331d892239069ffee1988e0731f5

Merge "Revert "Revert "Revert "Revert "Revert "Make sure that const-class linkage is preserved.""""""

Revert "Revert "Revert "Revert "Revert "Make sure that const-class linkage is preserved."""""

Bug: 30627598

This reverts commit e47172b75b8b04a450d235cf72bdc4e223a29cb5.

Change-Id: I038029c71195a80f041982e82ae556d3157db7b8

Move inline caches GC handling in JitCodeCache.

Make the classes they hold weak references and visit
them during SweepJitRoots.

This fixes the newly introduced deadlock:

Thread1:
1) Lock JitCodeCache lock to create Profiling info for
  ArtMethod m.
2) m is a copied method, we need to track the actual holder,
   needing to decode a weak reference.
3) Weak references are not accessible due to GC.

GC Thread:
- Disallow weak reference access.
- Wait for checkpoint.

Thread2:
- Try to lock JitCodeCache lock
- Deadlock, as Thread1 owns the JitCodeCache lock.

Test: test-art-host
bug: 31289185
bug: 33198826

Change-Id: I7ee17631015450ace8d2a0264415a81c5a902bb8

Improve invalid object logging

Prioritize holder logging instead of maps and stack traces.

Test: test-art-host

Bug: 31441673

Change-Id: Ibc0523ffe5a8f8ba207c2643eae65d44599dcc86

ART: Add dex::StringIndex

Add abstraction for uint32_t string index.

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

ARM: Fix JNI MacroAssembler after VIXL update.

This patch fixes an issue found here:
https://android-review.googlesource.com/#/c/307532/

Also re-enables vixl MacroAssembler for arm.

Test: export ART_USE_READ_BARRIER=true; export ART_HEAP_POISONING=true; \
      m test-art-host
Test: export ART_USE_READ_BARRIER=true; export ART_HEAP_POISONING=true; \
      m test-art-target
Test: dump-oat-boot with ART_USE_READ_BARRIER=true

Change-Id: I03e3014fb250791db2d24a0ab1a76ca96384c332

Merge "Use non vixl arm macro assembler"
am: 8662aaaae4

Change-Id: Ib11edf85220a6d09f0875b5854fd276328bb836d

Merge "Use non vixl arm macro assembler"

Use non vixl arm macro assembler

The VIXL one hits an assertion error for CC:
external/vixl/src/aarch32/macro-assembler-aarch32.h:1386:
void vixl::aarch32::MacroAssembler::Cmp(vixl::aarch32::Condition,
vixl::aarch32::Register, const vixl::aarch32::Operand &):
Assertion `!AliasesAvailableScratchRegister(rn)' failed.

Fixes dump-oat-boot with CC.

Test: test-art-target, N6P booting

Change-Id: I781fe8c8c2d6cd8b246ee4509281d6c645c89a28

Merge "Disabled fallback to interpreter in dexfuzz."
am: 52f52361c2

Change-Id: I9e095f9ef08fdb434776f7ee2abddeb242f2327b

Merge "Disabled fallback to interpreter in dexfuzz."

Merge "ARM64: Merge TBNZ+CMP+BGE/BGT to CMP+BHS/BHI in intrinsics."
am: ec428a7ab9

Change-Id: If52bab0008327f409ff5321d35ee67f95fe6911f

Merge "ARM64: Merge TBNZ+CMP+BGE/BGT to CMP+BHS/BHI in intrinsics."

Merge "Fix event reporting from the debugger thread"
am: 60438b4609

Change-Id: Ia6187cdfb3b2fb459093cd1b8f2aff8c8c9b4adb

Merge "Revert "Revert "Revert "Revert "Make sure that const-class linkage is preserved."""""
am: df743754f4

Change-Id: Ib2f48cb30bfc559563ddefba7c86f0e29c252f6d

Merge "Fix event reporting from the debugger thread"

Merge "Revert "Revert "Revert "Revert "Make sure that const-class linkage is preserved."""""

Merge "Fix the target Linux builds of ART"
am: ded461e9df

Change-Id: I16a851ff3892b8842fd918ff7cbf38a2c7828185

Merge "Fix the target Linux builds of ART"

ARM64: Merge TBNZ+CMP+BGE/BGT to CMP+BHS/BHI in intrinsics.

Test: m test-art-target
Change-Id: I7316dfc40555a3c47670a3b79b4c8fc0cd3edc4f

Disabled fallback to interpreter in dexfuzz.

Added the -Xno-dex-file-fallback option when running optimizing backends so
that it fails in case compilation fails.

Test: Introduce an artificial crash in the optimizing compiler. There should
be a divergence reported.
dexfuzz --input=Hello.dex --execute --arm --interpreter --optimizing --repeat=20 --dump-output

Change-Id: I8adfc68b176b7482efce0cc7335c3cc50857c5a2

Revert "Revert "Revert "Revert "Make sure that const-class linkage is preserved.""""

Avoid class loading triggered by verification in the
626-const-class-linking test. Expect spurious wakeups.

Add extra debugging output for a very strange failure
    image_writer.cc:1144] Check failed:
    !IsBootClassLoaderClass(as_klass)
    java.lang.Class<java.lang.NoSuchMethodError>
That class is supposed to be in the boot image (core.art for
tests), so the failing check should not be reached at all.

Test: m test-art-host
Test: m ART_TEST_OPTIMIZING=true \
        ART_TEST_JIT=true \
        ART_TEST_INTERPRETER=true \
        ART_TEST_INTERPRETER_ACCESS_CHECKS=true \
        test-art-host-run-test-626-const-class-linking
Bug: 30627598

This reverts commit 09ded9201fbeec605b12741ae255663c14bda225.

Change-Id: I65a68cb53f0707b8abb3555ac3dc44c82907aef5

Merge "Removed quick support from dexfuzz."
am: d34d4f303b

Change-Id: I4227804a789a36a0dd03d2527118c4ed5cfe2347

Merge "Removed quick support from dexfuzz."

Merge "Re-enable run-test 484-checker-register-hints with read barriers."
am: b981c5ebba

Change-Id: I8dab7d738677866cd4c636dd92ed7b57ebc25a1e

Merge "Re-enable run-test 484-checker-register-hints with read barriers."

Added support for android root in dexfuzz.

Test: Execute dexfuzz with option --android-root=folder
All tests should be VerifyFail since folder does not exist on device.

Change-Id: I3f1d75386475661f09bc2a4d2c8ae3e94a202526

Fix event reporting from the debugger thread

The debugger thread may trigger events (like CLASS PREPARE after the
initalization of a class while processing a command).

This CL removes the incorrect CHECK that makes the runtime abort in
this case. However, we do check that only the debugger thread can
report an event while it is already processing a command.

Bug: 33032664
Test: art/tools/run-jdwp-tests.sh '--mode=host' '--variant=X64'
Change-Id: I32a284cfb4df6eb44eac27bbdcabd3c426d726ce

Merge "Use junit-host instead of junit"
am: c23d5e4783

Change-Id: Ibb28ddbd1e64ac7d94bfc8e7f88bedecbc1a18a5

Merge "Use junit-host instead of junit"