Merge \\"Fix profiling output to properly display class info.\\" am: 9755c262df
am: d4a452714e

Change-Id: Ic69efc32ff05bcd4cb601ab4a7aaa2f95c3b28fd

Merge \"Fix profiling output to properly display class info.\"
am: 9755c262df

Change-Id: I417529769b10bd7693461c90f0306881158332a6

Merge "Fix profiling output to properly display class info."

Fix profiling output to properly display class info.

The profiler stores class def indexes, not type ids.

Change-Id: Ieec9374a2ba381387ed444f376922478183145f8

Merge \\"Add missing filter cards to zygote mod union table\\" am: 93afc5f20a
am: e42a192af4

Change-Id: Ieb57921d0fbc408f7c334df6682ff3b7121b08d8

Merge \"Add missing filter cards to zygote mod union table\"
am: 93afc5f20a

Change-Id: I40b355ebdabc4d53661836156b8106fa2c003fbd

Merge "Add missing filter cards to zygote mod union table"

Add missing filter cards to zygote mod union table

Without filtering the cards, every object in the zygote is grayed
during the GC. This was deleted in a PS of previous CL.

GrayAllDirtyImmuneObjects goes from 1.974ms to 376.464us for CC on
N9 maps. This happens during the pause. This CL also fixes regression
in zygote PSS.

Bug: 29516968
Bug: 12687968

Change-Id: I42014e78b1de3ce9b2eefa3bd32f0d24e2ff71c6

Merge \\"Improve CC handling for immune objects\\" am: e92730cbc5
am: 58387badba

Change-Id: I899f68854dd6b35d478e0efb36741e56dc70a520

Merge \"Improve CC handling for immune objects\"
am: e92730cbc5

Change-Id: Ie0226a28ebfc44595e614c3c942fba82797ecfa0

Merge "Improve CC handling for immune objects"

Merge \\"Remove redundant 601 test, it\\'s all in 600.\\" am: 90187c9858
am: 64af33cc09

Change-Id: I6a82ce53064a38362561f3d73535f5bb1ce50166

Merge \"Remove redundant 601 test, it\'s all in 600.\"
am: 90187c9858

Change-Id: Ibdc01b2f1047bca176c39942e5e80097706ad570

Merge \\"runtime: Refactor asm_support to be auto-generated (mostly)\\" am: 10656de47c
am: 5153722d83

Change-Id: I4392365edd778fcfc14637237ed3495c1ee5c012

Merge "Remove redundant 601 test, it's all in 600."

Merge \"runtime: Refactor asm_support to be auto-generated (mostly)\"
am: 10656de47c

Change-Id: Ida3192ed82ad2f835eae6f49c46f013ed012a77f

Merge "runtime: Refactor asm_support to be auto-generated (mostly)"

Merge \\"ART: Add pointer-size template to some reflection functions\\" am: 3023890d94
am: 7f53486ab4

Change-Id: Id56b9d72125de03bb1c334bdd60ddbea4db19d32

Improve CC handling for immune objects

Currently we reduce ram for immune objects by racing agianst the
mutators to try and finish processing them before the mutators
change many objects to gray. However there is still a window of time
where the mutator can dirty immune pages by changing the lock words
to gray. These pages remain dirty for the lifetime of the app.

This CL changes uses the FlipCallback pause to gray all of the
immune objects that have a dirty card. Once these objects are all
gray we don't to gray any more objects in the immune spaces since
these objects are the only ones that may reference non immune
objects.

Also only scan objects that are gray when scanning immune spaces to
reduce scanning time.

System wide PSS after boot on N9, before:
61668 kB: .art mmap
11249 kB: .Zygote

After:
36013 kB: .art mmap
12251 kB: .Zygote

Results are better than demonstrated since there are more apps
running after.

Maps PSS / Private Dirty, before:
.art mmap     3703     3116
  .Zygote      577      480

After:
.art mmap     1655     1092
  .Zygote      476      392

System server before:
.art mmap     4453     3956
  .Zygote      849      780

After:
.art mmap     2326     1748
  .Zygote      640      564

EAAC:
Before:
ScanImmuneSpaces takes 669.434ms GC time
Scores: 718, 761, 753 average 744
GC time: 4.2s, 4.35s, 4.3s average 4.28s

After:
ScanImmuneSpaces takes 138.328ms GC time
Scores: 731, 730, 704 average 722
GC time: 3.92s, 3.83s, 3.85s average 3.87s

Additional GC pause time is 285us on Maps on N9.
TODO: Reduce this pause time.

Test: N9 booting, test-art-host, EAAC all run with CC

Bug: 29516968
Bug: 12687968

Change-Id: I584b10d017547b321f33eb23fb5d64372af6f69c

Merge \"ART: Add pointer-size template to some reflection functions\"
am: 3023890d94

Change-Id: I4310ad3ef62f7645ff82427512f0ef975f386341

Merge "ART: Add pointer-size template to some reflection functions"

runtime: Refactor asm_support to be auto-generated (mostly)

Usage:

* If the defines are no longer up to date, re-run generate-asm-support

* To add a new field:
--- Edit one of the offset_ or constant def files.
--- Rebuild cpp-define-generator and re-run generate-asm-support

Change-Id: I772430fcf1ad9af40898ebb453848f8412612386
TODO: Integrate this into the build somehow
TODO: Account for 32 vs 64-bit and read barrier vs no read barrier

Remove redundant 601 test, it's all in 600.

Rationale:
Nicolas, I don't understand why you duplicated this test.
Since all (and now more) is in 600, removing this clone.

Change-Id: Ibc8b47549f054972f07b53b1554e7ab4dc0cb33e

Merge \\"Make java 8 run-tests runnable on the RI again.\\" am: 659ad8fe6b
am: e18ea7f2f2

Change-Id: I3430f7ea91b24810c09543aa783cd98f6723dccb

Merge \"Make java 8 run-tests runnable on the RI again.\"
am: 659ad8fe6b

Change-Id: I73852191af8517db26adf0f7b744441853c05187

Merge "Make java 8 run-tests runnable on the RI again."

Merge \\"Launch ahat server before processing the heap dump.\\" am: 0abe10a59b
am: cc706a4a1b

Change-Id: I854871991844e05c82e94f969b76db0093fd509a

Merge \"Launch ahat server before processing the heap dump.\"
am: 0abe10a59b

Change-Id: I051ad88d1460d129a5eb65fc36a6e28354bbc251

Merge "Launch ahat server before processing the heap dump."

ART: Add pointer-size template to some reflection functions

The unstarted runtime may run code for a different pointer size,
even when no transaction is active (e.g., during startup). To
retain performance when the runtime is up and executing under
normal conditions, add a template parameter and use sizeof(void*)
in places where it is adequate.

For maintainability, it is necessary to drop the default for
the transaction template parameter. Implicit conversions from
bool to size_t may lead to incorrect code and hard to diagnose
problems. So instead ensure that all callers must give all
template parameter values.

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

Merge \\"Fix stream-tracing with default methods\\" am: 38be534337
am: 2382f012bb

Change-Id: I420d52342c2d08ab2ede048372b362c642d4a0d2

Merge \"Fix stream-tracing with default methods\"
am: 38be534337

Change-Id: Ibad99838d15d0b87ea5ba1fe897f27d4ba83f447

Merge "Fix stream-tracing with default methods"

Merge \\"Make static helper methods member functions of OptimizingCompiler class\\" am: a69e790ad9
am: d786250bbb

Change-Id: I9401fc2711224c9c47285738bb139e25befb82c0

Merge \"Make static helper methods member functions of OptimizingCompiler class\"
am: a69e790ad9

Change-Id: Iaf23b0f1163ac836a3a4e4c5319517388e6cfa62

Merge "Make static helper methods member functions of OptimizingCompiler class"

Fix stream-tracing with default methods

Test: Ran mma ART_TEST_TRACE=true ART_TEST_TRACE_STREAM=true -j40 test-art-host

Bug: 30102284
Bug: 30286605

Change-Id: I12f2534bc2d42a7504e6c1972c4cfdda2f77286f

Merge \\"Revert \\"Revert \\"ART: Add dex2oat swap usage test\\"\\"\\" am: edbecee66f
am: 37d11c693a

Change-Id: Ia5b8bc6d23018bf14b9200ef74899ba7558b8449

Merge \"Revert \"Revert \"ART: Add dex2oat swap usage test\"\"\"
am: edbecee66f

Change-Id: I3321270cf3746b172208b628dbddc968db1c5482

Merge "Revert "Revert "ART: Add dex2oat swap usage test"""

Merge \\"Add fast path to arm64 READ_BARRIER macro\\" am: a71ad9a2d3
am: fb2092dab3

Change-Id: Ie0dedfae86f63ecfa0cf393d372070ea42fbfa0a

Merge \"Add fast path to arm64 READ_BARRIER macro\"
am: a71ad9a2d3

Change-Id: I0bebdeed40c436e59f294e2c2b94c7ac6085b374

Merge "Add fast path to arm64 READ_BARRIER macro"

Revert "Revert "ART: Add dex2oat swap usage test""

Now correctly ignores the target (but with enough infrastructure
to introduce this some day).

This reverts commit ec743ffd47ac9d3c10a449926d78c2eb51e5208e.

Bug: 29259363
Test: m test-art-host-gtest-dex2oat_test
Test: m test-art-target-gtest-dex2oat_test (shamu)
Change-Id: I11b4e755bc8cb1e2eea29cd006e8df67df632c00

Merge \\"Do not emit stack maps for runtime calls to ReadBarrierMarkRegX.\\" am: de4cf16f46
am: 6c652d78ce

Change-Id: Ic398c02cb7e3a3ffb1029681b3ac6c36eaaaba0c

Merge \"Do not emit stack maps for runtime calls to ReadBarrierMarkRegX.\"
am: de4cf16f46

Change-Id: I58975c1fa1cdc2627b5fdee0d84322b8282a6411

Merge "Do not emit stack maps for runtime calls to ReadBarrierMarkRegX."

Merge \\"Revert \\"ART: Add dex2oat swap usage test\\"\\" am: c4aa896181
am: da07be90c5

Change-Id: I9744169d0d5f109f07d1c67f3ed55ee19ba645fb

Merge \"Revert \"ART: Add dex2oat swap usage test\"\"
am: c4aa896181

Change-Id: I61c6aa7d5057a49639312262bc74f9162c827a70

Merge "Revert "ART: Add dex2oat swap usage test""

Revert "ART: Add dex2oat swap usage test"

The test doesn't correctly exclude the target, as it will only work
on the host.

This reverts commit ab2a54093386c85756fe78daedd11ff4408a5988.

Bug: 29259363
Change-Id: Ie50df2e6f0c63cb10359c3862ad44ee3c03d4e3b

Add fast path to arm64 READ_BARRIER macro

EAAC benchmark time from 978.7857143ms to 969.5714286ms on N9 based
on 42 samples. Reduces artReadBarrierSlow calls from 9M to 1M.

Not a huge improvement since we were already checking the lock word in
ReadBarrier::Barrier.

Test: N9 boots, test-art-host, EEAC runs. (All with CC enabled).

Bug: 30162165
Bug: 12687968

Change-Id: Ifb97b52ea84e21c7df83addfb91c5f05f41db32d

Merge \\"ART: Change run-*-tests to ANDROID_{HOST|PRODUCT}_OUT\\" am: 115c1de1f8
am: 88b5cdb5be

Change-Id: Ic5a32cf1eba832fe189c720898185ae876479bc7

Merge \\"ART: Add dex2oat swap usage test\\" am: da2b8dc3ed
am: f11b3aa623

Change-Id: I8164b21964584a3caeaee79689a0453ab6c9717a

Merge \"ART: Change run-*-tests to ANDROID_{HOST|PRODUCT}_OUT\"
am: 115c1de1f8

Change-Id: I6a977a1a3ed81c25ce20c43ee55a55c2028d20b1

Merge \"ART: Add dex2oat swap usage test\"
am: da2b8dc3ed

Change-Id: I019cccda99f190384f57bf2e23909ed79a74dc2d

Make static helper methods member functions of OptimizingCompiler class

Make RunOptimizations, MaybeRunInliner and RunArchOptimizations member
functions of OptimizingCompiler class.

Both versions of RunOptimizations are protected in preparation for
bisection bug search CL.

Change-Id: I596efa9ed3fccd1ed3798c6427cc166e2a5d28bd

Merge "ART: Change run-*-tests to ANDROID_{HOST|PRODUCT}_OUT"

Merge "ART: Add dex2oat swap usage test"

Merge \\"Combine offsets in loop-based dynamic BCE.\\" am: 7b922ff88b
am: f45e600ea6

Change-Id: Ied92881efb83a4a7dcd6238f459f8e5e5c96ead4

Merge \"Combine offsets in loop-based dynamic BCE.\"
am: 7b922ff88b

Change-Id: I6889e535ce4ea69404562c95451bf82d70c35a23

Merge "Combine offsets in loop-based dynamic BCE."

Do not emit stack maps for runtime calls to ReadBarrierMarkRegX.

* Boot image code size variation on Nexus 5X
  (aosp_bullhead-userdebug build):
  - total ARM64 framework Oat files size change:
    115584120 bytes -> 109124728 bytes (-5.59%)
  - total ARM framework Oat files size change:
    97387728 bytes -> 92517584 (-5.00%)

Test: ART host and target (ARM, ARM64) tests.
Bug: 29506760
Bug: 12687968
Change-Id: I979d9fb2b4e09f4c0c7bf33af2cd91750a67f989

Combine offsets in loop-based dynamic BCE.

Rationale:
Similar to what I did recently for dom-based dynamic BCE, this
CL combines offsets for the tests generated for loop-based
dynamic BCE. For a set of n references, this reduces the
number of generated tests from 2*n+1 down to at most 4
(in some cases even less).

TEST: 530-checker-loops3

BUG=27430379

Change-Id: Ic80c2563eaae23f514c1fd52965dd83bccb9d190

Merge \\"ARM64/x86-64: Fix mterp fill-array-data-payload pointer calculation.\\" am: 9e27d02040
am: 6aec037d15

Change-Id: I48398d2bcb731b60ded4aeb6f4263f5c3152136a

Merge \"ARM64/x86-64: Fix mterp fill-array-data-payload pointer calculation.\"
am: 9e27d02040

Change-Id: Ib9198592d068055b0caa2051072795cb77fc6a0f

Merge "ARM64/x86-64: Fix mterp fill-array-data-payload pointer calculation."

ARM64/x86-64: Fix mterp fill-array-data-payload pointer calculation.

Fix the pointer calculation to sign-extend the offset
instead of zero-extending it, just like we do for the switch
table pointer calculation. Clean up comments for the switch.

Test: Additional test in 412-new-array.
Change-Id: Ibb1d2d3fcb109f59280aca08de21e42edc4ce66b

Merge \\"ARM64: Improve mterp cmpl/cmpg.\\" am: 41c7e2e6ac
am: 1562d62bf2

Change-Id: I5596939b447cf2591c2197a12888c289a773e931

Merge \"ARM64: Improve mterp cmpl/cmpg.\"
am: 41c7e2e6ac

Change-Id: I8abfa3377a1e884e46f28ce413a67e97492335e2

Merge "ARM64: Improve mterp cmpl/cmpg."

JIT: Don\'t update the dex cache of another class loader.
am: e50a7b834b  -s ours

Change-Id: I0af74a19e19bc66fcca47aca21757136073a9161

Merge changes I295c7876,Ib4b84b7b am: 84413a7617
am: 7aee171992

Change-Id: Iccde165be1e432c9c3e86f03f07a1e2d45428d5f

Merge changes I295c7876,Ib4b84b7b
am: 84413a7617

Change-Id: I8b32598a46cb589cdefa1937ba4b9b69926f25b9

Merge changes I295c7876,Ib4b84b7b

* changes:
  ART: Remove PACKED from ArtMethod's ptr_sized_fields_
  ART: Rename ArtMethod JNI field

Merge \\"Use non-CAS thread flip root visitor.\\" am: d4b7ad7135
am: 0cbcc8b076

Change-Id: I7ace21c998f00f53dd5e9d56703b385dc1457e26

Merge \"Use non-CAS thread flip root visitor.\"
am: d4b7ad7135

Change-Id: I4c560952f5f4ed488d0a849feadf2a2d25e15a2b

Merge "Use non-CAS thread flip root visitor."

ART: Remove PACKED from ArtMethod's ptr_sized_fields_

Remove the PACKED(4) hack, as it's highly annoying when debugging
a 64-bit process. Instead, fix the actual offset and size computation
for cross-size accesses.

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

ART: Add dex2oat swap usage test

Add a test that checks that native alloc size goes down when using
swap, as an indication that we're actually effectively using swap.

Bug: 29259363
Test: m test-art-host-gtest-dex2oat_test
Change-Id: Ifa5aef1b97696309283de78be08699a6399a4d1d

Launch ahat server before processing the heap dump.

Change-Id: Ic8f18ad3010cb0937f5ef68228359258ec4793fd
Test: Manually launch ahat on a large heap dump with ahat already
      running. Verify that ahat fails immediately.
Test: Manually launch ahat on a large heap dump without ahat already
      running. Verify that visiting localhost:7100 immediately
      eventually resolves to the overview page rather than say the
      site can't be reached.
Bug: 28611294

Use non-CAS thread flip root visitor.

We don't need to use CAS to update the thread-local GC roots for the
thread flip.

Bug: 12687968
Bug: 29517059
Test: libartd.so boot. ART tests. Ritzperf EAAC.
Change-Id: Ia2acab824f756bd7d2ad501b2040233e0d394356

Merge \\"ART: Make run-test temp dir consistent\\" am: a8f4e9061a
am: fd3b946eca

Change-Id: I35b5bf8d27d6eb9340c161a80befc30df90e5cb2

Merge \"ART: Make run-test temp dir consistent\"
am: a8f4e9061a

Change-Id: I1d785103a7e4df2884384b0b482552a8e163e9f3

ART: Rename ArtMethod JNI field

The field is multi-purpose, rename it to data and clean up
accessors in preparation of more checks.

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

Merge "ART: Make run-test temp dir consistent"

Merge \\"Add a way to measure read barrier slow paths\\" am: d8b668fbb6
am: 7cf6333832

Change-Id: I511a006bed571faceb739448687cee44afb0198f

Merge \"Add a way to measure read barrier slow paths\"
am: d8b668fbb6

Change-Id: I010c834dc4c922e7986854cba12612823fa974b4

ART: Make run-test temp dir consistent

We use the username as a directory component in run-test. Use the
same when driven through the Makefile.

Drop the username in run-test when TMP_DIR is set.

Test: m test-art-host-run-test
Test: art/test/run-test --host 001-HelloWorld
Change-Id: I060997ffbd80cd4da30dd6ac8d3954641de3292b

Merge "Add a way to measure read barrier slow paths"

ARM64: Improve mterp cmpl/cmpg.

Use CSET+CNEG instead of MOV+CNEG+CSEL. Prefer the
CNEG/CSET alias over the CSNEG/CSINC for readability.

Test: Run ART test suite on Nexus 9 with the interpreter.
Change-Id: I5c4fb0cf2c053904253e8e82f3e7e05c774b0583

Merge \\"Change return type of artIsAssignableFromCode for MIPS64\\" am: b78b3a8d93
am: 56dc4f5349

Change-Id: I42be21594fe6dad907057e0342d568924642ff93

Merge \\"Fix the definition of MACRO_LITERAL for OS X on x86-64.\\" am: ed33b7357c
am: 37d65ef15f

Change-Id: I9f34ab8c1510c97aa22bbdd9c1c47395b31dfc56

Merge \"Change return type of artIsAssignableFromCode for MIPS64\"
am: b78b3a8d93

Change-Id: Ifcc0f61101ab0a424273b97877fbb444864fdce9

Merge \"Fix the definition of MACRO_LITERAL for OS X on x86-64.\"
am: ed33b7357c

Change-Id: Ida4c2fd8c2918b77c72423fdda7cd7fb52c10aab

Merge "Change return type of artIsAssignableFromCode for MIPS64"

Merge "Fix the definition of MACRO_LITERAL for OS X on x86-64."

Fix the definition of MACRO_LITERAL for OS X on x86-64.

Test: "ART_USE_READ_BARRIER=true mmma art" on OS X.
Change-Id: Ia2d4c7a3eb7fec346ddfa4c7b0f7b700f1137344

Change return type of artIsAssignableFromCode for MIPS64

This has been missed by Ic505befc6c94e2dccbc8abf2b13d4c2d662e68d1.
This fixes MIPS64 build.

Bug: 30232671
Test: make -j 32 out/target/product/generic_mips64/obj/SHARED_LIBRARIES/libart_intermediates/arch/mips64/entrypoints_init_mips64.o
Change-Id: Iec89d25e2d38c6efc0d1025767d0ac2a8bdb7dcd

Merge \\"Move caller-saves saving/restoring to ReadBarrierMarkRegX.\\" am: 057361ca33
am: cc25e86689

Change-Id: I615172164d2edf199f868dd6d3d0bd59a759d5a9