Merge "Revert "Revert "Make original dex file be more deduplicated"""

Merge "Add 'dist' to gtest targets to avoid jack issues."

Add 'dist' to gtest targets to avoid jack issues.

Change-Id: Ibde8ba3e64817859b2bd61e575ea45057c96cd8e

Merge "ARM: Reduce the number of branches generated for HCondition and HSelect"

Merge "ARM: Improve the code generated for HInstanceOf"

Merge "Improve user-friendliness of art script."

Merge "Improvements in the Inliner."

Merge "Makefile cleanup/fix after refactoring."

Makefile cleanup/fix after refactoring.

1) Put hprof-conf in ART_TEST_HOST_RUN_TEST_DEPENDENCIES
2) Remove duplicated code.

Change-Id: I11d141a13e94292f5a2bfefe1016be39aaece019

Merge "Fix issue with Android.run-test.mk"

Fix issue with Android.run-test.mk

We were clearing 2 makefile variables that was needed for our
buildbots to run correctly.

Test: m clean && ./test.py --target -j8
Test: m clean && ./test.py --host -j40
Change-Id: Ic15ccf6701bf12b7ee5aeddf940e0c343677c8b0

Revert "Revert "Make original dex file be more deduplicated""

This reverts commit a7c04c4fe6623985ba2d0cca942d5e1e584f4358.

Reason for revert: Issue with target buildbots was fixed

Test: m clean && ./test.py --target -j8
Bug: 31455788

Change-Id: If00ef9433a9308810a40ea4e0d809987c934cef8

Merge "Move backtrace logic to backtrace_helper.h"

Merge "Remove TimeZoneTest.testAllDisplayNames() failure"

Merge "MIPS64: Check for MSA presence in each MSA instruction"

Merge "Implement a SIMD spilling slot."

Merge "Silence ulimit in run-test"

Move backtrace logic to backtrace_helper.h

Refactor this out so we can use it other places.

Test: test/run-test --host --gcstress --dev --64 048

Change-Id: I1c357714badd417a67a34cd5d371ddbf1282a179

Merge "Deterministic DexCache in heap images."

Remove TimeZoneTest.testAllDisplayNames() failure

The test is being removed.

Bug: 22786792
Test: m checkbuild
Change-Id: Ie96a4088985b39bd15ea506830a394da79d5f624

Merge "Testrunner: Print colored text only when printing on terminal"

Silence ulimit in run-test

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

Improvements in the Inliner.

- Change from a depth limit to a total number of HInstructions
  inlined limit. Remove the dex2oat depth limit argument.
- Add more stats to diagnose reasons for not inlining.
- Clean up logging to easily parse output.

Individual Ritz benchmarks improve from 3 to 10%.

No change in other heuristics. There was already an instruction budget.
Note that the instruction budget is rarely hit in the "apps" I've tried
with.

Compile-times improve from 5 to 15%.
Code size go from 4% increase (Gms) to 1% decrease (Docs).

bug:35724239
test: test-art-host test-art-target

Change-Id: I5a35c4bd826cf21fead77859709553c5b57608d6

Improve user-friendliness of art script.

o Add --help and usage.
o Document common user errors.
o Add flags for gdb and callgrind.

Test: art/tools/run-jdwp-tests.sh --host & manual testing

Change-Id: Ie40b53f31df17c439daf5f6425d5315dcdbe785f

Merge "MIPS64: Remove memcpy() from java.lang.String.getChars intrinsic."

Merge "MIPS32: Remove memcpy() from java.lang.String.getChars intrinsic."

Merge "Revert "Revert^4 "CHA for interface method."""

Revert "Revert^4 "CHA for interface method.""

Still failing on the bots.

This reverts commit fb185130f35bbfe2ba1875974ed39ef0b3c7a77f.

Change-Id: I1629e237dd41ce4784ccadbfc346352dfcbdb137

Merge "Revert "Make original dex file be more deduplicated""

Revert "Make original dex file be more deduplicated"

Bug: 31455788

This reverts commit 9556cb2bfc6788656d53a9ddb2159ca2040f6e38.

Change-Id: If22f2bd9d1b788c2a784dbe0e7f0c2304d8e4a59

Implement a SIMD spilling slot.

Rationale:
The last ART vectorizer break-out CL    \O/
This ensures spilling on x86 and x86_4 is correct.
Also, it paves the way to wider SIMD on ARM and MIPS.

Test: test-art-host
Bug: 34083438

Change-Id: I5b27d18c2045f3ab70b64c335423b3ff2a507ac2

Merge "Reduce test 051-threads to 128 threads instead of 512"

Testrunner: Print colored text only when printing on terminal

Test: ./art/test/testrunner/testrunner.py
Change-Id: Ica9701d6acd459d47fb63a722234fa17b141c33f

Merge "Add test.py script to run gtest and run-test"

Merge "Remove test logic from Android.run-test.mk"

Add test.py script to run gtest and run-test

By default, it will run both run-test and gtest
If either of the two are explicilty asked for, it will
not run the other one.

Options:
--run-test : To run run-test
--gtest : To run gtest
-j : Number of jobs
--host: for host tests
--target: for target tests
All the other arguments will be passed to the run-test testrunner.

Test: ./art/test.py --host --target -j12 --run-test --gtest
Change-Id: I2fc4ebb900fb72db83165c6ae1615b8bdcfc33f4

Reduce test 051-threads to 128 threads instead of 512

It is suspected that 512 threads can timeout on loaded machines.

Bug: 36542176
Test: test/run-test --host 051
Change-Id: I8e4493d40edd74e845119b59956e4213dc750eab

Merge "Make original dex file be more deduplicated"

MIPS64: Remove memcpy() from java.lang.String.getChars intrinsic.

Test: run-test --64 --optimizing 020-string
Test: run-test --64 020-string
Test: run-test --64 --no-prebuild --optimizing 020-string
Test: run-test --64 --no-prebuild 020-string
Test: mma -j2 ART_TEST_OPTIMIZING=true test-art-target-run-test
Test: mma test-art-target-gtest -j2
Test: booted MIPS64R6 emulator.

Note: All tests run against MIPS64 QEMU.

Change-Id: I8ad3e39c66a489779476f7e9f9f88cd0c6270c20

MIPS32: Remove memcpy() from java.lang.String.getChars intrinsic.

Test: run-test --optimizing 020-string
Test: run-test 020-string
Test: run-test --no-prebuild --optimizing 020-string
Test: run-test --no-prebuild 020-string
Test: mma -j2 ART_TEST_OPTIMIZING=true test-art-target-run-test
Test: booted MIPS32R2 emulator.

Note: Tested against both the MIPS32R2, and MIPS64R6 emulators.

Change-Id: I3219bc47a3657b82c82b6ac94d0b8c82eb6f9bb9

Merge "Revert^4 "CHA for interface method.""

Remove test logic from Android.run-test.mk

As python testrunner replaces it.

This change removes the following targets:
test-art-host-run-test-[variant]

The following targets are still supported:
test-art-host-run-test
test-art-target-run-test
test-art-run-test

Each of the above targets invoke the new testrunner.
They don't pass any concurrency value to the testrunner
and the testrunner itself decides the concurrency
depending upon the type of tests.

Test: make -j test-art-host-run-test
      make -j test-art-target-run-test
      make -j test-art-run-test
      ./art/test/testrunner/testrunner.py --host
      ./art/test/testrunner/testrunner.py --target

Change-Id: I64a87d0f2e62c29b4d9c14527f9a83810ae7edc7

Merge "Saves full XMM state along suspend check's slow path."

Merge "Change 1/2 spill slots to more general number of spill slots."

Merge "Properly disassemble cmpeq for x86/x86_64"

Merge "Replace + by / to avoid vectorization of loop."

Deterministic DexCache in heap images.

Preload each slot in dex cache arrays with the candidate
entry with the lowest index.

Also fix a discrepancy where we used to keep an ArtField*
in the DexCache if we kept the declaring class rather than
based on the class referenced in the corresponding FieldId.
This can result in accessing a field through a FieldId with
an unresolved class as demonstrated by the regression test.

Test: 159-app-image-fields
Test: testrunner.py --host
Bug: 30627598
Change-Id: I34735047cb064ff462cd9859e86a61ddf8615ecd

Merge "Revert "Blacklist test that times out.""

ARM: Improve the code generated for HInstanceOf

Test: m test-art-target-run-test-009-instanceof
Test: m test-art-target-run-test-422-instanceof
Test: m test-art-target-run-test-494-checker-instanceof-tests
Test: m test-art-target-run-test-500-instanceof
Test: m test-art-target-run-test-530-instanceof-checkcast
Test: m test-art-target-run-test-603-checker-instanceof
Change-Id: Ia5e1421403605659d0f53bc794acb5e5b0af0c5e

Revert "Blacklist test that times out."

bug: 36501991

This reverts commit 882ea47197b59d812429ea2d788a96d5dcb2b77f.

Change-Id: I4f1dbad2c32422f069b1948df2152d1fb285ec3a

MIPS64: Check for MSA presence in each MSA instruction

Test: mma test-art-host-gtest

Change-Id: I123fe1e33de13af6a2b2a76f37f7ad23004a81c0

Merge "Fix bot breakages."

Change 1/2 spill slots to more general number of spill slots.

Rationale:
This prepares requesting a different number of spill slots
during SIMD vectorization.

Bug: 34083438
Test: test-art-host, test-art-host-gtest-register_allocator_test
Change-Id: I6d22966ba483deec72b5eea5061c403c12b2ada7

Merge "Add even more logging failed to copy GC error"

Replace + by / to avoid vectorization of loop.

Rationale:
Break-out CL of ART Vectorizer. Ensure loop is
not vectorized to avoid messing up checker test.

Bug: 34083438
Test: test-art-host-527-checker-array-access-split
Change-Id: Idab70a03a0f710e1e9b775891d726e1f044df916

Properly disassemble cmpeq for x86/x86_64

Rationale:
Break-out CL of ART Vectorizer.

Bug: 34083438
Test: test-art-host
Change-Id: I4027033cbe48a19c426326fc307fe4437b143d61

Merge "MIPS64: Extend instruction_set_features_mips64_test"

Merge "Add syntax for adding all class methods to profman"

Revert^4 "CHA for interface method."

This reverts commit 43e99be9db10111a2d6e094882cd06c248c69e11.

Also fix the issue that tries to change invoke-interface to
an invoke-virtual of a cross-dexfile proxy method. Added a
testcase.

Test: run-libcore-tests.sh --mode=host --variant=X64 --debug
Test: ART_TEST_JIT=true m -j20 test-art-host-run-test
Change-Id: Iacdf9d7ad93590e8163ed59d5838d70700a69018

Add even more logging failed to copy GC error

Add logging for object address and type.

Bug: 36516822
Test: m
Change-Id: I636044af673d9adc1079269b408fcdeecc33d21d

Saves full XMM state along suspend check's slow path.

Rationale:
Break-out CL of ART Vectorizer. We need to save 128-bit
of data (default ABI of ART runtime only saves 64-bit)
Note that this is *only* done for xmm registers that
are live, so overhead is not too big.

Bug: 34083438
Test: test-art-host
Change-Id: Ic89988b0acb0c104634271d0c6c3e29b6596d59b

Add syntax for adding all class methods to profman

Used to generate methods from the class list. The logic adds all
the methods if the method string is "*". The class is also added to
the profile.

Bug: 36457259

Test: mm test-art-host-gtest-profile_assistant_test -j32

Change-Id: Ibee856646b94eccc48d9ef6566475be74e77a0cd

Merge "Speed up test 080-oom-throw"

Merge "Log the errno code for failed flock() invocations"

ARM: Reduce the number of branches generated for HCondition and HSelect

Test: m test-art-target-run-test-570-checker-select
Change-Id: I87d2e87eb2fd30355101df07eb3754b013cedf63

Speed up test 080-oom-throw

Reduce heap size, optimize blowup. The test just checks that OOME
get thrown for a fe basic operations. This is the same on all heap
sizes, just slower for larger heaps.

Before:
real  1m13.247s
user  1m11.115s
sys 0m1.834s

After:
real  0m26.604s
user  0m25.470s
sys 0m0.696s

Bug: 36501991
Test: time test/run-test --host 080-oom-throw

Change-Id: Id9d1ba702949a1f6070f6f20fb457e1749f90f25

Make original dex file be more deduplicated

Now when retransformClasses is called we will use the DexCache from
the transformed class to hold the original dex file. This means that
there should be fewer redundant copies of dex files lying around from
class redefinition.

Bug: 31455788
Test: mma -j40 test-art-host-gtest
Test: ./test/testrunner/testrunner.py --host -j40

Change-Id: I60150e436d79aabb36da31ec679cb7ec4c820573

Log the errno code for failed flock() invocations

ScopedFlock was asserting the success of flock()
invocation without logging errno. This CL adds the errno log
before the abort.

Test: m
Bug: 36369345
Change-Id: I24209784b795c37463fb790fcd0793369eeab492

Fix bot breakages.

bug:36169180
test: testrunner.py -b
Change-Id: Ibfe035d5d7fb18d95d4b5a32e1e68967f1889bdd

Merge "Blacklist test that times out."

Blacklist test that times out.

bug: 36501991
Change-Id: Ibfa6541fdea781fe9862049eccbcacb9b174553f

Merge "ARM: Avoid branches to branches"

Merge "Assign pointer to quick code for copied methods"

Merge changes I17d48a60,I90bc4912

* changes:
  ART: Add dex2oat return code for runtime failure
  ART: Add dex2oat return codes

Merge "Run gtests in art-test"

Merge "Avoid use of pthread_getschedparam (except on the Mac)"

Merge "tools: Add golem/build-target and golem/env scripts."

Avoid use of pthread_getschedparam (except on the Mac)

Squashed commit of two cherry-picks:

    During Thread::DumpState the pthread may have exited.
    We have the tid, use that instead.

    Test: m -j50 test-art-host -k
    Test: adb shell kill -3 `pid system_server`
    Bug: 36445592

    (cherry picked from commit 08e762782a1e352c329945e0df3f35da7e71ad8a)

    Restore broken code for the Mac

    Test: m -j48
    Bug: 36445592

    (cherry picked from commit 4b16f0b860f4d59601706305b2949b98db586a47)

Test: m -j48 && flashall && m -j8 test-art-host
Bug: 36445592
Change-Id: I96d51614f569338d85de0fd59188445df3289dfd

Merge "Downgrade duplicate method DCHECKs to WARNINGs"

Merge "ART: Remove com.android.dex.Dex from well_known_classes."

Merge "Some more blacklisting."

Some more blacklisting.

bug:36467228
bug:36365552
Change-Id: Ic7c21dc229b55a8ef3f8aec99e835ebb1e9ca312

MIPS64: Extend instruction_set_features_mips64_test

Added "mips64r6" case and compared against "default".

Test: mma test-art-target-gtest-instruction_set_features_mips64_test

Change-Id: I1937912d69393fdd69a0e9779b597393dfa0bacf

Merge "Remove extra comma in knownfailures.json"

Remove extra comma in knownfailures.json

Test: testrunner.py --host
Change-Id: I4e374a419fa2fed19bbd1b86ce13f53c71d7b8bb

ART: Add dex2oat return code for runtime failure

Add a return code for failure to create a runtime. Add a test.

Bug: 36336122
Test: m test-art-host-gtest-dex2oat_test
Change-Id: I17d48a6052ead7a138d77c87832968a7294c3593

Merge "jdwp: switch to SOCK_SEQPACKET."

ART: Add dex2oat return codes

Refactor exit codes for dex2oat and export them. In preparation for
installd usage.

Bug: 36336122
Test: m test-art-host
Change-Id: I90bc49128886719f7595105ecc40405ddf821850

Merge "Fix missing comma in knownfailures.json"

Fix missing comma in knownfailures.json

Test: testrunner.py ...
Change-Id: I3c23b9be732074cacde8db75c9da0da4ead98ac9

jdwp: switch to SOCK_SEQPACKET.

When sending the process's PID to adbd, we use
TEMP_FAILURE_RETRY(send(...)), and check if the result is non-negative.
However, this could theoretically return short, leaving adbd waiting
for the rest of the PID. Switch the socket to a SOCK_SEQPACKET to
sidestep all of this.

Bug: http://b/36411868
Test: adb jdwp
Change-Id: I0ec75912ac579868f77554e3c6db23c7ab95a34c

Merge "Disable the use of inline caches in AOT mode"

Disable the use of inline caches in AOT mode

Until we figure out why we see the crashes in b/36371709.

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

Downgrade duplicate method DCHECKs to WARNINGs

These DCHECKs were causing spurious failures when testing with
debug ART. Duplicate methods could cause multiple methods in
a class to have the same name and signature. Overriding these
methods could make it so that methods are owned by the wrong type.
Though the corrupted methods arrays could be a cause for concern
(hence the DCHECKs) but in the most common case it should be fine
as the duplicated methods cannot be invoked.

As both of these situations can be caused by some versions of
proguard and are present in some released apps, we are downgrading
these to warnings even on Debug builds.

Bug: 32549051
Bug: 32546154
Bug: 36446088

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

Merge "Revert "Revert "Introduce a number of MSA instructions for MIPS64"""

ART: Remove com.android.dex.Dex from well_known_classes.

Unused after change 6b2dc3156a2140a5bfd9cbbf5d7dad332ab5f5bd.

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

Merge "Don't special case kRuntimeISA for getting image info."

Merge "Fix the artificial dependency in ARM/ARM64 SystemArrayCopy intrinsics."

Merge "Refactor SystemArrayCopy intrinsics."

Merge "More blacklisting."