Merge "Implemented first version of java fuzz testing script."
am: ca1d99f530

Change-Id: I2b364a160ff1536060bb0c1b161219a052e0769a

Merge "Implemented first version of java fuzz testing script."

Implemented first version of java fuzz testing script.

Test: run_java_fuzz_test.py

BUG=30610121

Change-Id: I2a802476bcb0986e2891748ad85f8feac21656a8

Merge "ART: Implement a fixed size string dex cache"
am: 9c07ab332b

Change-Id: I4911322c2d0294a78f0e86abf07ecc4dfb734c9f

Merge "ART: Implement a fixed size string dex cache"

Merge "Add support for Baker read barriers in SystemArrayCopy intrinsics."
am: 6670bd2098

Change-Id: Ic796cba890a32feeb9ad7d7fbb661b9880252cb4

Merge "Add support for Baker read barriers in SystemArrayCopy intrinsics."

Add support for Baker read barriers in SystemArrayCopy intrinsics.

Benchmarks (ARM64) score variations on Nexus 5X with CPU
cores clamped at 960000 Hz (aosp_bullhead-userdebug build):
- Ritzperf - average (lower is better):       -3.03% (slightly better)
- CaffeineMark - average (higher is better):  +1.26% (slightly better)
- DeltaBlue (lower is better):               -10.50% (better)
- Richards - average (lower is better):       -3.36% (slightly better)
- SciMark2 - average (higher is better):      +0.26% (virtually unchanged)

Details about Ritzperf benchmarks with meaningful variations
(lower is better):
- FormulaEvaluationActions.EvaluateAndApplyChanges: -13.26% (better)
- FormulaEvaluationActions.EvaluateCascadingSums:   -10.94% (better)
- FormulaEvaluationActions.EvaluateComplexFormulas: -15.50% (better)
- FormulaEvaluationActions.EvaluateFibonacci:       -10.41% (better)
- FormulaEvaluationActions.EvaluateLargeSums:        +6.02% (worse)

Boot image code size variation on Nexus 5X
(aosp_bullhead-userdebug build):
- total ARM64 framework Oat files size change:
  107047632 bytes -> 107154128 bytes (+0.10%)
- total ARM framework Oat files size change:
  90932028 bytes -> 91009852 bytes (+0.09%)

Test: ART host and target (ARM, ARM64) tests + Nexus 5X boot.
Bug: 29516905
Bug: 29506760
Bug: 12687968
Change-Id: I85431368d09965687a0301ae2eb3c991f276ce5d

Merge "Improve the thread flip."
am: f606c3a687

Change-Id: I93770b75a2d9e3ae3f3d53f37985d031425efe98

Merge "Improve the thread flip."

Merge "Color spill slots in gc regalloc"
am: 96954301ee

Change-Id: Ibeafcda9441236cddcc2c1ce239ee5c260848a13

Merge "Color spill slots in gc regalloc"

Merge "Log optimization passes in verbose mode"
am: 35938f90ef

Change-Id: I71332e121fe7a2deca1932b76281ca8a79427141

Merge "Log optimization passes in verbose mode"

ART: Implement a fixed size string dex cache

Previously, the string dex cache was dex_file->NumStringIds() size, and
@ruhler found that only ~1% of that cache was ever getting filled. Since
many of these string dex caches were previously 100,000+ indices in
length, we're wasting a few hundred KB per app by storing null pointers.
The intent of this project was to reduce the space the string dex cache
is using, while not regressing on time that much. This is the first of a
few CLs, which implements the new fixed size array and disables the
compiled code so it always goes slow path. In four other CLs, I
implemented a "medium path" that regresses from the previous "fast path"
only a bit in assembly in the entrypoints. @vmarko will introduce new
compiled code in the future so that we ultimately won't be regressing on
time at all. Overall, space savings have been confirmed as on the order
of 100 KB per application.

A 4-5% slow down in art-opt on Golem, and no noticeable slow down in the
interpreter. The opt slow down should be diminished once the new
compiled code is introduced.

Test: m test-art-host

Bug: 20323084

Change-Id: Ic654a1fb9c1ae127dde59290bf36a23edb55ca8e

Merge "dex_instruction: k45cc HasVarArgs"
am: 3049b2a54b

Change-Id: I46dd32227166260b1832f974256d654269ac18bb

Merge "dex_instruction: k45cc HasVarArgs"

dex_instruction: k45cc HasVarArgs

Change-Id: I3be92269bb78c96ec41ab533c5792bd369c02178
Test: make test-art-host
bug: 30550796

Merge "Fix optimizing compiler not building for svelte"
am: 1a3fd4a646

Change-Id: Ib90de25574f62cad2aa967a5a287ac91e21325c5

Merge "Fix optimizing compiler not building for svelte"

Improve the thread flip.

- In addition to the threads that are suspended in FullSuspendCheck(),
  prioritize the resume of threads that are blocking for the thread
  flip at the JNI critical section entry and threads are about to
  transition to runnable (eg. blocking at the SOA entry from JNI).
- Shorten the length of the thread flip critical
  section (ThreadFlipBegin/End).
- Add some systrace scopes.

Bug: 29517059
Bug: 12687968
Test: test-art-host, Ritz EAAC, N9 libartd boot.
Change-Id: Idecec9936ae432c23d5f83321ba13339852018df

Merge "Remove unused include."
am: 16f7008195

Change-Id: Ibf7fdc4a8adda45daa41b93c44b2c86716285f52

Merge "Remove unused include."

Fix optimizing compiler not building for svelte

Fixes compiler not building when some of the codegen paths
are disabled.

Test: mmma -j art ART_TARGET_CODEGEN_ARCHS=svelte
m -j32 test-art-host

BUG=30928847

Change-Id: I52c78e8a4e507f74b1f2a39352970079721b737e

Remove unused include.

BUG: 29824203

Change-Id: Iea74f7b166c3e81c0d00c92442f27e162631e5ae

Log optimization passes in verbose mode

Introduce verbose logging of optimization passes run during
compilation.

Test: m test-art-host -j32
art -Xcompiler-option --runtime-arg -Xcompiler-option -verbose:compiler
-classpath classes.dex Test
Change-Id: Iae98ce9dcafc252f2d0eec138aa05b34e424bd2a

Merge "Remove ART_DEFAULT_INSTRUCTION_SET_FEATURES"
am: 1106d769f4

Change-Id: I6d357ea0daa86c3ec1efd620ac9d1fccdd87b120

Merge "Remove ART_DEFAULT_INSTRUCTION_SET_FEATURES"

Merge "Revert "Logging improvements for C++ / Java structure issues""
am: c6efcaa173

Change-Id: I7b7bb49c7332ba50d147cbcbe29074b98e3edf56

Remove ART_DEFAULT_INSTRUCTION_SET_FEATURES

It is never used.

Test: mmma -j art
Change-Id: I014a8ab11d3a3241c075503bc2674279130cb86f

Merge "Revert "Logging improvements for C++ / Java structure issues""

Merge "Add newline for INTERNAL_FATAL automatically"
am: d99565069c

Change-Id: I8b64721bcb3d9e0f8693fef2e0baedf3c53b3bd2

Revert "Logging improvements for C++ / Java structure issues"

Bug: 30913672

This reverts commit d833525cfe3ac2086e3156a8b3193c05974d6316.

Change-Id: I4e51591b49d54b8bb3e920fbc221c21aca0084fb

Merge "Add newline for INTERNAL_FATAL automatically"

Color spill slots in gc regalloc

Coloring spill slots avoids pathologically large stack
sizes by reusing spill slots when possible.

Test: ART_TEST_OPTIMIZING_GRAPH_COLOR=true m test-art-host

Change-Id: I4b4aea859c78b0515758f8b057ee870dbbfc2300

Merge "Track cumulative objects and bytes copied for CC"
am: e70b9243ba

Change-Id: Ie6414078cb883c45e813aed84f160b7bdbb9b48e

Merge "Track cumulative objects and bytes copied for CC"

Add newline for INTERNAL_FATAL automatically

Before, the behavior was different compared to other log levels
that automatically added newlines.

Bug: 30913672

Test: test-art-host

Change-Id: I02147d8ac7ea38f3e93450ef5290cdfed7b5a2eb

Merge "Add skeleton of a jvmti plugin for art."
am: fec0190ab6

Change-Id: If442ad2a2d304375bb06f431bd1a77e22799bdc3

Merge "Add skeleton of a jvmti plugin for art."

Merge "Blacklist failing libcore tests."
am: 4dcb0fa798

Change-Id: Icfdf66da1430688546ed97ce7a767643e232b336

Merge "Blacklist failing libcore tests."

Blacklist failing libcore tests.

bug: 30917411
test: art/tools/buildbot_build.sh && art/tools/run-libcore-tests.sh
Change-Id: Ib5e9adb6e9222d607b6d79f0f7eebf3d313272d2

Add skeleton of a jvmti plugin for art.

Test: mma test-art-host-run-test-901-hello-ti-agent
Change-Id: If6807b6238d57471e4ba0dd75c717721246443f6

Track cumulative objects and bytes copied for CC

Also print out these values when dumping GC performance info.

Bug: 12687968

Test: Test that values are reasonable after running EAAC.

Change-Id: Id04fadeaf52511560fd4b261f5287ea0a5dae9d4

Merge "Logging improvements for C++ / Java structure issues"
am: d127255351

Change-Id: I4a22c81581064af95631c9f4c6cc689cf4cb6027

Merge "Adjustments for changes made to java.lang.Class"
am: 1d36e2c312

Change-Id: Ib5bed6b807fdabd3573044e0bc613006d1e2d6c5

Merge "Always mark zygote large objects for CC"
am: 8f7e75ac4c

Change-Id: Ib59726a69f9dff1bc0ce6fc4972eee5e131fb5f0

Merge "jni: Fast path for @FastNative annotated java methods"
am: 3cc35c324b

Change-Id: I9947ab55cbb1ed18c4918eb2feaa99d023c7332d

Merge "Update ahat README and manifest for release 0.7."
am: 3f410202b1

Change-Id: Ieabaac82590b3c0aad5cb55d5034bb0c8b7603f4

Merge "Remove unused GetDalvikCacheFilenameOrDie."
am: 95afcbcc25

Change-Id: I717d0c4d7a482c1544d934b7ed80f7c8f7493e6b

Merge "Welcome to JavaFuzz as our latest A[a]rt tools team member!"
am: 088debe40a

Change-Id: Ie7f9a0a568fbf196f80ecde88f3d9a95ea4df13f

Merge "Logging improvements for C++ / Java structure issues"

Merge "Adjustments for changes made to java.lang.Class"

Logging improvements for C++ / Java structure issues

When C++ and Java structures are different sizes or arranged
differently some log statements are useful but are hard to
pick out because they merge with other log lines. Adding
"\n" to the end of them makes it easier to read.

Change-Id: Ia6d78e0832c7814bf1fef42bb3d8c056f83766bd
Test: make build-art-host

Adjustments for changes made to java.lang.Class

Fix up the runtime after removal of the annotationType field
from java.lang.Class.

Test: make test-art-host / boot a device
Change-Id: I5f37567d2b5ec381b35e66fef29287abe307b840

Merge "Always mark zygote large objects for CC"

Always mark zygote large objects for CC

Prevent needing to gray holders of zygote large objects.
System wide zygote space PSS after boot:
12644 kB -> 5571 kB for CC.

Also PSS reduction in zygote large objects themselves since their
gray bit would have been set each GC.
Overall LOS savings hard to measure, could be up to 316 * 4KB per
app since there are 316 zygote large objects.

Also clear mod-union tables for image spaces to prevent dirty
image pages if any of the image spaces point to zygote large objects.
System wide .art mmap: 37432 kB -> 34372 kB

System server before (N6P):
LOS shared dirty: 12888 kB
Zygote space shared dirty: 700 kB
Zygote space private dirty: 868 kB
.art private dirty: 1696 kB

After:
LOS shared dirty 13672 kB
Zygote space shared dirty: 1072 kB
Zygote space private dirty: 496 kB
.art private dirty: 1432 kB

Bug: 29516968

Test: test-art-host with baker CC, debug N6P phone booting

Change-Id: Ia37ce2c11217cf56885bd1d1dc084332fcbb7843

Merge "jni: Fast path for @FastNative annotated java methods"

jni: Fast path for @FastNative annotated java methods

Adds a faster path for java methods annotated with
dalvik.annotation.optimization.FastNative .

Intended to replace usage of fast JNI (registering with "!(FOO)BAR" descriptors).

Performance Microbenchmark Results (Angler):
* Regular JNI cost in nanoseconds: 115
* Fast JNI cost in nanoseconds: 60
* @FastNative cost in nanoseconds: 36

Summary: Up to 67% faster (vs fast jni) JNI transition cost

Change-Id: Ic23823ae0f232270c068ec999fd89aa993894b0e

Merge "Update ahat README and manifest for release 0.7."

Merge "Remove unused GetDalvikCacheFilenameOrDie."

Merge "Welcome to JavaFuzz as our latest A[a]rt tools team member!"

Welcome to JavaFuzz as our latest A[a]rt tools team member!

Rationale:
JavaFuzz is tool for generating random Java programs with
the objective of fuzz testing the ART infrastructure. Each
randomly generated Java program can be run under various
modes of execution, such as using the interpreter, using
the optimizing compiler, using an external reference
implementation, or using various target architectures.
Any difference between the outputs (a divergence) may
indicate a bug in one of the execution modes.

Test: tbd

Bug=30610121

Change-Id: I92dcac35f5229996936d01a0ba7f5acf6dc7b433

Remove unused GetDalvikCacheFilenameOrDie.

Change-Id: I8987fa488a89865d4a5fb56dd00cc396ee33bcce
Test: m test-art-host-gtest-utils_test32
Test: m build-art-host-tests
Test: m build-art-target-tests

Merge "Change gcstress runtest option"
am: 36bf3a2d28

Change-Id: I5e76005cbbfbeb659d5bc8455fd260131c5fffcd

Merge "Change gcstress runtest option"

Merge "ART: Refactor ImageSpace loading"
am: b23e3b2156

Change-Id: I38e37a7f88ffc79705fe4acc6e6a27ec48e48a38

Merge "ART: Refactor ImageSpace loading"

Update ahat README and manifest for release 0.7.

Change-Id: Iccc5db7112cb92e24e9e294273a3ce2b0d4b7aba
Test: Launch ahat and verify it reports 0.7 as the version.

ART: Refactor ImageSpace loading

Make the code more straightforward so it is easier to reason
about and extend.

Also change return types to unique pointers so it's clear that
it is the caller's responsibility to free those when necessary.

Bug: 30832951
Test: m test-art-host
Test: device booting
Change-Id: I3216eb702e45357a48af5158dacbe40e79bd1ae9

Merge "Clean up use of GetDalvikCache."
am: 882c0e4899

Change-Id: Ifac13144494305120408b7981f8fbe7a5ada9314

Merge "Clean up use of GetDalvikCache."

Merge "Improve READ_BARRIER_MARK_REG for arm32"
am: 9e0d58b20c

Change-Id: I745d271177e260a487e3eac5dd3d92f0ec7168a5

Merge "Improve READ_BARRIER_MARK_REG for arm32"

Merge "Revert "Enable IntermediateAddress for primitive arrays with read barriers.""
am: 554b6fb875

Change-Id: Iebee937e50aa7943afc7fcfc3acbc3f2414c666d

Merge "Revert "Enable IntermediateAddress for primitive arrays with read barriers.""

Revert "Enable IntermediateAddress for primitive arrays with read barriers."

This CL breaks the angler-userdebug build with
`ART_USE_READ_BARRIER=true`.

Test: Build angler-userdebug with `ART_USE_READ_BARRIER=true`.
Bug: 30762467
Bug: 26601270
Bug: 12687968

This reverts commit 12ecf0800d465acdaa3deccd383ff8ed3428a183.

Change-Id: Ia2069ac9436d2336311dd8d0f183c02e587586ae

Merge "ARM64: VIXL: Support a newer version of VIXL."
am: fe74ba9ea6

Change-Id: I5ff0142506107668ea9010a34c4e21f1b0e0533d

Merge "ARM64: VIXL: Support a newer version of VIXL."

Change gcstress runtest option

No longer passes down SS as the collector. Motivation is to allow
running gcstress with concurrent copying.

New usage is for SS:
mm test-art-host ART_TEST_GC_STRESS=true ART_DEFAULT_GC_TYPE=SS

Bug: 30807718

Test: mm test-art-host ART_TEST_GC_STRESS=true ART_DEFAULT_GC_TYPE=SS

Change-Id: I4416c1ca9ead027536acc9d24b1ec80d5a68c6a1

Merge "Fix heap poisoning"
am: 85b1811f0d

Change-Id: Icaa40d52acfe3899e16da0b0713c97fa0e1e288e

Improve READ_BARRIER_MARK_REG for arm32

Use blocked register IP as scratch, avoid pushing in fast path.

Clean up slow path to not have simpler logic and one less memory
write.

Add simple fast path handling for region space TLAB object
allocation.

Test: test-art-target, N6P booting with CC baker

Bug: 30162165

Change-Id: I6594e42d3d6277ffe7bb79df09df8be6bee85eb5

Merge "Fix heap poisoning"

Merge "Make test work when running ndebug."
am: 3469575a63

Change-Id: I7280780d2580ad9a3dc1ca78e5399a26730263ec

Merge "Make test work when running ndebug."

Fix heap poisoning

Was wrong register.

Test: test-art-host-run-test CC table lookup + poisoning.

Bug: 30162165
Change-Id: If347a40963bcc2fb9e21c54b5405c852bcd3af08

Merge "Revert "Revert "Iterative move coalescing for gc regalloc"""
am: 912cc4aad3

Change-Id: Iccbd0bd4eea87a1682ec11a16f2c428d90887771

Merge "Revert "Revert "Iterative move coalescing for gc regalloc"""

Merge "ART: Remove zygote OTA code"
am: ef1a3209ab

Change-Id: I7f52590e1818b3deb8e188b7c4ad13e1d613c8f6

Merge "ART: Remove zygote OTA code"

Make test work when running ndebug.

Change-Id: Iea58ccef51fddaeae4b1fb7084a7956400a81dfe
Test: ART_TEST_RUN_TEST_NDEBUG=true mma -j40 test-art-host-run-test-900-hello-plugin

Merge "Use TestCodeGeneratorX86 in codegen unit tests"
am: ba1642dec6

Change-Id: I2bdb9b704f019e035593faa7a8c66498c947c491

Merge "Use TestCodeGeneratorX86 in codegen unit tests"

Revert "Revert "Iterative move coalescing for gc regalloc""

This reverts commit 6f61ee5623a676ce983ccfa1aba9b2ae1237e163,
and fixes the arena allocator issues in the original CL.
Functionality from the original CL has not changed.

Test: m valgrind-test-art-host-gtest-register_allocator_test
Test: ART_TEST_OPTIMIZING_GRAPH_COLOR=true m test-art-host-run-test

Change-Id: Idd979f4e03f0c2800e0c07a0a7d392c8a4230aab

ART: Remove zygote OTA code

The renaming and relabeling of the A/B artifacts is now done in
a special-purpose script started by init, so as to not complicate
the logic here (e.g., name of the current slot).

Partially reverts commit 754479f7aa056602de8282f4f03718b42c20b89b.

Bug: 25612095
Bug: 28069686
Change-Id: I4d965361000fd448e199c9af059b6d18bfe928bb
(cherry picked from commit ff2a8ef2ed6b3f21da26f85197170d7d2a9f5134)

Clean up use of GetDalvikCache.

Switch the only caller of the non-void version of GetDalvikCache that set
create_if_absent true to use the void version of GetDalvikCache. Then no
callers of the non-void version of GetDalvikCache set create_if_absent to
true, so that argument is no longer needed. And without create_if_absent,
there is no case when the function will abort, so the 'OrDie" option is no
longer necessary.

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

Merge "ART: Reduce stack frame size limit on host"
am: 82d33b295f

Change-Id: I08214c2d5a737be9a64948d9cce500b6abf42b75

Merge "ART: Reduce stack frame size limit on host"

Merge "Optimize x86_64 TLAB allocation speed"
am: 29daeb62da

Change-Id: I2bee7d069403c3a8a3a1756672b6bac6b0f82705