Merge "build: Add golem run_build_test_targets for go/ab"

Merge "SIMD pcmpgtb,w,d,q for x86/x86_64"
am: 7cd18fb5a7

Change-Id: I7e54ae1206be1dd8bc0f15b89a0567ca640cfabc

Merge "SIMD pcmpgtb,w,d,q for x86/x86_64"

Merge "Add analyzer for dex memory map"
am: 674bc3c984

Change-Id: Ica8f665892856bbf42cdb7b9ae425011fb374a3b

Merge "Add analyzer for dex memory map"

SIMD pcmpgtb,w,d,q for x86/x86_64

Rationale:
Enables fast compare gt.

Test: assembler_x86[_64]_test
Change-Id: I0a069649480529f3fec2c2b100e2aaaa2cd79820

Add analyzer for dex memory map

Use libpagemap to process vdex pages that are resident in memory.
Walks over each vdex file mapping in a given pid, processing in one of
two ways.

Using the -s (summarize) option gives, e.g.:

MAPPING /system/framework/arm64/boot.vdex: 7e90133000-7e90641000
DEX classes.dex: 7e90133000-7e90608000
Section name                resident     total % of   % of
                               pages     pages sect.  total
Header                             0         0   0.00   0.00
AnnotationSetReferenceItem         0         0   0.00   0.00
StringId                           1        33   3.03   0.08
TypeId                             0         4   0.00   0.00
ProtoId                            1        21   4.76   0.08
FieldId                            0        25   0.00   0.00
MethodId                           2        69   2.90   0.16
ClassDef                           0        26   0.00   0.00
AnnotationSetItem                  0        13   0.00   0.00
CodeItem                           1       555   0.18   0.08
AnnotationsDirectoryItem           0        34   0.00   0.00
TypeList                           0        10   0.00   0.00
StringData                         2       174   1.15   0.16
DebugInfo                          0       175   0.00   0.00
AnnotationItem                     0        29   0.00   0.00
EncodedArrayItem                   0         3   0.00   0.00
ClassData                          0        59   0.00   0.00
GRAND TOTAL                        7      1230          0.57

Using the -v (verbose) option gives, e.g.:

MAPPING /system/framework/arm64/boot.vdex: 7e90133000-7e90641000
DEX classes.dex: 7e90133000-7e90608000
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
STTTTPPPPPPP.PPPPPPPPPPPPPPFFFFF
FFF.FFFF.F.F.FFFF...MMMMMMMMMMMM
MMMMMMMM.MM.....MMMMMMMMMMM..MMM
.M.MMM..MMMM.MMMMMMMMMMM.CCCCCCC
CCCCCCCCCCC.C.CCCCCC22.2........
.XX...XXX..XXXXXXX.............X
.XXXX......XXXXX..XXXX.XXXX..XXX
.............X..XX....X.XXXX....
..X....X...X..X..XXX.XXXXX.XXX.X
X.X.X..........X.X...........XXX
X...............................
.......X.....................XXX
..XX...XX.....X............XX.X.
.....XXXX......X.X..X...........
.XXXXXXX..XXXXXXXX..............
......XXXX...XX...XX..XXXX...X..
.......................X........
.......................X.XX..XX.
X...........X.X.X.X.............
X.....XXX..........XXXXX.XX.XX.X
XXXXX...........................
............X.................X.
X..............4....4...........
............4.tttttttttts...sss.
.ss.ss..s.ss.sssssss.sssssssssss
ssssssssssssssssssssssssssssssss
ssssssssssssssssssss.ss...ssss.s
ssssssssssssssssssssssssssssssss
ssssssssssssssssssssssssssssssss
sssssssss..DD.DD...DD...........
............D...................
................................
...................D.....DD.....
...........................D.D..
.D........................3...3.
......................EEE.c..ccc
.cc.ccc.c.c.....c.cc.ccc..c.....
.c...ccc.cc.ccc......

Where the letters indicate a page of a particular dex file section are
resident in memory and '.' indicates a page is not resident.

Bug: 35800981
Test: dexdiag <pid>
Change-Id: Ib4662fe499ad82d4f0baace2affab0d56f0890c9

Merge "Allow -Xjitsaveprofilinginfo for system server"
am: 8682fce263

Change-Id: I01af28a95bb1722725131588ea2ecefa6bb4896d

Merge "Allow -Xjitsaveprofilinginfo for system server"

Allow -Xjitsaveprofilinginfo for system server

setenforce 0 is still required to prevent selinux violations.

Test: adb shell setprop dalvik.vm.extra-opts "'-Xusejit:false -Xint -Xjitsaveprofilinginfo'" && adb shell start

Bug: 34927277
Bug: 36457259

(cherry picked from commit 449f0589c8c8572e32c57befee089123e9eea1ba)

Change-Id: I644cefe8aebf9ff47dc736eda8143d2f6258c324

Merge "Dump profile logs after the last save"
am: 25732aec3d

Change-Id: Ic88433c75e095974e672245d6aecc5ad311b5700

Merge "Dump profile logs after the last save"

Merge "Make base64 explicitly check dst_size for nullptr"
am: 1a5a1e5527

Change-Id: I0deeef9e5e45e7e788888df48e8547d18498579c

Merge "Make base64 explicitly check dst_size for nullptr"

Merge "MIPS64: Add ldi.df MSA instruction"
am: e2a2395609

Change-Id: Ie380252315099eb4b59eb9eb9e1e2ac6c2698ac2

Merge "MIPS64: Add ldi.df MSA instruction"

build: Add golem run_build_test_targets for go/ab

Adds following golem build targets:
 * art-golem-android-armv7
 * art-golem-android-armv8
 * art-golem-linux-armv7
 * art-golem-linux-armv8
 * art-golem-linux-ia32 (this is a host build)
 * art-golem-linux-x64  (this is a host build)

Call the art/tools/golem/build-target.sh with its corresponding target name as the
machine type, and it uses art-opt-cc for golem config type.

Ensures that we don't accidentally break the golem targets.

Also refactor the target_config.py to make it more clear
what's a make, run-test, or golem command.

Bug: 35766356
Change-Id: Id1a72e5eaed8b20d1d208cf0720b1cb45d48e74d

Merge "Add a cpplint preupload hook."
am: 432fccc4c0

Change-Id: Ie3edcc46b0781d92e137dc7ceb0f13506aceb748

Merge "Add a cpplint preupload hook."

Merge "ART: TI Agent test library refactor"
am: 1927f66beb

Change-Id: Ie14195b4dcd9063af20acb21e0877a2222c8a277

Merge "ART: TI Agent test library refactor"

Add a cpplint preupload hook.

This runs cpplint.py over modified .cc and .h files before uploading
them to gerrit. This generally takes ~1-2 seconds based on my testing
and the longest one I've been able to find was ~10 seconds.

Test: PREUPLOAD_COMMIT=d9911eeca ./tools/cpplint_presubmit.py
Test: PREUPLOAD_COMMIT=405284789 ./tools/cpplint_presubmit.py
Test: PREUPLOAD_COMMIT=9763f2eb7 ./tools/cpplint_presubmit.py
Test: Manual

Bug: 35810246
Bug: 36855589

Change-Id: Ie0ed9136bdde15d78f2aa40c73fef97e12d12dda

Make base64 explicitly check dst_size for nullptr

Also moved it out of utils.h and into common_runtime_test.h since it
is only used for tests.

Test: ./test.py --host -j40
Change-Id: Ib3c18cb510be833928482c0b2f47f9fb02933f7f

Merge "ART vectorizer."
am: 08ae45625d

Change-Id: I123a156aaf593762a0f21dde475ffb1d73929c20

Merge "ART vectorizer."

Merge "Make GetSortedDexFileSections a utility"
am: 2eaacede1e

Change-Id: Ida699a678f7edfab5778da4432d03eaf784b6d25

Merge "Make GetSortedDexFileSections a utility"

Merge "SIMD pavgb,w for x86/x86_64"
am: 0d9ec0535c

Change-Id: Idcda1302f7aa7da99f7ddf6c7480cae2a9eeb9b0

Merge "SIMD pavgb,w for x86/x86_64"

Merge "ARM64: Faster forwarding address check in mark entrypoints."
am: fcab6513b5

Change-Id: I4388e3939347c7b5c8269c58f10eae0e1ea143e3

Merge "ARM64: Faster forwarding address check in mark entrypoints."

Merge "Kill logd in buildbot-setup.sh."
am: 589e13337d

Change-Id: I1f80d00b49f51f3438b208b6201120820a3cdc55

Merge "Kill logd in buildbot-setup.sh."

Make GetSortedDexFileSections a utility

Make this method a utility in preparation for other users outside
dexlayout.

Bug: 35800981
Test: make -j 40 test-art-host
Change-Id: I054c1e1c82834a0c94be6b9ee63d1992fa9f52df

Kill logd in buildbot-setup.sh.

The daemon seems to leak on fugu with Android N.

Change-Id: I07202c0a6ce644e313c310e0592e3e0d58d158a0

ART: TI Agent test library refactor

More refactoring, lowering the reliance on ART-provided functionality
and adding a library that includes all the code that can run without
ART.

Bug: 32072923
Test: m test-art-host
Change-Id: I67d84056a6fd7722c58855fccbdea3f6869b2efb

SIMD pavgb,w for x86/x86_64

Rationale:
Break-out CL of ART Vectorizer.
Enables fast halving add with rounding

Bug: 34083438
Test: assembler_x86[_64]_test
Change-Id: I09173376b803d671a6b05a33e630f45f778cea52

Merge "Clean up ArtClassDefinition"
am: 5b92c48f99

Change-Id: I6a993f84a133c1157364a106f8736b0a10623435

Merge "Clean up ArtClassDefinition"

ART vectorizer.

Rationale:
Make SIMD great again with a retargetable and easily extendable vectorizer.

Provides a full x86/x86_64 and a proof-of-concept ARM implementation. Sample
improvement (without any perf tuning yet) for Linpack on x86 is about 20% to 50%.

Test: test-art-host, test-art-target (angler)
Bug: 34083438, 30933338

Change-Id: Ifb77a0f25f690a87cd65bf3d5e9f6be7ea71d6c1

Merge "Lower daemons priority to 124 (art)."
am: 7298b1ae3e

Change-Id: I91eb1be02b879939a3b2c1bf7dac25fd7b8da0cf

Merge "Lower daemons priority to 124 (art)."

Merge "ART: Refactor JVMTI run tests"
am: 7201fe3b67

Change-Id: I544cd644d15dbaecf841c2451a41c21c7907c4c3

Merge "ART: Refactor JVMTI run tests"

Clean up ArtClassDefinition

This struct was somewhat messy. Clean it up and consolidate functions
into it.

Bug: 31455788
Test: ./test.py --host -j40

Change-Id: Ia3abe92dcf3313677de5e200f673252f8e41d69e

MIPS64: Add ldi.df MSA instruction

Also fixes RepeatTemplatedRegisterImmBits template.

Test: mma test-art-host-gtest

Change-Id: Ib23f8a65ba924623f8c3a2d75d4ec4491d18feb0

ARM64: Faster forwarding address check in mark entrypoints.

Bug: 12687968
Test: testrunner.py --target
Change-Id: I352971ddf8f574e600c054cf2b4c0a973d20f829

Merge "Change thread suspend timeout to be fatal for non-debug"
am: bb75449355

Change-Id: I2699992871038089c2edf89cd938d6a4cda89678

Merge "Change thread suspend timeout to be fatal for non-debug"

Merge "Re-enable test 983 with speed-profile"
am: 113ecfe9e7

Change-Id: Icfd4f04b675079f6f92da825253b02d07ef82e84

Merge "Re-enable test 983 with speed-profile"

ART: Refactor JVMTI run tests

In preparation for code deduplication with CTS.

Bug: 32072923
Test: m
Change-Id: Ibbe005c94252bd29eae7f88aad301b0b20ddb80d

Dump profile logs after the last save

The DumpInfo statement did not capture the last profile save.

Also, update -Xps-min-methods-to-save and -Xps-min-classes-to-save in
the art script. A value of 0 will attempt to write the profile even if
no method has been profiles and that's useless.

Test: ./out/host/linux-x86/bin/art --profile --64 -Ximage:$PWD/out/host/linux-x86/framework/core.art -Xusejit:true -Xcompiler-option --compiler-filter=speed-profile -Xcompiler-option --compiler-backend=Optimizing -verbose:oat -Djava.library.path=$PWD/out/x86_64/host/linux-x86/lib64 -cp $PWD/generated/benchmarks.dex benchmarks.ExoPlayerBench.java.ExoPlayerBench

Change-Id: I5927ab87d7c134c271fbc252e62fdc03ab02a255

Re-enable test 983 with speed-profile

We make sure to fixup dex files in all situations before passing it to
JVMTI agents. This means that dexlayout won't break this test anymore.

Test: stress --cpu 60; \
      while ./test/run-test --host --random-profile 983; do; done

Change-Id: I047ffcbbba51db4c7bf5c22e482bd5864bc8b6d1

Merge "(Re)enable the use of inline caches in AOT"
am: dac3f7d74d

Change-Id: Ic5bf0b119ac5624513e53cd8ad7ff9e85aebbb35

Merge "Update profile version"
am: 4f0250ca95

Change-Id: I7ce4ac7363af1dbe558c496ed55132b393d55933

Merge "(Re)enable the use of inline caches in AOT"

Merge "Update profile version"

Merge "ART: Fix VISIT_OBJECTS and ABORT for FollowReferences"
am: f62940eab1

Change-Id: Ifecc04ef4212576771d15fd00dc876a568435145

Merge "ART: Fix VISIT_OBJECTS and ABORT for FollowReferences"

Update profile version

I forgot to do it in cea9e9d26c32372ec8c75e6c89b1bfc08c5e1911. This will
force regeneration of possibly faulty profiles.

Test: m test-art-host
Bug: 36371709
Change-Id: I476c97005b6c17a22c57117c06028a2371276326

Lower daemons priority to 124 (art).

To avoid jank.

(chery picked from commit 38a57ef702964b82836f89a1f996aff4d1453d94)

Bug: 36631902
Test: boot marlin oc-dev and check priorities.
Test: test-art-host
Change-Id: I8f809fbf5ac9605f656884fcf68643c87f22d154

ART: Fix VISIT_OBJECTS and ABORT for FollowReferences

If a root report does not set VISIT_OBJECTS, the referree should
not be added to the worklist.

Whenever an ABORT is requested, do not report the remaining roots.

Bug: 36727422
Test: art/test.py --host -t 913
Change-Id: I942559f6ab356fe2573a3a9bb88b2662cd5b23d0

Merge "Dedup original dex file for classes transformed on first load."
am: c700f00067

Change-Id: I79190abfb7998b0684054e47bcccfed9926c13ba

Merge "Dedup original dex file for classes transformed on first load."

Merge "Revert^2 "Improve user-friendliness of art script.""
am: 02311bd39d

Change-Id: I5403504da49995ca5cd42efb4c926f9b9af653a9

Merge "Revert^2 "Improve user-friendliness of art script.""

Revert^2 "Improve user-friendliness of art script."

Fixes quoting for mksh for arguments with whitespace.

Test: art/tools/run-jdwp-tests.sh --mode=device
Change-Id: I0b8983a0c0ebcc1b64ad943859fbb85d3774ab75

Merge "Do more GCs for test 141"
am: b68e12c7d9

Change-Id: I9767192431c17959104722ef7cfe6485902b4244

Merge "Do more GCs for test 141"

Merge "Refactor image writer"
am: 6004e14691

Change-Id: I6713a029794f5b267d16e6749743e334946e5777

Merge "Refactor image writer"

Merge "Update deopt statements in 638-checker-inline-caches"
am: 15996a3d73

Change-Id: Id50c44faea5adb7d2921ea7f502b90ea3611298e

Merge "Update deopt statements in 638-checker-inline-caches"

Dedup original dex file for classes transformed on first load.

We set the originalDexFile field to a native pointer to the
art::DexFile (as a boxed Long). In the case where there were no
non-can_retransform_classes agents this DexFile will be owned by the
transformed classes ClassLoader and is guaranteed to be valid for as
long as the Class is. This means we can simply keep track of that
pointer to hold onto the original Dex file.

Test: ./test.py --host -j40
Bug: 31455788
Change-Id: I0e5a74f20f86d32dc73babe64b81469b3155c4dd

Do more GCs for test 141

Try to prevent rare race conditions that could cause the class loader
to not be unloaded.

Bug: 36377932
Test: test/testrunner/testrunner.py --host -t 141

Change-Id: I9429350b73c241478d474af37262576a512cc6a6

Merge "Downgrade another duplicate method DCHECK to WARNING"
am: 5e07b5acc0

Change-Id: I57ccc02db9b3e964e6de5725e81d4f8ecc50f402

Merge "Downgrade another duplicate method DCHECK to WARNING"

Refactor image writer

Aim to have common functions for copying references and pointers.
Required for adding support for faster image fixups.

Test: test-art-host -j32

Bug: 34927277
Bug: 34928633
Change-Id: Ia654efc483b332eea3535570496bfeccd7c635ee

(Re)enable the use of inline caches in AOT

Test: m test-art-host (638-checker-inline-caches, 644-checker-deopt)
Bug: 36371709
Change-Id: Ic6e1b58845e7f7107be5a7041c47c44702af7b02

Update deopt statements in 638-checker-inline-caches

Test: m test-art-host-638-checker-inline-caches after inline
      caches have been activated
Bug: 36371709
Change-Id: I8378b0027a94bda89097fb0d3081e9184069643d

Merge "Undo dex2dex compilation before invoking LoadHook"
am: e2a739acb4

Change-Id: I3f975722e811da45ab3e646d8456a760215ee8f6

Merge "Undo dex2dex compilation before invoking LoadHook"

Merge "Layout string data"
am: 591654e70f

Change-Id: Ib827c3407f4ce0ea881b7658b2d767b047890f80

Merge "Layout string data"

Merge "Change Ti::Agent collection from vector to List"
am: 4e03442619

Change-Id: I41864acc50a4114c999183b4fc91a5a5f0219b6c

Merge "Change Ti::Agent collection from vector to List"

Downgrade another duplicate method DCHECK to WARNING

This DCHECK got missed when writing change
'Ib26f30718726a9064056855192cef302f3df1eeb'. This is also caused by
duplicate methods in dex files causing some ArtMethod's to appear in
vtables more than once.

Bug: 32549051
Bug: 32546154
Bug: 36446088
Bug: 36399552

Test: ./test/testrunner/testrunner.py --host -j40
Change-Id: Id8ef07d7185958a91f008c2297db96c090dbecd7

Undo dex2dex compilation before invoking LoadHook

We need to undo any dex_to_dex compilation that might have taken place
before passing a dex file to any registered ClassFileLoadHooks to
ensure that no internal opcodes are present in any methods.

Test: ./test.py --host -j40
Bug: 36653594

Change-Id: Ia42c77312e685d69f6b3ea764fad01710b10ab45

Merge "Gracefully handle bogus profiling info for inline caches."
am: d07f2ba2c6

Change-Id: I846cf611255488675854f5ad3b43039c82e48749

Merge "Gracefully handle bogus profiling info for inline caches."

Merge "Fix profile merges in profman"
am: 1e45a522ea

Change-Id: If488dd18668acecf4910d4503b7472fffc06776a

Merge "Fix profile merges in profman"

Layout string data

Inspect dex code in class initializers and hot methods to find out
what const strings are likely to be resolved. Along with these,
look at static field accesses and group the field name and type with
other hot strings.

Group method shorties together.

Maps vdex PSS: 7478k -> 6900k
No change in odex PSS.

Changed dexlayout_test to use -v instead of comparing to a hex
output.

Bug: 36457259
Bug: 35800981

Test: test-art-host

Change-Id: Iaa5de649c5b2b1d9f178525d2f42fe00e3fb879f

Change Ti::Agent collection from vector to List

We want these to be in a list to avoid the need for moving them
around, which would introduce the need to perform annoying accounting.

Test: ./test.py --host -j40
Bug: 36716572
Bug: 31455788
Change-Id: Ie16523f4e7d79934fa7844fb9b9a26e125cfa9eb

Merge "ART: Fix ObjectFree reporting"
am: 8b71f086ad

Change-Id: Id414ff482a9e19b0f032bffa80d4140b4bf9f5b4

Merge "ART: Fix ObjectFree reporting"

Merge "ART: Fix GetThreadInfo"
am: c5e0f20479

Change-Id: I2b01fcd1ac98265b512b4cb762235d040e3ada41

Merge "ART: Fix GetThreadInfo"

Merge "Performance improvements in invoke-polymorphic runtime support"
am: 26ed72a372

Change-Id: Ide4297866ad3f797efb2e42056838f5b9fdabf96