Revert^2 "OatFileAssistant: look at vdex file for IsDexOptNeeded"

This reverts commit ba037b121edc7259b3fa5607e6730402ce2ca61e.
Test: m test-art-host, with new oat file assistant test added.
Bug: 34339100

Change-Id: I1ab9ec9f6ec299be7b8144612b71e6054d84412c

Check vdex file is valid in VdexFile::Open.

Test: m test-art-host, including new vdex_file_test.
Bug: 34339100
Change-Id: I0fe01760779f3876d6790a5443785a4f7289a717

Merge "Make wording of comment more precise."

Merge "MIPS32R6: Improve PC-relative string/class loads and invokes."

Make wording of comment more precise.

Test: mma -j40 test-art-host
Change-Id: I2501d183669d2981f230876366e472acbe02cfb4

Merge "Remove Deoptimization code from class transformation."

Merge "Add to time to suspend in FlipThreadRoots to histogram"

Merge "Revert "Revert "Make --debuggable rely on JIT code."""

Merge "Blacklist test now that debuggable means JIT."

Revert "Revert "Make --debuggable rely on JIT code.""

Fixed by https://android-review.googlesource.com/#/c/330165/

This reverts commit 81356645157af44152c7b7db383596b5cf3479b5.

Change-Id: Ifb74e1cc90ab6dea621f7f54a00b540d6ccd0cf6

Merge "Print to System.err in test 909 to avoid output races."

Merge "Revert "Make --debuggable rely on JIT code.""

Revert "Make --debuggable rely on JIT code."

Breaks 137-cfi with ART_TEST_RUN_TEST_DEBUGGABLE=true

This reverts commit a0619e25aacf8b8074132f4951f75fdbcfd42925.

bug: 28769520
Change-Id: Ifd7b8fc7c9b72c0a523fd57c4b1b80edd3547caa

Remove Deoptimization code from class transformation.

Since we removed the current_method from the compiled code we don't
need to deoptimize all frames anymore.

This is a partial revert of commit dba614810.

Bug: 32369913
Bug: 33630159

Test: ART_TEST_TRACE=true \
      ART_TEST_JIT=true   \
      ART_TEST_INTERPRETER=true mma -j40 test-art-host

Change-Id: I44a6dd89e1d96bd8c82c2c24a2f42fef023a80be

Merge "Merge layout-speed compiler filter into speed-profile."

Print to System.err in test 909 to avoid output races.

Throwable.printStackTrace() prints to System.err, so print
all other output to System.err as well to avoid races like

    #################### info
    # Tests jvmti plugin attaching during live phase.
    #################### diffs
    --- expected.txt 2017-01-13 18:22:13.897224469 +0000
    +++ output.txt 2017-01-30 17:51:21.091550290 +0000
    @@ -6,6 +6,6 @@
     Goodbye!
     Hello, world!
     java.io.IOException: Process is not debuggable.
    +Goodbye!
      at dalvik.system.VMDebug.attachAgent(Native Method)
      at Main.main(Main.java:27)
    -Goodbye!
    ####################

Test: m test-art-host-run-test-909-attach-agent
Change-Id: I956bd24db056154b938568c156df80d4ff56ba11

MIPS32R6: Improve PC-relative string/class loads and invokes.

Use PC-relative addressing on MIPS32R6 instead of
HMipsDexCacheArraysBase and allow such PC-relative
addressing in presence of irreducible loops.

Also save a couple of instructions when handling
string and class loads from bss.

Test: test-art-host-gtest
Test: booted MIPS32R2 in QEMU
Test: "make -j1 ART_TEST_DEFAULT_COMPILER=false ART_TEST_OPTIMIZING=true
       ART_TEST_INTERPRETER=false ART_TEST_JIT=false
       ART_TEST_PIC_TEST=true test-art-target-run-test"
Test: booted MIPS64 (with 2nd arch MIPS32R6) in QEMU
Test: "make -j1 ART_TEST_DEFAULT_COMPILER=false ART_TEST_OPTIMIZING=true
       ART_TEST_INTERPRETER=false ART_TEST_JIT=false
       ART_TEST_PIC_TEST=true test-art-target-run-test32"

Change-Id: I5d0fcbf271541294a3d4479987d52e2aaff084d9

Blacklist test now that debuggable means JIT.

Change-Id: I658dea8e2b7327ad9631edfe62b45a0c7fcfc845

Merge "Make --debuggable rely on JIT code."

Merge "Add ART_TEST_RUN_TEST_IMAGE."

Merge "MIPS: Refactor code for unresolved field entrypoint."

Merge "Hash-based dex cache type array."

Make --debuggable rely on JIT code.

Removes -Xfully-deoptable in the process, which was added as a
temporary workaround until this CL.
Partial revert of https://android-review.googlesource.com/#/c/302232/

Makes things consistent with existing infrastructure:

- Parse the --debuggable from the compiler options, just like
  --compiler-filter.

- Add DEBUG_JAVA_DEBUGGABLE, passed by the zygote, for debuggable apps.

- Java debuggable now solely relies on JIT for simplicity.

- Debugging under userdebug for non-java-debuggable apps is still
  best effort.

Test: test-art-host, jdwp

bug: 28769520
Change-Id: Id0593aacd85b9780da97f20914a50943957c858f

Add ART_TEST_RUN_TEST_IMAGE.

To prepare disabling this run on the no-image bot config.

Change-Id: Iebd51761ebde42d07beb7ecce3b84e738d03afe3

Hash-based dex cache type array.

Test: m test-art-host (Interpreter, Optimizing, JIT)
Test: m test-art-target on Nexus 6P (Interpreter, Optimizing, JIT)
Test: Nexus 6P boots
Test: m valgrind-test-art-host
Bug: 30627598
Bug: 34659969
Bug: 30419309
Change-Id: Ic00eda89e58088a3573fc9ec0ad04c0e69e161d1

Merge "Make no-relocate the default."

Merge "Revert "ART: Give JIT thread pool workers a peer""

Revert "ART: Give JIT thread pool workers a peer"

This reverts commit 1ee6dd71e1b9e51e15aae4d3bc95d799f11dbd04.

Has unintended consequences that need to be discussed.

Bug: 31684920
Change-Id: If0ff1b5d7473f98cb3a0efc3b4bb2c0f96ad684d

Merge "ART: Give JIT thread pool workers a peer"

Merge "ART: Add ThreadPool mode that creates peers"

Merge "Make run-test --jvm set DEX_LOCATION"

Add to time to suspend in FlipThreadRoots to histogram

Add the time to suspend to the suspend_all_historam_ to determine if
GC pauses are caused by GC or bad mutator activity.

Also exclude ThreadFlipBegin time being counted as paused since it
happens before threads are suspended.

Test: test-art-host

Change-Id: Ia55db0ebbdc7ba6f23d7212c0e42c7806b297c2b

Merge "Remove alignment bits in between stack maps"

ART: Give JIT thread pool workers a peer

To allow reporting those threads, give JIT pool threads a peer.
This is necessary as the compiler may load classes.

Bug: Bug: 29547798
Bug: 31684920
Test: m ART_TEST_JIT=true test-art-host
Change-Id: I7c8e44132475f38995542da76285ca3dd63c559a

Make run-test --jvm set DEX_LOCATION

Test: ./test/run-test --jvm 934
Change-Id: Ib8848cf4d29e1357b696e384a7943b14d8fc4a4f

MIPS: Refactor code for unresolved field entrypoint.

This is a follow-up to
https://android-review.googlesource.com/#/c/325423/.

Test: booted MIPS32R2 in QEMU
Test: test-art-target-run-test-optimizing in QEMU
Test: booted MIPS64 (with 2nd arch MIPS32R6) in QEMU
Test: test-art-target-run-test-optimizing
      (MIPS64R6 and MIPS32R6) in QEMU

Change-Id: Ie663ecf0489e7b182434708bef71686df5f37273

ART: Add ThreadPool mode that creates peers

Add a mode where worker threads in a thread pool will get a Java
peer. Add a test.

Bug: 29547798
Bug: 31684920
Test: m test-art-host-gtest-thread_pool_test
Change-Id: I3654cc2be1294a79881b6ac9f84445a1e7f24b70

Merge "Ensure ClassPreDefine returned dex file is on the Classpath"

Merge "Avoid read barriers for ArtMethod::GetDexFile"

Ensure ClassPreDefine returned dex file is on the Classpath

Test: mma -j40 test-art-host
Change-Id: Icf70a78f3a1149d0e5bf9aa64f74f2ca8d025802

Merge "ART: Add OnLoad system classloader search support"

Remove alignment bits in between stack maps

Saves 0.65% of boot.oat size, probably similar on apps. Added
BitMemoryRegion to avoid requiring adding state to StackMap. Added
test to memory_region_test.

Test: clean-oat-host && test-art-host

Bug: 34621054

Change-Id: I40279c59e262bd5e3c6a9135f83e22b5b6900d68

Merge "Don't use VIXL's temp registers in LocationsBuilderARM64::HandleFieldGet."

ART: Add OnLoad system classloader search support

Add support for extension of the system classloader search path
during the OnLoad phase. Add test.

Bug: 34359699
Test: m test-art-host-run-test-936-search-onload
Change-Id: Ie0a5b4bb77999dfae87f955dde40b73ffe053b8e

Make no-relocate the default.

With the move to not support relocatable oat files, testing "relocate"
does not make sense anymore (the runtime would just discard the prebuild
oat file and generate a new one in dalvik-cache).

Just make no-relocate the default for now. We can remove relocation types
once Richard is done removing relocation.

Test: test-art-host test-art-target
bug:33192586

Change-Id: If39a0ad63451c68d4cfbbb26b767ad98b97d208e

Merge "Add ARM support for `floor` and `ceil` intrinsics."

Merge "MethodHandles: Support and tests for revealDirect / reflectAs."

Merge changes from topic 'VIXL-update-270117'

* changes:
  ARM: VIXL32: Keep jump table base label within range of adr.
  ARM64: VIXL: Fix breaking changes to ternary operator with Register inputs.

MethodHandles: Support and tests for revealDirect / reflectAs.

We need runtime support to materialize a Constructor, Field or
Method object from a MethodHandle and its associated ArtField
or ArtMethod pointer.

An alternate approach might be to have all handles hold a reference
to their associated Field or Method object (in addition to the
raw ArtField or ArtMethod) but that seems unnecessary given that
these APIs are not expected to be called often.

Test: test-art-host

Change-Id: I9450706b9c30359edebf2e8a7afbc8d89bd68b26

Don't use VIXL's temp registers in LocationsBuilderARM64::HandleFieldGet.

Before this CL, when emitting a volatile field load with a
large offset, it was possible to deplete the pool of VIXL
ARM64 temporary registers (IP0, IP1) in the concurrent
collector configuration.  To avoid this, we now request a
temporary register from the register allocator instead.

Test: m test-art-target-run-test-635-checker-arm64-volatile-load-cc
Bug: 34726333
Change-Id: Idf73a0306142c6133e259783aacaf7ad5401a2fd

Add ARM support for `floor` and `ceil` intrinsics.

Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-host
Test: ART_USE_VIXL_ARM_BACKEND=true m test-art-target

Change-Id: Ia6fe41ea912e32a73d78629c616bc64ad1ecfe66

Merge "Revert "Revert "CHA for abstract methods."""

ARM: VIXL32: Keep jump table base label within range of adr.

Macro assembler instructions between adr and the jump table base label
could cause literal pool or veneer emission, which would push the jump
table start out of adr range. Such a large emission is highly unlikely,
but possible, so prevent this by using an ExactAssemblyScope.

Test: export ART_USE_VIXL_ARM_BACKEND=true && test-art-target
Change-Id: I46a038e452e197d73206b415376b5f2246388317

ARM64: VIXL: Fix breaking changes to ternary operator with Register inputs.

Test: mma art
Change-Id: I33d1e05e47f337f1a9271b35dba9227057cda096

Merge "Shoot in the dark to try and fix mac build."

Shoot in the dark to try and fix mac build.

Use MACRO_LITERAL in assembly macros instead of LITERAL.

Change-Id: I9fd921e4ae59550cfcc1f573be1798ea9823a0cb

Merge "Add per array size allocation entrypoints."

Revert "Revert "CHA for abstract methods.""

This reverts commit 8ebc8bf055e8bd8f6f167e65a69cf4dae136db55.

When we set the single-implementation method for an abstract method, it
need to be protected by the cha-lock.

Test: new testcase and ART_TEST_JIT=true m -j20 test-art-host-run-test
Change-Id: I66acb20ffa7e49dd9c391f001e3bb52f961872e4

Merge "Pass the correct name to ClassFileLoadHook"

Pass the correct name to ClassFileLoadHook

We were passing the dotted name instead of the slashed one.

Test: ./test/run-test --host 937
Change-Id: I4f5f4128cd87ed3d592b7af5724554160f15bdae

Avoid read barriers for ArtMethod::GetDexFile

Shows up in pmd benchmark from Class::FindDeclaredDirectMethod and
Class::FindDeclaredVirtualMethod. There are still calls to
IsProxyMethod that could probably be eliminated.

ReadBarrier::Mark goes from 12.39% to 3.45% according to perf.

Test: test-art-host

Change-Id: I6a4f2fa2d68bf5f393f83b9b70e8d6fcc9dbdaa2

Merge "Re-enable tests 934 & 935"

Merge "You shall know your arm32 calling conventions."

You shall know your arm32 calling conventions.

64bit values are passed starting an even register.

test: ./art/test/run-test --no-relocate 529-checker-unresolved

Change-Id: Ic3012f34898702b48fdab4d7a77dfdd4c2a1d916

Re-enable tests 934 & 935

There were two issues causing problems with these tests.

1) The loading of classes could occur prior to the transformer being
   enabled. This was fixed by putting the classes under test into a
   different ClassLoader and disabling app-images.

2) DexToDex compilation was devirtualizing some calls meaning that
   reordering the methods was causing incorrect dispatch.

This reverts commit 50cc0cceb5ca0abd4fff2cb3467ea74e7590cb10.
This reverts commit e9f36b7cb42348ef5c4eb3c88bd3cfdd56390a3e.
This reverts commit 1e7e96e4a66f668255a1683b837863228489faf9.

Test: ART_TEST_INTERPRETER=true \
      ART_TEST_JIT=true \
      mma -j40 test-art-host

Test: ART_TEST_INTERPRETER_ACCESS_CHECKS=true \
      ART_TEST_RUN_TEST_NO_DEX2OAT=true \
      ART_TEST_RUN_TEST_RELOCATE_NO_PATCHOAT=true \
      ART_TEST_RUN_TEST_NO_RELOCATE=true \
      mma -j40 test-art-host-run-test-934-load-transform \
               test-art-host-run-test-935-non-retransformable

Change-Id: Ia600ba37d45a5f9489b03c7f4d143849c887b8a7

Merge "Make verify-profile not look at the profile."

Merge "Avoid accessing the heap without mutator lock in Monitor::Lock."

Make verify-profile not look at the profile.

It doesn't work well with vdex. In follow-up CLs,
I'll just remove/rename  the filter.

For the record, compiling sheets on an angler:
verify-profile: dex2oat took 5.383s (6.672s cpu)
vdex: dex2oat took 1.686s (1.562s cpu)

bug: 34715556
bug: 34691602
bug: 34691854

Test: test-art-host
Change-Id: I286637a3d7ebe1b18985e96845b0b8c4f67a1a39

Merge "Forgot stack alignment on x86."

Forgot stack alignment on x86.

test: ART_TEST_RUN_TEST_NO_RELOCATE=true m test-art-host test-art-target
Change-Id: I1996c0d85228e5cad7bee4387260821d9d209b11

Merge "Forgot this change from last commit."

Forgot this change from last commit.

Test: 529-checker-unresolved
Change-Id: I14b6a8d81a7a885cff36d3361f83d891e3ff9956

Merge "ARM: Update `ArmInstructionSetFeatures` to track ARMv8-A."

Merge "Refactor code for unresolved field entrypoint."

Merge "Also blacklist 934 and 935 for no-relocate."

Merge "Update vixl/arm32 backend after new allocation entrypoint change."

Also blacklist 934 and 935 for no-relocate.

test: test-art-host
Change-Id: I0e88ffffa25b5174eecfc4e0bfebf43811d33959

Refactor code for unresolved field entrypoint.

- Do macro magic to avoid source code duplication.
- Do not fetch the referrer from the assembly, but
  from the C entrypoint instead.

Test: test-art-host test-art-target

Change-Id: Ib139c94bc8f74686640cad538ba75dc56fa00e1d

Update vixl/arm32 backend after new allocation entrypoint change.

Fix after https://android-review.googlesource.com/#/c/325419/

Spotted by artem.serov@linaro.org. Thanks!

test: test-art-target with vixl/arm32
Change-Id: I5fdb4836c80f98696b07a06f52919f33a6284fbc

Add per array size allocation entrypoints.

- Update architectures that have fast paths for
  array allocation to use it.
- Will add more fast paths in follow-up CLs.

Test: test-art-target test-art-host.
Change-Id: I138cccd16464a85de22a8ed31c915f876e78fb04

Merge "Revert "Disable optimization of Atomic::LoadJavaData for x86 targets.""

Merge "Revert "Ignore warning for instantiation after specialization.""

Revert "Disable optimization of Atomic::LoadJavaData for x86 targets."

This reverts commit fbd5118c1c36900910cabe89e3a73d598160e629.

Reason for revert: <INSERT REASONING HERE>

Change-Id: If7aac5e7f4e5376167e896c40f8bf30287f81dd1

Revert "Ignore warning for instantiation after specialization."

This reverts commit 445584d3d1f8e3a9a7d12eded3b2dce5f8691d8c.

Reason for revert: <INSERT REASONING HERE>

Change-Id: I911049f64c4e963cfa47fd8bc7f16ed14e25b057

Merge changes from topic 'clang-3625443'

* changes:
  Disable optimization of Atomic::LoadJavaData for x86 targets.
  Ignore warning for instantiation after specialization.

Merge "Temporarily disable tests 934 & 935 in fallback and interp_ac config"

Temporarily disable tests 934 & 935 in fallback and interp_ac config

These should have been with the earlier disable but I missed them.

Test: build change
Change-Id: I491f14d5666fff30dbadcf9105a96748a4aaaf5e

Merge "Add capability checks to all functions."

Avoid accessing the heap without mutator lock in Monitor::Lock.

ScopedThreadStateTransition doesn't do annotations and we are
accidentally accessing the heap without holding the mutator lock in
Monitor::PrettyContentionInfo. Use ScopedThreadSuspension instead.

Bug: 34674595
Test: test-art-host

Change-Id: If3ec86429ff99405ab14b3588f192ab1c0472cc8

Merge "Make jvmti tests runnable on the RI again."

Add capability checks to all functions.

Also ensure that we declare all capabilities we support.

Bug: 31684920
Bug: 31455788
Test: mma -j40 test-art-host
Change-Id: Ic3f251f451cf6dad9934109594652f23d228d0f4

Merge "Minor edits to make nightly reports."

Merge "[ART] Prepare profile streaming for framework to use."

Merge "Refactor and optimize memory region bit functions"

Merge "ART: Remove experimental flags for plugins and agents"

Merge "Preserve verification invariant after compile-time verification."

Merge "Temporarily disable tests 934 & 935 for interpreter and jit."

Temporarily disable tests 934 & 935 for interpreter and jit.

These need some additional support from the ClassPreDefine event which
will be added shortly.

Test: mma -j40 test-art-host ART_TEST_INTERPRETER=true ART_TEST_JIT=true
Change-Id: Id60da22d96a9f6c9084fdf6bf810874284d5d2a8

Make jvmti tests runnable on the RI again.

Test: ./test/run-test --host --jvm 932
Change-Id: Ia35c9e2b4f93884956993dc9c5e4d041344c9680

ART: Remove experimental flags for plugins and agents

Remove experimental flags guarding plugins and agents.

Partial revert of commit 7233c7e752c0d26387d143ee74420e9cd1f09390.
Partial revert of commit 185d134a3b43ab7529053e965917e0fa74bceba4.

Bug: 31455788
Test: m test-art-host
Change-Id: I11a19f24b88ced8da7a28e2cb7a2f602aab884a8