Merge "ART: Separate out Realloc reuse and alignment tests"

Merge "Ongoing improvements in java fuzz testing"

Merge "Revert "x86/x86-64: Avoid temporary for read barrier field load.""

Revert "x86/x86-64: Avoid temporary for read barrier field load."

Fault handler does not recognize the instruction
    F6 /0 ib    TEST r/m8, imm8
so we get crashes instead of NPEs.

Bug: 29966877
Bug: 12687968

This reverts commit ccf06d8f19a37432de4a3b768747090adfbd18ec.

Change-Id: Ib7db3b59f44c0d3ed5e24a20b6c6ee596a89d709

Merge "ART: Clean up interpreter includes"

Merge "x86/x86-64: Avoid temporary for read barrier field load."

x86/x86-64: Avoid temporary for read barrier field load.

Add TEST instructions for memory and immediate. Use the byte
version to avoid a temporary in read barrier field load.

Test: Tested with ART_USE_READ_BARRIER=true on host.
Test: Tested with ART_USE_READ_BARRIER=true ART_HEAP_POISONING=true on host.
Bug: 29966877
Bug: 12687968
Change-Id: Ia415d3c2e1ae1ff6dff11d72bbb7d96d5deed6ee

ART: Clean up interpreter includes

Move goto-interpreter special-cased Clang declarations to goto
implementation file. Separate out declarations into their own
headers. Tag the declarations with thread safety annotations.

Minor compile-time impact when running with -j1.

Change-Id: I03664d4e7dc6361b2510a8c9beb693d23a657c9d
Test: m test-art-host
Test: m ART_TEST_INTERPRETER=true test-art-host

Merge "Use full pass name when building optimizations"

Merge "MIPS32: Optimize R6 round(float) intrinsic."

ART: Separate out Realloc reuse and alignment tests

Separate out the tests for Realloc buffer reuse and realloc
alignment tests. This allows to turn off reuse tests under
memory tools (where we do not reuse).

Change-Id: I93375c6fbc577f9b796325f8143317610948455a
Test: m test-art-host-gtest-arena_allocator_test
Test: m valgrind-test-art-host-gtest-arena_allocator_test

Merge "Improve art_quick_resolve_string for arm64 CC"

Merge "Fix build"

Use full pass name when building optimizations

If possible, use full pass name provided in --run-passes rather
than its base version.

Test: m test-art-host -j32

1. Prepare a run-passes file with content:
dead_code_elimination$initial
instruction_simplifier
x86_memory_operand_generation
2. Run art for a dex file like:
art -Xcompiler-option --run-passes=run-passes -Xcompiler-option
--dump-passes -classpath classes.dex Test
3. Verify that dead_code_elimination$initial string is present in
dump-passes output.

Change-Id: I92d9ed0c8b919ea03f625f549123f546dffe546b

Merge "ART: Fix x86_64 entrypoint breakage with SYMBOL()."

Improve art_quick_resolve_string for arm64 CC

Check gc_is_marking instead of the mark bit first, this makes it
that we don't go slow path if the GC is not running.

Also reduced the code by a few instructions.

EAAC CC: ~2000 -> 1256

Test: test-art-target-run-test -j4

Bug: 20323084
Change-Id: I57a1f7a52f1909e2e5dd1b2cfd2612b4a642fe37

Fix build

Move instruction_simplifier_arm.cc to the arm source list to fix
full-eng builds.

Change-Id: Ic7fc884f5cc2acec5579dc417acece21b7e674aa

Merge "ART: Test arena allocator alignment"

ART: Fix x86_64 entrypoint breakage with SYMBOL().

Change-Id: Id42b6e9a7e5444536f6fb672ec40e9f37c77a649

Merge "ART: String DexCache asm fast path for ARM64"

Merge "ART: String DexCache asm fast path for ARM"

Merge "ART: String DexCache asm fast path for x86"

Merge "Only compile device codegen for device architectures"

Merge "ART: String DexCache asm fast path for x86_64"

Merge "Revert "Revert "Improve the thread flip."""

ART: Test arena allocator alignment

Add tests for Alloc and Realloc alignment.

Change-Id: If17b1d898200752a6a1ae25f436c73e5b7eb23f7
Test: m test-art-host-gtest-arena_allocator_test

ART: String DexCache asm fast path for ARM64

In the entrypoints, before a string is looked up in the slow path (in
the intern table), I added assembly to check the dex cache and return a
string pointer if the string is already in the dex cache.

Test: test-art-host

Change-Id: Ic9a724848c61e4fa66e82334e2c7b20b13bf8e6c

ART: String DexCache asm fast path for ARM

In the entrypoints, before a string is looked up in the slow path (in
the intern table), I added assembly to check the dex cache and return a
string pointer if the string is already in the dex cache.

Test: test-art-host

Change-Id: I245946b3efa9c2e4fa56f761effb90963ac22ec4

Merge "Fix debugger missing handle between adding and tagging object."

ART: String DexCache asm fast path for x86

In the entrypoints, before a string is looked up in the slow path (in
the intern table), I added assembly to check the dex cache and return a
string pointer if the string is already in the dex cache.

Test: test-art-host

Change-Id: Id0c337b380adc0b2bde5325745c928290d9da1b0

Merge "Add define needed by new vixl"

Ongoing improvements in java fuzz testing

Added --device option to runner to define target in case
        multiple devices are attached

Added more cases to JavaFuzz, removed a few cases that
are likely to hang the program (such as changing loop
variables) since throughput is more important for fuzzing

Test: run_java_fuzz_test.py

BUG=30610121

Change-Id: I0cd6f5d3b7fb35ddb91d91c0c0c00afaf83218d8

Fix debugger missing handle between adding and tagging object.

Bug: 30951794
Change-Id: Ic9c5f296efe839a8961ab00d08afc54c5443bc17

Revert "Revert "Improve the thread flip.""

This reverts commit db3204f87c3f7c4de89762ce9e8502a9dc25c2d8.

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.
- Add a read barrier for the locked objects during the thread dump in
  case the thread is in the middle of flipping.

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

Merge "ART: Adding asm support for string dex cache"

ART: String DexCache asm fast path for x86_64

In the entrypoints, before a string is looked up in the slow
path (in the intern table), I added assembly to check the dex cache and return a
string pointer if the string is already in the dex cache.

Test: test-art-host

Change-Id: Id1b0314d597c2abff3f7d332f4ab66271f026dc1

ART: Adding asm support for string dex cache

Added constants and offsets for the string dex cache fast paths that
will be introduced into entrypoints in subsequent commits.

Test: test-art-host

Change-Id: I19f97526e06cbedc782d5ef11359796731cdad82

Add define needed by new vixl

vixl now requires -DVIXL_INCLUDE_SIMULATOR_AARCH64 when including
simulator-aarch64.h.

Tests: mmma -j art, m -j build-art-tests
Change-Id: I9b9bb7ca618ccbb505de943825f5e9517542fa1b

Merge "Fix debugger calling new String()."

Merge "ARM/ARM64: Improve comments for kSaveEverything assembly."

ARM/ARM64: Improve comments for kSaveEverything assembly.

Addresses post-submit comments for
    https://android-review.googlesource.com/255290

Test: Rely on TreeHugger (just adds comments).
Bug: 30212852
Change-Id: I6ccc004774e18bbd83e3940e908cfc6c521142db

Merge "ART: Add thread offset printing hook to disassembler"

Merge "Do not run 537-checker-arraycopy on non-Baker read barrier configs."

Do not run 537-checker-arraycopy on non-Baker read barrier configs.

Test: ART_USE_READ_BARRIER=true ART_READ_BARRIER_TYPE=TABLELOOKUP make test-art-host-run-test-537-checker-arraycopy
Bug: 29516905
Bug: 12687968
Change-Id: Ie77f3a7ce19f3ff78d2132ca70fb10b595eb54d5

Merge "ART: Enforce arena alignment in Realloc"

ART: Enforce arena alignment in Realloc

Enforce the expected alignment in Realloc when the given buffer
can be expanded.

Add debug checks to prove alignment guarantees.

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

Fix debugger calling new String().

Used to return an empty string instead of the newly created String.
Requires accompanying jdwp test fix to test String contents.

Bug: 30951794
Change-Id: I27a8e0afcbe7e32a14dd3c394e44b6cd4d3c58f6
Test: mm -j24 run-jdwp-tests-host

Merge "ART: Added DCHECKS to find alignment breakage."

ART: Add thread offset printing hook to disassembler

To prepare separation of disassembler from libart, add a function
hook to the disassembler options for thread offset name printing.

Bug: 15436106
Change-Id: I9e9b7e565ae923952c64026f675ac527b560f51b

ART: Added DCHECKS to find alignment breakage.

Change-Id: I47aed866ec3c3858dc3ee27cb84cfaddf44009d1

Merge "Revert "Improve the thread flip.""

Merge "ART: Fixing build breakage (sharpening DCHECK)."

Merge "ART: Fixing build breakage."

ART: Fixing build breakage (sharpening DCHECK).

Change-Id: I7ba19808dde0712739a278075da57f724166a233

ART: Fixing build breakage.

Caused by Ic654a1fb9c1ae127dde59290bf36a23edb55ca8e.

Change-Id: Ia0c0fbcb6c8b141c5ee30a89ab6bc1c653916698

Merge "Fix building tests with partial arch codegen support"

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

Revert "Improve the thread flip."

This reverts commit e61e6fdd404b8f04bd33fc3ea49e38e92e31ad69.

ThreadStress failing.

Bug: 29517059
Bug: 12687968
Change-Id: I0ee8ef04f77c5f9378e4bd21c5dedb4e435a2dc6

Only compile device codegen for device architectures

Change the default for ART_TARGET_CODEGEN_ARCHS to svelte, so that only
the architectures used on the device are included in the device
libraries.

Tests: m -j test-art-target
Bug: 30928847
Change-Id: I95c8a77a5520a1030abcd45e8bb6f56bc8fc7814

Fix building tests with partial arch codegen support

Add conditionals around more code that is only used for codegen for
specific architectures, and move a few more files into the
architecture-specific codegen lists.

Tests: ART_HOST_CODEGEN_ARCHS="x86_64 mips" m -j ART_TARGET_CODEGEN_ARCHS=svelte test-art-host
Bug: 30928847
Change-Id: I0444d15e1cafe4c9b13ff78718c3b13b544270e7

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

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."

Merge "Color spill slots in gc regalloc"

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"

dex_instruction: k45cc HasVarArgs

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

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."

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"

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""

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"

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."

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"

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!"