Merge "Add verify-art-runtime usage to dex2oat"
am: 9ee48fb

* commit '9ee48fbf7395488ea34d7d268edb31d7abe4ebe9':
  Add verify-art-runtime usage to dex2oat

Merge "Add verify-art-runtime usage to dex2oat"

Add verify-art-runtime usage to dex2oat

Bug: 27688793

(cherry picked from commit c6068709a482065d6f5426640f1c9316797508e3)

Change-Id: Idcf6b8023a15c816168ce42456d2bd7b463d0951

Merge "Fix race in AllocEntrypointsInstrumented"
am: 6f1b7ec

* commit '6f1b7ecf61a7be88bcbb259726e99a78a7235d15':
  Fix race in AllocEntrypointsInstrumented

Merge "Fix race in AllocEntrypointsInstrumented"

Merge "Fix issue with proxy invocation on default methods"
am: 932e6bf

* commit '932e6bf238a00ae9813e91acb7249cb063fab205':
  Fix issue with proxy invocation on default methods

Merge "Fix issue with proxy invocation on default methods"

Fix race in AllocEntrypointsInstrumented
am: 77d9931

* commit '77d993107773b7b9bd7f07ce08d0aaac1631bf84':
  Fix race in AllocEntrypointsInstrumented

Fix race in AllocEntrypointsInstrumented

We were using the quick_alloc_entry_points_instrumentation_counter_,
this counter is updated before the threads are suspended. The
allocator could come out of a suspend point, see that threads are
supposedly still no instrumented, continue the allocation, then
suddenly quick_alloc_entry_points_instrumentation_counter_ becomes
1 and alloc_tracking_enabled_ becomes true resulting in a failing
DCHECK.

The fix is to add a boolean that is updated only when the threads
are suspended.

Bug: 27506909

(cherry picked from commit 77d993107773b7b9bd7f07ce08d0aaac1631bf84)

Change-Id: Id12983ef77c4fddb0394e4439ce0829777f1a70b

Fix race in AllocEntrypointsInstrumented

We were using the quick_alloc_entry_points_instrumentation_counter_,
this counter is updated before the threads are suspended. The
allocator could come out of a suspend point, see that threads are
supposedly still no instrumented, continue the allocation, then
suddenly quick_alloc_entry_points_instrumentation_counter_ becomes
1 and alloc_tracking_enabled_ becomes true resulting in a failing
DCHECK.

The fix is to add a boolean that is updated only when the threads
are suspended.

Bug: 27506909
Change-Id: Ic3c6716d2af9c00189ce9271f7a2825b03cdd8a0

Merge "Remove --native-debuggable compiler option."
am: 8e57978

* commit '8e57978b5f330a322f05644c2db7fa8ce89b1b46':
  Remove --native-debuggable compiler option.

Merge "Make art::HCompare support boolean, byte, short and char inputs."
am: 03fdc90

* commit '03fdc90721a1874a4aef9adfd8fc1b9b4b4bc249':
  Make art::HCompare support boolean, byte, short and char inputs.

Merge "Forgot space."
am: 1a738fc

* commit '1a738fc659a7b91456b168d2ee1edcd9eb777569':
  Forgot space.

Merge "runtime: delete native/java_lang_Runtime.h."
am: 615017b

* commit '615017b07b05c29604144612897e5f7989948d24':
  runtime: delete native/java_lang_Runtime.h.

Merge "Fix inliner crash related to type propagation."
am: 8b5e04b

* commit '8b5e04b3573f430dc0475c17f4b3c6ada8aaccf5':
  Fix inliner crash related to type propagation.

Merge "Re-enable checker tests 536 and 537"
am: 713c519

* commit '713c519db15aaa8d6f33b744fd28adddb97a07c2':
  Re-enable checker tests 536 and 537

Merge "Remove --native-debuggable compiler option."

Merge "Make art::HCompare support boolean, byte, short and char inputs."

Remove --native-debuggable compiler option.

Check if both --debuggable and --generate-debug-info are set instead.

History: I have recently added the --native-debuggable flag to control
whether extra stack maps are generated in order to produce accurate
native line number mapping of all generated code. I didn't want to
include it into --debuggable since we compile boot.oat as debuggable
and so it would be affected.

On second thought, it would have been reasonable to generate the extra
stackmaps only if both --debuggable and --generate-debug-info are set,
instead of introducing another compiler flag. This means we do not
affect boot.oat but we still get the extra stack maps if DWARF debug
information is explicitly requested.

Change-Id: I7e5e557e4850a88b3b6f86178d2cb645fb1e1110

Merge "Forgot space."

Merge "runtime: delete native/java_lang_Runtime.h."

Forgot space.

Change-Id: If9eced6d5bd6d8d0248bad0e7b12095fb5a84d26

Merge "Fix inliner crash related to type propagation."

Make art::HCompare support boolean, byte, short and char inputs.

Also extend tests covering the IntegerSignum, LongSignum,
IntegerCompare and LongCompare intrinsics and their
translation into an art::HCompare instruction.

Bug: 27629913
Change-Id: I0afc75ee6e82602b01ec348bbb36a08e8abb8bb8

Merge "Re-enable checker tests 536 and 537"

Merge "Clean up NullCheck generation and record stats about it."
am: 914d71e

* commit '914d71ead70bb6f2084b2ed39a9fd58fd014f67d':
  Clean up NullCheck generation and record stats about it.

Merge "Improve compiler stats"
am: 1583e62

* commit '1583e624d4c970d8e571b265b9a8f08402d91f82':
  Improve compiler stats

Merge "Add more statistics for memory use of the JIT."
am: 532da65

* commit '532da65d01f13facd261367beb34ee41146671d4':
  Add more statistics for memory use of the JIT.

Re-enable checker tests 536 and 537

They were disabled due to a bug in Jack that is fixed now.

Bug: 25635944
Change-Id: Id68824a8b4abbb6710e61897ffb3ed5daf839413

Fix inliner crash related to type propagation.

- Do not agressively try to resolve a return type.
- Deal with unresolved return type.

bug:25492619

Change-Id: Idc9c96a0b376fe5ee86b411c02ce7078c7f48c84

runtime: delete native/java_lang_Runtime.h.

Now unused, except for an unnecessary include.

bug: 27387202
Change-Id: Ie77dc0da156a64e8ea03af1878b44d20c16c5ae5

Merge "Clean up NullCheck generation and record stats about it."

Merge "Improve compiler stats"

Merge "Add more statistics for memory use of the JIT."

Add more statistics for memory use of the JIT.

Collect data for stack maps, profiling info, and compiled code.

bug:27520994

Change-Id: Ic87361230c96ce0090027a37d750e948d806c597

Merge "Call __jit_debug_register_code() indirectly via global variable."
am: 011b511

* commit '011b51175b9e4c96f26aea376afdf23b84276e9a':
  Call __jit_debug_register_code() indirectly via global variable.

Merge "Call __jit_debug_register_code() indirectly via global variable."

Clean up NullCheck generation and record stats about it.

This removes redundant code from the generators and allows for easier
stat recording.

Change-Id: Iccd4368f9e9d87a6fecb863dee4e2145c97851c4

Improve compiler stats

- report the max size of arena alloc
- report how many virtual or interface invokes were inlined

Change-Id: I82f154a8e25b5e3890181a1aa11346cdc3f93e37

Merge "Set USE_JACK to true."
am: 9a70d58

* commit '9a70d583029958d033f2ceca864032ca8fdd6598':
  Set USE_JACK to true.

Merge "Set USE_JACK to true."

Set USE_JACK to true.

This fixes running checker when just doing art/test/run-test.

Change-Id: Icfac5896cdfd64eb16b0127d5f16f552b93ad03e

Merge "Move specific build rule of 529 into the default script."
am: 767c3b9

* commit '767c3b9779cfa877867ba0e8b9f66a726cf24ecf':
  Move specific build rule of 529 into the default script.

runtime: delete native/java_lang_Runtime.cc
am: f525272

* commit 'f525272b1edec721058e7d888545dd5d97b5045d':
  runtime: delete native/java_lang_Runtime.cc

Merge "Move specific build rule of 529 into the default script."

Move specific build rule of 529 into the default script.

Change-Id: I90ff4b4c64a4e077b360bbec7a8e21b52139f395

runtime: delete native/java_lang_Runtime.cc

We're now using Runtime.c from ojluni, which does the same set
of things via the JVM interface. This removes unnecessary duplication
of code between Runtime.cc and OpenJdkJvm.cc.

Also includes a few changes to JVM_NativeLoad to bring it in sync
with the version in Runtime.cc.

bug: 27387202

(cherry picked from commit 5f97157263d1b52bf4aea523b28294b21d694da1)

Change-Id: Ic2848755505a0cb25da8237150b07dadf0a42929

Merge "Collect memory use for the JIT."
am: f808e8a

* commit 'f808e8a0cc218c2b98023ef0e91f3c5b74ad2962':
  Collect memory use for the JIT.

Merge "Accept boolean as an input of HDivZeroCheck."
am: 576a13a

* commit '576a13ab2f1bc82e72245db5fbb4dd8587b19bf1':
  Accept boolean as an input of HDivZeroCheck.

Merge "Collect memory use for the JIT."

Collect memory use for the JIT.

bug:27520994

Change-Id: I67b0c5b822001bfde8738a988c1ade69f1a26e3f

Merge "Accept boolean as an input of HDivZeroCheck."

Accept boolean as an input of HDivZeroCheck.

All our arithmetic operations accept it.

bug:27624718
Change-Id: I1f6bb95dc77ecb3fb2fcabb35a93b31c524bfa0a

Merge "Fix checker test."
am: 41954d8

* commit '41954d8f0d1cdada0886875508b0653b1be4996c':
  Fix checker test.

Merge "Fix checker test."

Fix checker test.

Change-Id: I1388868ca69c9f964f2019ff79d5c7aa0f00d57c

Merge "Revert "Assembly TLAB allocation fast path for arm64.""
am: d35f4a2

* commit 'd35f4a2eacf9ee9c9d75bb0c00eec7ae31ad1949':
  Revert "Assembly TLAB allocation fast path for arm64."

Merge "Revert "Tentative fix for build breakage with read barriers.""
am: 4344e11

* commit '4344e11598b3079032773ad7e381658f39bed92e':
  Revert "Tentative fix for build breakage with read barriers."

Merge "Revert "Assembly TLAB allocation fast path for arm64.""

Revert "Assembly TLAB allocation fast path for arm64."

Assembly code does not compile
Bug: 9986565

This reverts commit b7e52b02a82c69e59c88f06945fb21672dfe9923.

Change-Id: I80de1bcd6270e2d76f79cb708811a41c039ea5bb

Merge "Revert "Tentative fix for build breakage with read barriers.""

Revert "Tentative fix for build breakage with read barriers."

This reverts commit 448f21e2c1cb43b193736598719979ebd2aed0b0.

Change-Id: Id821effa06697b1464581bcd9c8f34cd0ea69896

Merge "Optimizing: Fix arm64 shifter operand test."
am: a0ef558

* commit 'a0ef5582d325485866d34fbaa6276a43bc72de09':
  Optimizing: Fix arm64 shifter operand test.

Merge "Make art::HCompare side effect free."
am: 3e90681

* commit '3e906819c33d19c9586ecd9441f0ffb8433f2bad':
  Make art::HCompare side effect free.

Merge "Optimizing: Fix arm64 shifter operand test."

Merge "Make art::HCompare side effect free."

Merge "ART: Fix ignoring duplicate fields."
am: d9ebad9

* commit 'd9ebad9b52605ed785222a86c84e58fa7cb66686':
  ART: Fix ignoring duplicate fields.

Optimizing: Fix arm64 shifter operand test.

Generic simplifier removes 3 shifts from the test since
    https://android-review.googlesource.com/208199
and then the arm64-specific shifter operands don't apply.

Bug: 27638111
Change-Id: Ie6c6ec1f40ec263f3e40e8129f29e0191fd9de50

Merge "ART: Fix ignoring duplicate fields."

Merge "Tentative fix for build breakage with read barriers."
am: 41bcd81

* commit '41bcd81aebfb48cb848c84bc40f241a18c376af3':
  Tentative fix for build breakage with read barriers.

Merge "Tentative fix for build breakage with read barriers."

Tentative fix for build breakage with read barriers.

Change-Id: I490821357525bdeb19c330f350635bd645f8ab53

Merge "Fix HDoubleConstant::IsZero and HFloatConstant::IsZero."
am: 4d20c08

* commit '4d20c081ec372a5ec2a405e9ac4521fe09148922':
  Fix HDoubleConstant::IsZero and HFloatConstant::IsZero.

Merge "Fix HDoubleConstant::IsZero and HFloatConstant::IsZero."

Merge "Disable test."
am: 553ed05100

* commit '553ed05100f079203371da55e8a9d824b8e9948e':
  Disable test.

Merge "Disable test."

Disable test.

Change-Id: Id06de23646187967df3035ca423f4cd6c73cf0a7

ART: Fix ignoring duplicate fields.

Ignore duplicate fields when calculating class size without
embedded tables. We already ignored them when loading class
members. This fixes a check failure in debug build that the
final class is no smaller than the temporary class.

Bug: 21868015
Bug: 27626707
Change-Id: Id72121724abc093d536c2329c61a24854a20122c

Fix issue with proxy invocation on default methods

Bug: 27621360

Change-Id: I617eee243f475872235fd75b6a401e056111ea23

Merge "Assembly TLAB allocation fast path for arm64."
am: 877b18bcf4

* commit '877b18bcf4c6a69502b9a9b838dedd2528d3de40':
  Assembly TLAB allocation fast path for arm64.

Merge "Assembly TLAB allocation fast path for arm64."

Assembly TLAB allocation fast path for arm64.

This is the arm64 version of CL 187537.

Speedup (GSS GC with TLAB on N9):
        BinaryTrees:   591 ->  493 ms (-17%)
        MemAllocTest:  792 ->  755 ms (-5%)

Bug: 9986565

Change-Id: Icdad28cab0fd835679c640b7eae59b33ac2d6654

Merge "ART: Fix shift simplification, x >>> 64."
am: 5942e53716

* commit '5942e537161a059ef1969a81db101876555cd77d':
  ART: Fix shift simplification, x >>> 64.

Merge "ART: Fix shift simplification, x >>> 64."

ART: Fix shift simplification, x >>> 64.

Fix braino in
    https://android-review.googlesource.com/208199

Bug: 27638111
Change-Id: I8f12008af8bba943664c8a9eac3f2d2f7c820e79

Merge "ART: Fix valgrind failure in image_test64."
am: 12af36e091

* commit '12af36e09116dc677820f98e33ff2982e07d652a':
  ART: Fix valgrind failure in image_test64.

Call __jit_debug_register_code() indirectly via global variable.

This is speculative future optimization. It allows the debugger to
change the value of the global variable to point to its injected code.
This, in turn, allows it to handle JIT events in-process without
triggering breakpoint and without the inter-process cost of it.

Change-Id: I4961c8862603bfb43c84974a3a51110ee202bb1b

Merge "ART: Fix valgrind failure in image_test64."

Fix HDoubleConstant::IsZero and HFloatConstant::IsZero.

bug:27639313
Change-Id: I2f30a65a07662dfce0a6d6f4ed356a8a0b3dcdef

Merge "Optimizing: Improve shift simplification, x >>> 64."
am: 6ce4766f13

* commit '6ce4766f13326bdf704b5205bfbda2c7dfbc72ae':
  Optimizing: Improve shift simplification, x >>> 64.

Merge "Optimizing: Improve shift simplification, x >>> 64."

Merge "runtime: delete native/java_lang_Runtime.cc"
am: 706d2cd559

* commit '706d2cd55938075f4215e64dd1deeaf966a43316':
  runtime: delete native/java_lang_Runtime.cc

Merge "runtime: delete native/java_lang_Runtime.cc"

Optimizing: Improve shift simplification, x >>> 64.

Simplify shifts by a multiple of bit size, not just 0.
ARM codegen does not expect to see such shifts and it
is guarding against them with a DCHECK().

Bug: 27638111
Change-Id: I3ae8383d7edefa0facd375ce511e7a226d5468a1

runtime: delete native/java_lang_Runtime.cc

We're now using Runtime.c from ojluni, which does the same set
of things via the JVM interface. This removes unnecessary duplication
of code between Runtime.cc and OpenJdkJvm.cc.

Also includes a few changes to JVM_NativeLoad to bring it in sync
with the version in Runtime.cc.

bug: 27387202
Change-Id: I92aac81d131661aca588e352aee7a167593e17d4

Make art::HCompare side effect free.

All our back ends implement all comparisons without making a
runtime call, so we can mark art::HCompare as a side effect
free instruction unconditionally.

Change-Id: I9a9e7c09156c642edb6af1fe84408f887e762f2e

Merge "Optimizing: Prevent potential valgrind error."
am: 5681b6638b

* commit '5681b6638b0001c62d02795b0853e2e82787edf1':
  Optimizing: Prevent potential valgrind error.

ART: Fix valgrind failure in image_test64.

Explicitly clear LengthPrefixedArray<ArtMethod> padding
in ImageWriter. This also removes a potential source of
non-deterministic data in compiled boot image.

Bug: 26687569
Bug: 27552451
Change-Id: Icf4bf34c9ea5b3159462760fbf556d287841191d

Merge "Optimizing: Prevent potential valgrind error."

Optimizing: Prevent potential valgrind error.

This CL preemptively extends the workaround from
    https://android-review.googlesource.com/208230
to an almost identical bit of code.

Bug: 27651442
Change-Id: I7683d42b46b16f2293916defc6ef1d871dc9af6c