Fix RegType invariant checks.

Move the checks from the base constructor to constructors of
the most derived classes. While in the base constructor, the
vtable points to the base class implementations of virtual
function, so we were not checking what we intended. And when
the check failed, we tried to call a pure virtual function
RegType::Dump() and crashed while constructing an abort
message.

Also, avoid an unnecessary std::string construction in
RegType::CheckInvariants() as StringPiece can be compared
with const char* directly.

Test: testrunner.py --host
Bug: 32548008
Change-Id: Ice7ca298132f87f8b1671b6578aeec221f0e7930

Merge "Introduce EnvUsePosition for liveness analysis."

Merge "Revert "ARM: VIXL32: Use VIXL backend by default.""

Revert "ARM: VIXL32: Use VIXL backend by default."

Revert while investigating.

bug:35977033

This reverts commit e6316892821287b1d1906b9962eae129fbdc37be.

Change-Id: I51e24a6e539072a6d0d470dfe41855a4847f3e96

Merge "Added new -v option to dexlayout to verify output dex file."

Added new -v option to dexlayout to verify output dex file.

Passing -v will cause dexlayout to do an IR level comparison of the
output with the original input dex file. This checks that the data in
the dex files are the same, but allows for different offsets since the
output dex file may have a different layout.

Test: mm test-art-host
Bug: 36107940
Change-Id: If75a93973ffdd2d91111727f089713c800d8cee8

Merge "Fix issues due to updated toolchain."

Merge "Testrunner: Print information about the total tests run"

Introduce EnvUsePosition for liveness analysis.

Normal and environment use positions are held in separate
lists and the code never mixes them together. By using two
separate classes, we can reduce complexity and avoid an
unnecesary data member, reducing the memory usage.

Tracking allocations for a certain big app, the peak arena
memory usage is
  before:
    MEM: used: 79245960, ...
    SsaLiveness    31221600
  after:
    MEM: used: 78754024, ...
    SsaLiveness    30729664

Test: testrunner.py --host
Bug: 34053922
Change-Id: I02d3c9f564bbe3b1da0e03c33cf7c0f810f235dc

Merge "Revert^5 "Hash-based dex cache type array.""

Testrunner: Print information about the total tests run

Bug: 35985100
Test: manual
Change-Id: I879637e8c06ca401cd2a9c10111b4408300a7fcd

Merge "Revert "Hash-based DexCache field array.""

Revert^5 "Hash-based dex cache type array."

For app images, ImageWriter does not add boot image
classes to the app image class table even though it
keeps them in the dex caches. The reason for that is
unknown, the code looks OK.

Bug: 34839984
Bug: 30627598
Bug: 34659969

Also reverts "Improve debugging output for a crash."

This reverts commits
    bfb80d25eaeb7a604d5dd25a370e3869e96a33ab,
    8dd56fcb3196f466ecaffd445397cb11ef85f89f.

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

Revert "Hash-based DexCache field array."

Reverting to allow rebasing the revert
    https://android-review.googlesource.com/351689
without too many conflicts.

Bug: 30627598

This reverts commit 1aea3510b8dd0c512cec61c91c5ef1f1e5d53d64.

Change-Id: I4af65e9f41c8bad8106c028947eca7c5a9534c53

Merge "ART: String intrinsics for Mterp interpreter"

Merge "Fix dex file verification check for dexlayout output file."

Fix dex file verification check for dexlayout output file.

For debug builds, we want to run the output dex file through the dex
file verifier as a sanity check. The initial check was actually checking
the input dex file again...

Bug: 36107940
Test: mm test-art-host
Change-Id: Ibd434d67f661c325a443d4ca9cced481e7a9b02c

Fix issues due to updated toolchain.

Test: mma -j64 showcommands > make_log in platform/art. Also able to
produce abi dumps using header-abi-dumper.

Bug: 35845221

Change-Id: I569a1abff8df86da65af869b1c766ddeebf59ff4

Merge "Ensure we have the correct thread when allocating obsolete methods."

Ensure we have the correct thread when allocating obsolete methods.

We were using the incorrect thread to allocate memory for obsolete
methods. This could cause DCHECK failures as one tries to lock a mutex
with a non-running thread.

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

ART: String intrinsics for Mterp interpreter

Adds the most common java.lang.string intrinsics.  Includes change
to jvalue handling to zero-exend setting of chars and booleans to
64 bits (aligns with current sign-extension of shorts and ints).

Bug: 30933338

Benchmarks:
  2x boost for Caffeinemark String
  11% improvement for Dhrystone

Test: ART_TEST_INTERPRETER=true m test-art-host
Test: ART_TEST_INTERPRETER=true m test-art-target (Bullhead)

Note: Added intrinsics have existing test coverage via
082-inline-execute and 123-inline-execute2.

Change-Id: I64b35b1d7bdfe14da0c662594c0edf5cde667277

Merge "ARM: VIXL32: Improve BoundsCheck for constant inputs."

Merge "Add section size statistics"

Merge "ARM: Generate UBFX for HAnd"

Add section size statistics

Add an option to display the size of dex file sections for easier
analysis.

Bug: none
Test: test-art-host

Change-Id: I779f776657ad79e7a50c18c2b9486e6778c70f13

ARM: Generate UBFX for HAnd

Test: m test-art-target-run-test-538-checker-embed-constants
Change-Id: I8e6af76b99543331e8ffec01bd8df3f09890708e

Merge "Hash-based DexCache field array."

ARM: VIXL32: Improve BoundsCheck for constant inputs.

Test: mma test-art-host && mma test-art-target

Change-Id: I05051c03dbd3684c674096def84020494d28364b

Hash-based DexCache field array.

Test: m test-art-host, both AOT and interpreter
Test: m test-art-target, both AOT and interpreter
Test: m valgrind-test-art-host
Bug: 30627598
Change-Id: If992f091aadd862d17b09928d21659573dd285a0

Merge "When profiles are requested, also generate then for non-prebuild."

When profiles are requested, also generate then for non-prebuild.

Test: test-art-host-run-test-debug-no-prebuild-optimizing-no-relocate-ntrace-cms-checkjni-picimage-npictest-ndebuggable-638-checker-inline-caches
Change-Id: Ib9243d453b723eda894f98a0fd25cea155251003

Merge "Add dex file verification check for dexlayout output file."

Merge "Catch signals that happen inside a fault handler."

Merge "ART: Make test 924 more robust"

Add dex file verification check for dexlayout output file.

For debug builds, run the output dex file of dexlayout through the dex
file verifier as a sanity check.

Bug: 36107940
Test: mm test-art-host
Change-Id: I2b556db32cfd360cc4fe8d1d080508deebd27aa0

Merge "Use offline inline caches during AOT compilation"

Use offline inline caches during AOT compilation

Also:
- extend the testing script to understand profile when --profile is
passed
- filter inline cache types which are not loaded by the caller class
loader

Test: m test-art-host-run-test-638-checker-inline-caches
Bug: 32434870
Change-Id: Ifcc27b3cebc79b84617412aaae64a73324151b55

Merge "Do not allow input-vdex with dexlayout."

Do not allow input-vdex with dexlayout.

input-vdex and dexlayout are incompatible with each other.

bug: 36084905
bug: 36076104
bug: 36059948
bug: 35872504

Test: device boots, vdex works on OTA, input-vdex is not passed
      for speed-profile.

Change-Id: I630c90acf0a8f49f50b7909e2dec995dd4a4d13f

ART: Make test 924 more robust

To avoid rare scheduling issues, add a loop checking the Java-side
Thread state.

Bug: 35987799
Test: m test-art-host-run-test-924-threads
Change-Id: I0433d6f09684579ca1c03d335674a8013da05034

Merge "Update UnstartedRuntime for System.identityHashCode."

Merge "ARM(64): Improve the code generated for HSelect"

Merge changes Ia26b07f0,Id3d2758c

* changes:
  Revert "Revert "Use the holder's gray bit in Baker read barrier slow paths (ARM, ARM64).""
  Revert "Revert "Use the "GC is marking" information in compiler read barriers (ARM, ARM64).""

ARM(64): Improve the code generated for HSelect

Test: m test-art-target-run-test-566-checker-codegen-select
Test: m test-art-target-run-test-570-checker-select
Change-Id: If0140892303490701782df9a818e6d8346bf3d6c
Signed-off-by: Anton Kirilov <anton.kirilov@linaro.org>

Merge "Track libcore change 2c0c8889ff415431e3c2d6d5ac95748f8b284384."

Merge "Don't inline methods that throw in graph with irreducible loops."

Merge "codegen_test: Add missing dependency."

Merge "Revert "Stop interpreter from accessing code items of compiled code.""

Revert "Stop interpreter from accessing code items of compiled code."

Bug: 35800981

This reverts commit df79ddb545f0d6e71d6eebb9cb94aa6916351ee9.

Change-Id: I04b1cb8d002ca330c0aa6e68b431c7f80c2779d6

Merge "Remove --include-patch-information option from dex2oat."

Merge "Stop interpreter from accessing code items of compiled code."

Merge "ART: Add field offset caching"

Merge "Fix duplicate classes check to iterate over classes properly."

Merge "Modify invoke interface trampoline to pass interface method."

Modify invoke interface trampoline to pass interface method.

This avoids touching the code item in the case that the interface method
is resolved and in the dex cache. If it's not, the trampoline will still
have to go to the code item to try to resolve it.

Bug: 35800981
Test: mm test-art-host
Change-Id: Ia0e1d23429b3b928bcec727f4f2f16b7834408e5

Stop interpreter from accessing code items of compiled code.

The ArtInterpreterToCompiledCodeBridge accesses the code item in a
number of places to handle argument marshalling. However, the code item
of a compiled method should have no need to be accessed by the runtime
at all, since the code has been compiled. By removing these accesses,
there is a drop in the memory footprint of the dex file, since these
code items remain untouched by the runtime.

Bug: 35800981
Test: mm test-art-host
Change-Id: Ib7d29c17e80b1690aa819d083f5b12739492ebd6

Merge "ART: Refactor ObjectTagTable"

Merge "MIPS: Optimize code generation of check-cast and instance-of."

Catch signals that happen inside a fault handler.

Unblock some signals (SIGABRT, SIGBUS, SIGSEGV) that could happen
inside of the ART internal fault handlers, to report crashes inside of
the signal handler. Because we can't use sigaction to change the
handler when this happens, because it modifies global state, add a new
member variable in Thread to track whether a call to the fault handler
is reentrant or not.

Remove the old nested signal implementation that attempted to do this.

Bug: http://b/35853436
Test: changed the #if 0 to #if 1, ran a dummy process that
      threw a NullPointerException, inspected logcat
Change-Id: I04bb4a09433c6817933d64ec681ec433b528f2a5

Merge "Allow store elimination for singleton that's returned"

Merge changes I2f8edd95,I17d67431

* changes:
  Allow building Android builds locally
  Allow test runner to build master-art-host targets

Allow building Android builds locally

Test: manual
Change-Id: I2f8edd952287b5e4b6bc9f87df91d322af5c4a5a

Allow store elimination for singleton that's returned

Allow store elimination for singleton that's visible after method return
or deoptimization. Add additional detection for keeping stores for such
singletons at block merge/deoptimization point.

Bug: 35745320
Test: m test-art-host
Change-Id: I8a75a304491dafaeb689787402afa3d7468e3789

Merge "Inlining a few small methods based on profiling dex2oat with perf."

Merge "Interpreter intrinsics"

Fix duplicate classes check to iterate over classes properly.

Previously would iterate in class index order, but classes are not
organized alphabetically. This changes the check to iterate over the
classes's type indexes, which are in alphabetical order.

Unit test to come.

Bug: 35477728
Test: mm test-art-host
Change-Id: Iddaad84c30be042d09dde56c998221fd07f04ab6

Inlining a few small methods based on profiling dex2oat with perf.

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

Merge changes I16000c0c,I325003cb

* changes:
  Add two locks to expected_mutexes_on_weak_ref_access_.
  Revert "Add jdwp event_list_lock_ to expected_mutexes_on_weak_ref_access_."

Interpreter intrinsics

First of several intrinsic CLs.  This one provides the basic
mechanism plus a handful of the most simple examples.  Note that
Intrinsic support is limited to Mterp both to keep the switch
interpreter as a pure reference, and to avoid mixing debugging
and intrinsics.

Bug: 30933338

Test: ART_TEST_INTERPRETER m test-art-host
Test: Note: existing 082-inline-execute, 123-inline-execute2
Test:   cover intrinsics enabled by this CL.

Change-Id: I9d79e23a84bf00cadb54e1db52d8ed193bbf8887

Update UnstartedRuntime for System.identityHashCode.

The method is not native anymore.

Test: unstarted_runtime_test
Change-Id: Ib5cc378f05e9b6ee50168129e855803773a9c5e6

Merge "Improve isunit computation, some tests edits needed for SIMD."

Merge "Improve debugging output for a crash."

Merge "Forgot this line for blacklisting."

Forgot this line for blacklisting.

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

Improve debugging output for a crash.

Bug: 34839984
Test: m test-art-host-gtest
Test: testrunner.py --host
Change-Id: I7cf1e33c42cfae75d7a132d66a4d293173ed9b6c

codegen_test: Add missing dependency.

The lack of this dependency causes two categories of errors :
- Running without the simulator on clean builds, since the dlopen
  in code_simulator_container.cc fails (and that isn't fatal).
- Spurious crashes whilst running against an old version of
  libartd-simulator, coupled with the fact that the new / free
  pair are in different shared libs.

Test: make -j32 test-art-host-gtest-codegen_test64
Change-Id: Ifd59688e62e248fcaf06ef14350eec989b853a61

Merge "Blacklist test for NON_BAKER_READ_BARRIER."

Blacklist test for NON_BAKER_READ_BARRIER.

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

Merge "Experiment with ulimit settings to see if it fixes test failure."

Track libcore change 2c0c8889ff415431e3c2d6d5ac95748f8b284384.

Update the size of Class' vtable.

Test: make test-art-host
Change-Id: I6613a3f1924a2347e0e4f9c4db31d126ab4deacc

Remove --include-patch-information option from dex2oat.

Because we no longer support running patchoat on npic oat files, which
means the included patch information is unused .

Bug: 33192586
Test: m test-art-host

Change-Id: I9e100c4e47dc24d91cd74226c84025e961d30f67

Don't inline methods that throw in graph with irreducible loops.

Re-computing the loop information is not supported in graphs
with irreducible loops, as it is not deterministic, and the
loop header of a loop could change. That would lead to having the
suspend check in the wrong block.

Test: test-art-host
Test: 641-irreducible-inline
bug: 35757766
Change-Id: I6a435885461fbeca035e4f5d94f055fc3262adca

Merge "Invoke typed arraycopy for primitive arrays."

Merge "Return the right value in VerifyClass."

Experiment with ulimit settings to see if it fixes test failure.

RELOCATE test is failing on a build bot with file size limit exceeded.
The suspicion is ANDROID_HOST_OUT is being set oddly on the build bot so
we don't find the right directory to use as the basis for setting the
ulimit on RELOCATE and end up with much too small a limit.

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

Merge "ART: Update LZ4 library to v1.7.5"

ART: Add field offset caching

Add caching to reduce the computational complexity of deriving
field indices. Currently cache the number of interface fields.
In the future, we may also consider the superclass count to
accelerate class visiting.

Bug: 31385354
Test: m test-art-host
Test: m test-art-host-run-test-906-iterate-heap
Test: m test-art-host-run-test-913-heaps
Change-Id: I19da243d1678f9a4a047d849193fb8bac01ed53c

Merge "Encode inline caches with missing types in the profile"

Add two locks to expected_mutexes_on_weak_ref_access_.

The ObjectRegistry lock and the jdwp event list lock to avoid DCHECK
failures.

Some cleanup.

Bug: 35360959
Bug: 35745310
Test: test-art-host
Test: jdwp test.
Test: angler boot.
Change-Id: I16000c0c7624b1271d40c4c1a01a4e249271b67e

Revert "Add jdwp event_list_lock_ to expected_mutexes_on_weak_ref_access_."

This reverts commit 35b456a05af864da6580dd01c033efb0ed3e3e44.

Will be superceded by aog/346324.

Bug: 35360959
Test: test-art-host.
Test: jdwp test.

Change-Id: I325003cb85b29c0245de6ba86b719b51201a840c

Merge "Change "could not find class" to warning"

Improve isunit computation, some tests edits needed for SIMD.

Rationale:
Break-out CL of ART Vectorizer: number 4.
The purpose is making the original CL smaller
and easier to review.

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

Merge "Bug fix in overflow detection on pow. With regression test."

Merge "ART: Clean up code related to foreign dex use"

ART: Clean up code related to foreign dex use

We simplified the way we track whether or not a dex file is used by
other apps. DexManager in the framework keeps track of the data and we
no longer need file markers on disk.

Test: device boots, foreign dex markers are not created anymore

Bug: 32871170

(cherry picked from commit 868515e2606820ea96f8b9022b442f5bcc770867)

Change-Id: If51b3301c845a8c3bbaf87d0e35a12c700e1d0aa
Merged-In: I5a04d3bba29581dedf05e21b8b20d79afa9b405a

ART: Refactor ObjectTagTable

Factor out the plain holder from the event handling.

Bug: 31385354
Bug: 31385027
Test: m test-art-host
Change-Id: If6af36e5275f8f1b924ab7dba1fb14f1ce2de8fe

Allow test runner to build master-art-host targets

With --build-target <target-name>, where target-name is a run-test
target on master-art-host, testrunner setup environment and runs the
tests for the target.

Test: manual
Change-Id: I17d674319283f2c455742df63e8fddb277d9bb31

Return the right value in VerifyClass.

We used to return kNoFailure when a class was already processed.
But it could have had soft failures.

Also remove IsCompileTimeVerified to avoid future confusions and
introduce ShouldVerifyAtRuntime.

Add some more checks to make sure we record the right things in
the vdex file.

bug: 33845394
test: verifier_deps_test test-art-host
Change-Id: Iff11a96e825c85db416083413761981515f405b7

Bug fix in overflow detection on pow. With regression test.

Rationale:
Missed potential overflow in base update.

Test: test-art-host
Bug: 35706963
Change-Id: I117d96c1fa95dde37f5676f9dc750f0a71becafa

Invoke typed arraycopy for primitive arrays.

Apps will always call the Object version of arraycopy. When
we can infer the types of the passed arrays, replace the method
being called to be the typed System.arraycopy one.

10% improvement on ExoPlayerBench.

Test: 641-checker-arraycopy
bug: 7103825
Change-Id: I872d7a6e163a4614510ef04ae582eb90ec48b5fa