Merge "Remove boolean arguments from NewLibrary()"
am: bc1f469af0

Change-Id: I771aeb2010658b0215b955ef862ff495e548ca56

Merge "Remove boolean arguments from NewLibrary()"

Merge "Add exclusion for Reference.clear and reference processing"
am: 816f78479d

Change-Id: Ie3332a0c4191e798035e83964b82016bf2953873

Merge "Don't re-read referent in ReferenceProcessor::GetReferent"
am: c9da0f8bb0

Change-Id: I727e7d180c72a8c9547e0bddd505df5b2cffd6d5

Merge "Add exclusion for Reference.clear and reference processing"

Merge "Don't re-read referent in ReferenceProcessor::GetReferent"

Merge "Fix AssemblerMIPS64Test.LongBalc() running out of memory."
am: afca9306a6

Change-Id: I1268e080ff3f3d1fdbdd79d89c4d5730d09bf079

Merge "Fix AssemblerMIPS64Test.LongBalc() running out of memory."

Fix AssemblerMIPS64Test.LongBalc() running out of memory.

Pre-allocate the necessary buffer because the reallocation
strategy doesn't work well for a test of this size.

Test: m test-art-host-gtest-assembler_mips64_test
Change-Id: I16bc75f0ae83a62e5f3ed2cd8228dd40f0eb9524

Merge "ARM: Fix breaking changes from recent VIXL update."
am: b73e659be3

Change-Id: Id83c34cb94aa4785c070a35d30e1d4f45aa4d236

Merge "ARM: Fix breaking changes from recent VIXL update."

Merge "Implement VisitShouldDeoptimizeFlag for MIPS/MIPS64"
am: 3f92bdea52

Change-Id: Ia2f5cc7e7fed0fa78105befa7e781bafada6c7ca

Merge "Implement VisitShouldDeoptimizeFlag for MIPS/MIPS64"

Merge "MIPS64: Improve method invocation."
am: b70b66e701

Change-Id: Ie5cc97825258886bac942565e420f6d608433fce

Merge "MIPS64: Improve method invocation."

Merge "Support GVN for HLoadClass::LoadKind::kJitTableAddress."
am: 001e1c1202

Change-Id: I14e454cf6f5adbda61291af3d481f8c993743aff

Implement VisitShouldDeoptimizeFlag for MIPS/MIPS64

This is follow-up change for I18bf716a601b6413b46312e925a6ad9e4008efa4.

Test: mma ART_TEST_JIT=true test-art-target-run-test-jit on CI20 and QEMU

Change-Id: I750814ae740a4549f1a2af11be7ae4318ae26a2f

Merge "Support GVN for HLoadClass::LoadKind::kJitTableAddress."

Merge "Handle primitive types in VerifierTypes::AddAssignability."
am: dfc453b2ab

Change-Id: I087fd12b6215a92b5c8180c239303d2a71999137

Merge "Handle primitive types in VerifierTypes::AddAssignability."

Merge "ART: Make switch back-edge suspension point consistent"
am: a248587487

Change-Id: I8eb60834f9f1247860c97a740294020eabc27add

Merge "ART: Make switch back-edge suspension point consistent"

Add exclusion for Reference.clear and reference processing

Prevents race conditions like unclearing cleared references or
calling IsMarkedHeapReference on null references.

Bug: 33389022

Test: test-art-host

Change-Id: Iee83b76d84453e929172f1a83f284aa4910e126c

ART: Make switch back-edge suspension point consistent

Both compiled code and mterp attribute a back-edge suspend point
to the target. Make switch consistent by proactively updating
the frame's dex PC in that case.

Add another macro for reuse.

Bug: 31684812
Test: m test-art-host-run-test-911-get-stack-trace
Test: m ART_TEST_INTERPRETER=true ART_TEST_JIT=true ART_TEST_TRACE=true test-art-host
Change-Id: I9969cc4ffbc34cf1d8849938a1cd7e2c9aec462d

Support GVN for HLoadClass::LoadKind::kJitTableAddress.

Fixes performance regressions seen in eg Dhrystone.

Also add comment on why a class may not be found when sharpening.

Test: manual Dhrystone run, performance recovers
Test: ART_TEST_JIT=true test-art-host-run-test-jit
Change-Id: I8e879f1c390f83e8bc930f343beb7b4a41c2f190

MIPS64: Improve method invocation.

Improvements include:
- support for all kinds of method loads and static/direct calls
- 32-bit and 64-bit literals for the above and future work
- shorter instruction sequences for recursive static/direct calls
Also:
- include the MIPS64 dinsu instruction (missed earlier) and minor
  clean-up in the disassembler
- properly prefix constant names with 'k' in relative patcher tests

Test: test-art-host-gtest
Test: booted MIPS64 (with 2nd arch MIPS32R6) in QEMU
Test: "make -j1 ART_TEST_DEFAULT_COMPILER=false ART_TEST_OPTIMIZING=true
       ART_TEST_INTERPRETER=false ART_TEST_JIT=false
       ART_TEST_PIC_TEST=true test-art-target-run-test64"

Change-Id: I19876fa5316b68531af7dfddfce90d2068433116

Merge "ART: Clean up utils.h"
am: d812f78297

Change-Id: I5bbe18362ccd1b13413efe5d346b39436e739820

Merge "ART: Clean up utils.h"

Remove boolean arguments from NewLibrary()

NewLibrary is going to be used for header only libraries.  Instead
of adding more boolean arguments, replace the existing ones with
BuildOnlyStatic and BuildOnlyShared calls on the libraryDecorator
returned by NewLibrary.

Test: m -j, compare build.ninja
Change-Id: I832627938f3fbc8cc1565c695048a5b5c3553090

Don't re-read referent in ReferenceProcessor::GetReferent

Re-reading has the issue that it may read a null value after already
having done the null check. Using a cached value prevents this from
happening and causing DCHECK failures.

Added a related stress test.

Bug: 33569625
Bug: 33389022

Test: test-art-host

Change-Id: Ic42d540e035d41ac6e5b01762f9510cd6632b28c

Merge changes Iac3db17f,I4221ef81
am: 0955e34843

Change-Id: Ic30935bfd9aff2ac04fd42ac58932170d94266f5

ART: Fix stack trace location reporting
am: 2340e3f3aa

Change-Id: I481b7d5b2840a0c72507f61743f325a64572945a

ART: Rewrite test 911 stack trace reporting
am: ceafe35513

Change-Id: I8b81ba40224fc36b045bddb92c82a619ac62b74a

Merge changes Iac3db17f,I4221ef81

* changes:
  ART: Fix stack trace location reporting
  ART: Rewrite test 911 stack trace reporting

Merge "Address some review comments"
am: 017c55c120

Change-Id: Iafd709eacf9a986452179b5661c70b150b17e3f1

Merge "Address some review comments"

ART: Clean up utils.h

Remove functionality provided by libbase. Move some single-use
functions to their respective users.

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

Merge "Prune uses library classes even without profile"
am: 319261e13a

Change-Id: I372d44be97b4a94296ebf52d5f597478a48669a8

Merge "Prune uses library classes even without profile"

Address some review comments

Addressed comments in dex cache and class table. Added class table
test.

Test: mm test-art-host-gtest-class_table_test -j20

Change-Id: I3ec0282247187acb1ec7af25b309501f001a1c3e

Merge "Fix CC DCHECK failure in 152-gc-and-run-finalization."
am: aa2657d6d9

Change-Id: I71f78805750b4c695eeb829c56c7ff2b86fd67ad

Merge "Fix CC DCHECK failure in 152-gc-and-run-finalization."

Merge "Remove obsolete DeduplicateDexCacheAddressLiteral()."
am: ea63e4d1c3

Change-Id: Icdfcecbcfe20e24e04c220c308407efbf163cfee

Merge "Remove obsolete DeduplicateDexCacheAddressLiteral()."

Handle primitive types in VerifierTypes::AddAssignability.

The dex2dex compiler might indirectly call it with arrays
of primitives.

Also remove dead APUT_OBJECT optimization in VerifiedMethod.
It was added for optimizing Quick:
https://googleplex-android-review.googlesource.com/#/c/371193

Commit hash:
a9a8254c920ce8e22210abfc16c9842ce0aea28f

But Quick is gone.

Test: 630-safecast-array
bug: 33498750, 32546807, 32546608

Change-Id: Icfd3f8d915bfe225f05dbec42f9ba25dc1243fdc

Merge "ARM: VIXL32: Test both current and new assemblers with optimizing_cfi_test."
am: 831cb268e0

Change-Id: I5aa68f3529c0d4cc37eb050344bc38d77c56b506

Merge "ARM: VIXL32: Test both current and new assemblers with optimizing_cfi_test."

Remove obsolete DeduplicateDexCacheAddressLiteral().

Test: Rely on TreeHugger
Bug: 30627598
Change-Id: Ia3c7a1d528f62b730d7ac1cc7b67f21d9ff06c9e

Merge "Disable 626-const-class-linking for JIT."
am: 1e35a69a44

Change-Id: I4bb6ba663767d7a8fb61a4b7c3b55396f648f407

Merge "Disable 626-const-class-linking for JIT."

Merge "Fix string compression for embedded zero chars."
am: 3a0c53f93c

Change-Id: Iafa28e39957877e329365ed9e7fd991bdd3baee6

Merge "Fix string compression for embedded zero chars."

ARM: Fix breaking changes from recent VIXL update.

Test: m test-art-host
Test: m test-art-target
Change-Id: I5c0c38fcd935e97fc94721f645e51251c7a87d39

Disable 626-const-class-linking for JIT.

Test: m ART_TEST_JIT=true test-art-host; test skipped.
Bug: 33567581
Change-Id: Ib530a27b14d918bc8a24f85473c8705f023280b0

Fix string compression for embedded zero chars.

Treat embedded zero character as non-ASCII.

Test: m test-art-host
Test: m test-art-host with string compression enabled.
Bug: 31040547
Change-Id: Iea6e92f89d424953814953ba27f1c7a991589c65

Merge "Revert "Revert "Add kJitTableAddress for HLoadClass."""
am: fb678bfaad

Change-Id: I35eb743806ca582d729ddbe8a95719a4959d8599

Merge "Revert "Revert "Add kJitTableAddress for HLoadClass."""

ART: Fix stack trace location reporting

The frame info contains the location, not the line number. Add
location to output.

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

ART: Rewrite test 911 stack trace reporting

Return an array of arrays, which is easier to extend and doesn't
have the tight coupling of a collapsed array.

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

Merge "ART: Add missing space in log line"
am: 7c73bf3518

Change-Id: Ie69f9cd33fc6b1bb91db015558e1547b14b78d5f

Merge "ART: Add missing space in log line"

Revert "Revert "Add kJitTableAddress for HLoadClass.""

This reverts commit d2d5262c8370309e1f2a009f00aafc24f1cf00a0.

Change-Id: I6149d5c7d5df0b0fc5cb646a802a2eea8d01ac08

Prune uses library classes even without profile

The previous pruning relied on the classes being pruned from the
profile, and then using the profile to prune classes. If there was
no profile, the uses library classes were incorrectly left unpruned.

Leaving these classes unpruned caused aborts during compilation.

Bug: 30688277

Test: adb shell dex2oat --runtime-arg -classpath --runtime-arg /system/framework/com.google.android.maps.jar --dex-file=/data/app/comb.BBClient-1/base.apk --dex-location=/data/app/comb.BBClient-1/base.apk --oat-file=/data/app/comb.BBClient-1/oat/arm/base.odex  --app-image-file=/data/app/comb.BBClient-1/oat/arm/base.art --image-format=lz4 --compiler-filter=speed

Change-Id: I261b8894847b5b0a4f7330f49666e823a1b38bb0

ART: Add missing space in log line

Follow-up to commit c560fc0b430816825add4125134b20eb791f6036.

Test: m
Change-Id: Ie558b2984ec629706d2122eca9dd96b8c3497298

Merge changes Ieb86f678,I8e7280f0
am: 0ee6447c63

Change-Id: If0ddecf05d828c42a50b77916c4a350389269bcb

ART: Stack locals
am: f10dfcdb76

Change-Id: I1a3b0eb5f7a55602bc7606c2d7097f0f58271704

ART: Add precise root visiting
am: 585da955bc

Change-Id: I3dca109401326b4494de963d37c490f2185beb61

Merge "Disable test 964 with no-image"
am: d9fb4dd3f9

Change-Id: I398437a3be31d2913de3209c65d3c73885ec4ea2

Merge "ART: Report cputime in dex2oat"
am: 3c7b8f4c45

Change-Id: Ib1709151de04c20b452580a2e4c2a7ea36f9cb5f

Merge changes Ieb86f678,I8e7280f0

* changes:
  ART: Stack locals
  ART: Add precise root visiting

Merge "Disable test 964 with no-image"

Merge "ART: Report cputime in dex2oat"

ART: Stack locals

Add reporting of stack-locals roots. Use the new
precise root visiting to get dalvik register
information for compiled frames.

Bug: 31385354
Test: m test-art-host-run-test-913-heaps
Change-Id: Ieb86f67829e546692c30faa08eb44e8dcf2b2c6a

ART: Add precise root visiting

Add VisitRootFlags::kVisitRootFlagPrecise to signal a
request for precise RootInfo objects.

Move VisitRootFlags to gc_root.h. Refactor VisitRoot
functions to pass flags. Add code in Thread visiting
to decode vregs in compiled code.

Bug: 31385354
Test: m test-art-host
Change-Id: I8e7280f0ab682871f729f2a1c6b18670cf2dbf82

Merge "ART: Implement FollowReferences for initial object"
am: ce29d18139

Change-Id: I350196d99ee333320aeda6bced525dcac382bad9

ART: Report cputime in dex2oat

Add ProcessCpuNanoTime. Log cputime in dex2oat completion message.

Sample:

 dex2oat took 20.036s(64.843s cpu) (threads: 48) arena alloc=25MB (26760672B) java alloc=2MB (2311688B) native alloc=44MB (46792784B) free=35MB (37502896B)

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

Merge "ART: Implement FollowReferences for initial object"

Disable test 964 with no-image

It often times out due to the large number of classes involved in the
test. Since it doesn't test anything particularly interesting for the
no-image case we are disabling it to reduce flakiness.

Test: TEST_ART_RUN_TEST_NO_IMAGE=true mma -j40 test-art-host
Change-Id: I9e51481e52dd1eb332d7563795f490b0e97001a5

Merge "Use store release for resolved types"
am: 7d785fca4f

Change-Id: I33d3cc9daeedf6f63c5a2f2c97bf5abea39cce3b

Merge "Use store release for resolved types"

Merge "Revert "Add kJitTableAddress for HLoadClass.""
am: 4dd3f7d2e9

Change-Id: Ib55836b23c6cb5d61c09ef3d0b7dd45ee90915db

Merge "Revert "Add kJitTableAddress for HLoadClass.""

Revert "Add kJitTableAddress for HLoadClass."

One test failure after merge.

This reverts commit 5b12f7973636bfea29da3956a9baa7a6bbe2b666.

Change-Id: I120c49e53274471fc1c82a10d52e99c83f5f85cc

Merge "Add kJitTableAddress for HLoadClass."
am: 7d81bf2c9d

Change-Id: If90c5e91c2be184f5c521d881e2a060830750eb6

Merge "Add kJitTableAddress for HLoadClass."

ARM: VIXL32: Test both current and new assemblers with optimizing_cfi_test.

Test: m test-art-host

Change-Id: I71b97113d9bc3ad5abe5f5f89a0d94c243c8f2e2

Add kJitTableAddress for HLoadClass.

This new kind loads classes from the root table associated with
JIT compiled code.

Also remove kDexCacheAddress, which is replaced by kJitTableAddress.

test: ART_TEST_JIT=true test-art-host-jit test-art-target-jit
Change-Id: Ia23029688d1a60c178bf2ffa7463927c5d5de4d0

Merge "Revert "Revert "ARM: VIXL32: Use DontCare for SetFlags + fix for GenerateFrameEntry."""
am: c836b5a8a5

Change-Id: I99d7792e8107b29aea43355851f74f58ccf55d0d

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

Change-Id: I4eee4925831c3366e413b48e92c75c068a312b0a

Merge "Revert "Revert "ARM: VIXL32: Use DontCare for SetFlags + fix for GenerateFrameEntry."""

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

Merge "Use System.out only."
am: bf4e45476d

Change-Id: I511dd09e4b44374e7d4888944288de268b81c003

Merge "Use System.out only."

Use System.out only.

Otherwise streams can get interleaved when printing.

test: 956-methodhandles
Change-Id: Ie734e24ed5bb73786e1972df784b365176bfd593

ART: Implement FollowReferences for initial object

Add code for FollowReferences with initial object. Simply skip
root visiting and add the initial object to the work list.

Bug: 31385354
Test: m test-art-host-run-test-913-heaps
Change-Id: I7d2a852fac54b13219df975d131f5b95e8094d9b

Use store release for resolved types

For DefineClass, use a release store for SetResolvedType. This is
done to prevent other threads from seeing a class but not necessarily
seeing the loaded members like the static fields array.

It is theorized that a load acquire is not required since obtaining
the resolved class will always have an address depedency or a lock.

Bug: 32075261

Test: test-art-host

Change-Id: I8ab18edc2cc7c9eb4a30897903d5cf1c7f9eb24e

Merge "Added polynomial induction variables analysis. With tests."
am: 57096b7231

Change-Id: I279b7719950eaa64921f70f43568a51ea74c0328

Merge "Added polynomial induction variables analysis. With tests."

Added polynomial induction variables analysis. With tests.

Rationale:
Information on polynomial sequences is nice to further enhance
BCE and last-value assignment. In this case, this CL enables more
loop optimizations for benchpress' Sum (80 x speedup). Also
changed rem-based geometric induction to wrap-around induction.

Test: test-art-host

Change-Id: Ie4d2659edefb814edda2c971c1f70ba400c31111

Merge "Clean up Class::GetDirectInterface()."
am: 88e9168c5c

Change-Id: I112269a5b34bdb06f10390af9d5c9125a223aa98