Inline methods that throw.

Forked from https://android-review.googlesource.com/214292

test: 637-checker-throw-inline
bug: 30933338
Change-Id: I184be82dfab0710af3f3796e9e486c7817fa9c60

Merge "Add a DCHECK in thread::GetPeer."

Merge "Change behavior of $noinline$ to now force not inlining."

Merge "Fix DEX zip open error message printing"

Merge "ART: Rewrite test 911"

Merge "ART: Change test 121-modifiers to use smali"

Merge "Address some comments for aog/333864"

Merge "Revert "Revert "runtime: Mmap uncompressed dex files (in zip) as clean memory"""

Address some comments for aog/333864

Mostly comment changes.

Bug: 34621054

Test: test-art-host

Change-Id: I95f77651d47a8d0c71d580778d1126ab4aa1d751

ART: Rewrite test 911

Use explicit thread names to avoid output reliance on implicit
thread naming.

Bug: 31684812
Test: m test-art-host-run-test-911-get-stack-trace
Change-Id: I82004fea6f49831b5d378bef2c290b61c694a07a

Change behavior of $noinline$ to now force not inlining.

In order to start supporting more inlining, like inlining
methods that have a throwing branch, we need to change our
testing strategy.

test: test-art-host
Change-Id: I65354884898d86a18e78b96b7c21a8728c83f86d

Merge "ART: Add operator == and != with nullptr to Handle"

Merge "Fix test scripts to account for new linker behavior"

Revert "Revert "runtime: Mmap uncompressed dex files (in zip) as clean memory""

This reverts commit 960b2af8a05f0844e78004e2d0d3ae6ab058d430.

Fix failing 071-dex-file-map-clean on target
which was due to an IO race in the run-test script.

Test: make test-art-target
Bug: 27650033
Original-Change-Id: I18efbd392c5980ffe0d983833b6cc581e0237b92
Change-Id: I6f4ff1e85f8326916c2ae0842a32f53fb7901639

Merge "Update test 044 for java-8"

Merge "ART: Make dex2oat watchdog configurable"

ART: Add operator == and != with nullptr to Handle

Get it in line with ObjPtr and prettify our code.

Test: m
Change-Id: I1322e2a9bc7a85d7f2441034a19bf4d807b81a0e

Fix test scripts to account for new linker behavior

Remove /system/lib from the java.library.path to
avoid loading system libraries in classloader-namespace
by dependencies originated from libarttestd.so

Add additional libraries used by libarttestd.so to
the list of public libraries.

Bug: http://b/26833548
Test: m -j32 test-art-target
Change-Id: I8cfb1b1bec17ca32b2c691597a97b4e9d1667a2f

Merge "Added a few integral SIMD extensions for x86/x86_64 (SSE)."

Merge "Revert^6 "Make class redefinition work with native methods on stack.""

Fix DEX zip open error message printing

We were printing the pointer to the error message string rather than the
string itself.

Bug: none
Test: test-art-host
Change-Id: I06bd9afd100c425039847c52df1f4912966053b8

Revert^6 "Make class redefinition work with native methods on stack."

We were incorrectly trying to obtain the profiling information of a
native method.

This reverts commit 02b2349b1cd2a78e86b1a7542f8330e6c3aaeb35.

Reason for revert: Fixed test failure with jit configurations

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

Change-Id: Ic7112104aec64e597d2df80db5dc2a505d2cc2dd

Merge "Revert "Enable dex2oat swap test for CC on x86.""

Revert "Enable dex2oat swap test for CC on x86."

Is't still failing and the difference can be
rather huge:
    Expected: (native_alloc_2_) < (native_alloc_1_),
    actual: 379712 vs 248600

Bug: 29259363

This reverts commit 38f5663a970b633f08e144b4dff33633c51352ee.

Change-Id: Ie1d62fbba688aeadffb9d5aafd5b9cf632742ed6

Merge "ART: invoke-custom support"

Merge "ARM64: Improve String.equals() intrinsic for const strings."

Add a DCHECK in thread::GetPeer.

Motivated  by https://android-review.googlesource.com/#/c/333205/.

Test: test-art-host run-jdwp-test.sh
Change-Id: I173c060324aa0dc39144db55e3a97e672c012ba8

Merge "Revert "Disable libcore tests for now.""

Revert "Disable libcore tests for now."

Vogar has been fixed.

bug:35308152

This reverts commit b1cde6a6d0101328e683fe07f81d40a1368eaae9.

Change-Id: I2440da0a97561f6794f8827b536481e1a6536c91

ART: invoke-custom support

Adds invoke-custom instruction to the interpreter.

Bug: 33191717,30550796
Test: art/test/run-test --host 952
Change-Id: I3b754128649a8b3a00ade79ba2518d0e377f3a1e

ARM64: Improve String.equals() intrinsic for const strings.

And add additional tests to 021-string2.

aosp_angler-userdebug:
  before:
    arm64 boot*.oat: 43324664
    arm64 boot*.oat/string compression: 43411112
  after:
    arm64 boot*.oat: 43300136 (-24528)
    arm64 boot*.oat/string compression: 43345464 (-65648)

The string compression code size difference drops from
86448 to 45328.

Test: m test-art-target on Nexus 6P
Test: m test-art-target on Nexus 6P with string compression enabled.
Bug: 31040547
Change-Id: I99a3777b91b248da2b0ac25abd260f9e5abb2c09

Merge "Bump oat version."

Bump oat version.

Two recent changes should have bumped the oat version but
they did not:
    https://android-review.googlesource.com/336161
    https://android-review.googlesource.com/338409

Test: Rely on Treehugger.
Change-Id: I105f2f2d54953ed808ded92234c06efcc155d313

Merge changes I1042e87f,I58b17ecf

* changes:
  Remove remaining uses of DexFile::GetChecksum.
  Improve checking of multidex dex checksums.

Merge "Revert "Revert^4 "Make class redefinition work with native methods on stack."""

Revert "Revert^4 "Make class redefinition work with native methods on stack.""

ART run-test 945-obsolete-native is failing in automated
testing.

This reverts commit 724f77e2fed038d57a3d08fdcf656d703e3473ea.

Change-Id: I4eceb45765826ca442ee8f4800b2bed51f98e662

Remove remaining uses of DexFile::GetChecksum.

Remove or replace with DexFile::GetMultiDexChecksums as appropriate.

Bug: 34604632
Test: m test-art-host, added image_space_test to test ValidateOatFile.
Change-Id: I1042e87f29a242046d055f0be374ad9fbce4671a

Improve checking of multidex dex checksums.

* Fall back to odex file for all multidex entries if the apk is stripped,
  not just for the main multidex entry.
* Verify the number of multidex entries has not changed.
* Improve performance by getting all checksums from the apk in one go
  and cache the results instead of repeatedly opening the apk.
* Stop referring to non-main multidex entries as "secondary" dex files.

Bug: 34604632
Test: added tests to dex_file_test and oat_file_assistant_test
Test: m test-art-host

Change-Id: I58b17ecfbc9165a5bfeffd5281ee21d108f64479

Merge "ART: Add DEX support for MethodHandle and CallSite info."

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

Merge "ART: Correctly handle temporary classes in class-load events (4/3)"

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

Assert failing for "earchbox:search":
F zygote64: class_linker.cc:4612] Check failed: handle_scope_iface.Get() != nullptr

Test: m test-art-host
Bug: 34839984
Bug: 30627598
Bug: 34659969

This reverts commit 85c0f2ac03417f5125bc2ff1dab8109859c67d5c.

Change-Id: I39846c20295af5875b0f945be7035c73ded23135

Merge "Revert^4 "Make class redefinition work with native methods on stack.""

ART: Correctly handle temporary classes in class-load events (4/3)

When a temporary class is given out in a ClassLoad event, all stored
references need to be fixed up before publishing a ClassPrepare event.

This CL handles objects stored in the heap as referents.

Bug: 31684920
Test: m test-art-host-run-test-912-classes
Change-Id: If140ecae675cd7bc648f622eaf200f8ad8b15438

Merge "Deduplicate register maps for both inline and normal stack maps"

Update test 044 for java-8

Jack will create a synthetic default bridge method for interfaces
where there is return-type variance. This changes the output of test
044-proxy very slightly since it ends up choosing the default version
when it is executed.

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

Revert^4 "Make class redefinition work with native methods on stack."

We had a problem where there was a brief period of time where the dex
file for an obsolete method could not be obtained. This meant that not
all functions of ArtMethod could safely be called, causing some
problems.

This reverts commit 38c20d4a694eea44a1bd3af32a6a99512c139358.

Reason for revert: Fixed issues with interp-ac and relocate-npatchoat

Test: ART_TEST_RUN_TEST_RELOCATE_NO_PATCHOAT=true \
      ART_TEST_INTERPRETER_ACCESS_CHECKS=true \
      mma -j40 test-art-host

Change-Id: I04991f3e76813831b6446f97636b6fa404397f36

Deduplicate register maps for both inline and normal stack maps

Before it only deduplicated the normal stack map dex register maps.
Code size for a large app: 93341616 -> 92678040 (-0.7%)

Added test.

Bug: 34621054

Test: test-art-host

Change-Id: I4fab4e40915bfa12cb978edbb3cbc19e2cf00954

Added a few integral SIMD extensions for x86/x86_64 (SSE).

Rationale:
ART vectorizer needs SIMD for integer operations too.

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

Merge "Enable dex2oat swap test for CC on x86."

Merge "Remove special treatment of native method for CHA."

Merge "Disable libcore tests for now."

Disable libcore tests for now.

bug:35308152
Test: run-libcore-tests.sh

Change-Id: I77f9a516d6dd99d332a214e838848cdc3fdf359a

Merge "Clarify comment on JIT cache flush issue"

Merge "Fix the empty checkpoint timeout."

ART: Make dex2oat watchdog configurable

Allow the watchdog timeout to be set on the command line. Add a
test.

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

Enable dex2oat swap test for CC on x86.

The test should be OK after
    https://android-review.googlesource.com/335763

Test: m ART_USE_READ_BARRIER=true test-art-host-gtest
Bug: 29259363
Change-Id: I16f4051036b247b4f520533e77eaa71d6ff7bb41

Merge "ART: Correctly handle temporary classes in class-load events (3/3)"

Merge "Use VerifierDeps test file for dex2oat swap tests."

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

ART: Add DEX support for MethodHandle and CallSite info.

Adds new DEX file map items for CallSiteIds and MethodHandles.

Initializes CallSiteIds and MethodHandles from the DEX file map_list.

Bug: 33191717,30550796
Test: m test-art-host-gtest-dex_file_verifier_test
Change-Id: I3ad9c7342b661c3f6a8264709412650eee6bde01

Merge "String Compression for MIPS32 and MIPS64"

Merge changes I425a7d61,I30365ad6

* changes:
  Regression test for issue with VIXL AArch64 veneer pool.
  ARM64: Remove all uses of BlockPoolsScope.

String Compression for MIPS32 and MIPS64

Changes on intrinsics and Code Generation on MIPS32 and MIPS64 for
string compression feature.

Testing is done with STRING_COMPRESSION_ENABLED = true (in libcore),
mirror::kUseStringCompression = true and STRING_COMPRESSION_FEATURE set
to 1.

Test: booted MIPS32 and MIPS64 in QEMU
Test: mma test-art-target-run-test on CI20 (MIPS32R2)
Test: mma test-art-target-run-test in QEMU (MIPS64R6)

Change-Id: If50a6b6c0792bfa34d4fdff6bf2c7542211d2689

Merge "Convert preload list to profile"

Merge "Revert "runtime: Mmap uncompressed dex files (in zip) as clean memory""

Revert "runtime: Mmap uncompressed dex files (in zip) as clean memory"

Getting on the bots:
+ERROR: Memory mapping for /data/run-test/test-6248/071-dexfile-map-clean-ex.jar is unexpectedly dirty
+Private_Dirty:         4 kB

Bug: 27650033

This reverts commit 19e5f834501c5e69fbd731038b88c10332cc6bc7.

Change-Id: I833ef95401187b764d336ab4558936c63678061c

Merge "Allow testrunner to run test when inside ART directory"

Merge "Use null error_msg for pic images"

Merge "Add extra logging for image decompression speed"

Use null error_msg for pic images

Reduces 20ms of overhead for reading proc maps when the map can't be
placed at the optimal address.

Bug: 34927277

Test: test-art-host && device boot

Change-Id: Ib91fff2b832f61c49e67edcd2b7a222ab4409984

Add extra logging for image decompression speed

Sample output:
I zygote  : Decompressing image took 3.173ms (678MB/s)

Test: Phone boot with -verbose:image

Bug: 34927277
Change-Id: If0689594bca52d5b98d32d10adfe64f6d2402020

Merge "Revert "Revert "Revert "Make class redefinition work with native methods on stack.""""

Revert "Revert "Revert "Make class redefinition work with native methods on stack."""

This reverts commit 4493f92dfc70e01d95ad57bc4c0c38f59a616f70.

Reason for revert: Fails interpreter-ac and relocate-npatchoat

Change-Id: I15eb4964c9101fa04e34b52acf9ffe25e6d4e60d
Test: revert

ART: Correctly handle temporary classes in class-load events (3/3)

When a temporary class is given out in a ClassLoad event, all stored
references need to be fixed up before publishing a ClassPrepare event.

This CL handles objects stored in the heap.

Bug: 31684920
Test: m test-art-host-run-test-912-classes
Change-Id: Ia0456c81fd848618e637b93301edf4dbc8d848f2

Fix the empty checkpoint timeout.

The problem happens when a thread is indirectly blocked on a mutex
that another thread holds and is blocked on a weak ref access.

Add a way to do a dummy wakeup on a thread that's blocked on a mutex
so that the thread will respond to the empty checkpoint request. Do
this for the mutexes that are expected to be held when a weak ref is
accessed. Add a check that detects an unexpected case.

Bug: 33006388
Bug: 12687968
Test: test-art-host.
Change-Id: Iefec69b9a21aa25a928cb31fcf4fb872f867a8c2

Merge "ART: Remove run-test blacklist"

Merge "Remove breakpoints from redefined classes"

Merge "Fix a moving gc bug in Instrumentation::ExceptionCaughtEvent."

Allow testrunner to run test when inside ART directory

Test: ./test/testrunner/testrunner.py -j64 --host -b
Bug: 35257742
Change-Id: Ib6264378971ad9f83ece7a51afc95c81935e08fc

Merge "ART: Fix potential deadlock in class-load events"

Merge "Revert "Revert "Make class redefinition work with native methods on stack."""

Convert preload list to profile

Add functionality to profman to take a list of classes in a text file
(dot notation rather than descriptors) and build a compiler profile
file from it.

Bug: 34929204
Test: test-art-host
Change-Id: Id867ec7b436cbd8412809a8adb093f588ff474b7

Merge "ART: Correctly handle temporary classes in class-load events (2/3)"

Remove special treatment of native method for CHA.

This special treatment is not necessary and causes complexity and issues.

Bug: 35104526
Test: m test-art-host
Change-Id: Icb07d147b433388d0b9430212b9b4c4bf04eafae

ART: Remove run-test blacklist

Follow-up to commit e2744c615e5808d335eed512349d67d724773c1f.

Bug: 31684920
Test: m ART_TEST_JIT=true test-art-host-run-test-912-classes
Test: art/test/testrunner/testrunner.py --jit --host -b -t 912-classes
Change-Id: I482af4d4b202d44f94cf941165a9b0247fa07112

Remove breakpoints from redefined classes

Test: Manual
Change-Id: If8d9a38635bda7a0d69925b735b6f10055192b34

Merge "runtime: Mmap uncompressed dex files (in zip) as clean memory"

Fix a moving gc bug in Instrumentation::ExceptionCaughtEvent.

This fixes JDWP exception caught test crashes.

Bug: 34929692
Test: CtsJdwpTestCases on angler.
Test: test-art-host
Test: The jdwp test on host.
Change-Id: I2f6a66d7e871e6c7d65a8ad287249286b301c3b1

ART: Fix potential deadlock in class-load events

When a temporary class is given out in a ClassLoad event, all stored
references need to be fixed up before publishing a ClassPrepare event.

This CL fixes a potential deadlock when multiple classes are loaded
at the same time.

Bug: 31684920
Test: m test-art-host-run-test-912-classes
Change-Id: Iccabd8c17838f686a81a825badc7c2b1a1589708

ART: Correctly handle temporary classes in class-load events (2/3)

When a temporary class is given out in a ClassLoad event, all stored
references need to be fixed up before publishing a ClassPrepare event.

This CL handles objects stored as local references.

Bug: 31684920
Test: m test-art-host-run-test-912-classes
Change-Id: I3c6bbfdaca31d0c0d8ae9bf1facef404e949b6aa

Revert "Revert "Make class redefinition work with native methods on stack.""

When we were scanning the stack for tracing we were getting the wrong
stack-frame size for obsolete native methods. This fixes it by
creating real obsolete native methods so we can recognize them and
then doing the (rather long) calculation to find their real
stack-frame size.

This reverts commit 7558d27ccd0837fef7c4bfbff7fc82f07a787316.

Reason for revert: Fixed tracing failures.

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

Change-Id: Ic65da1b51a43733ff60131832753afa0c4ce66b1

Merge "ART: Correctly handle temporary classes in class-load events (1/3)"

runtime: Mmap uncompressed dex files (in zip) as clean memory

Saves 3MB of RAM (system-wide) in AGSA (search toolbar).
Also makes 3MB of RAM clean (previously dirty).

----

classes.dex from different processes will now have
their memory shared (as clean) globally.

To take advantage of this, the app/service just
needs to build their .JARs zipaligned and with
classes.dex being stored uncompressed.

-----

classes[N].dex within ZIP files will now get memory-mapped
as file-backed, clean memory when opening a DexFile.

(The zip entry must be at least 4-byte aligned).

Compressed DEX files still get extracted first to dirty memory.

Test: art/tools/run-test 071-dexfile-map-clean
Bug: 27650033
Change-Id: I18efbd392c5980ffe0d983833b6cc581e0237b92

Regression test for issue with VIXL AArch64 veneer pool.

Dex2oat used to crash on this assertion:

  external/vixl/src/aarch64/macro-assembler-aarch64.cc:245: void vixl::aarch64::VeneerPool::CheckEmitFor(size_t, vixl::aarch64::Pool::EmitOption): assertion "masm_->GetCursorOffset() < unresolved_branches_.GetFirstLimit()" failed

Test: test-art-target (on ARM64)
Bug: 34850123
Change-Id: I425a7d612cc20bf2a6fb1125028af35a73794b52

Clarify comment on JIT cache flush issue

The original fix (#304947) is needed, but the comment was suggesting a
hardware cause and a link with A53 errata. In all likelihood neither
of these is true, and the issue is actually caused by a kernel issue
affecting many Qualcomm devices (like those using the msm tree).

Change-Id: Ie750e6fd9eb7130982d184c22917b5c3ea4beed0

Merge "testrunner: Speed up printing --help by 4x"

ARM64: Remove all uses of BlockPoolsScope.

BlockPoolsScope should not be used because it is a VIXL scope
for VIXL internal usage only. In arm64 backend the intent was to
block pools between a particular instruction (Ldr, Str, Blr, etc)
and a subsequent MaybeRecordImplicitNullCheck or RecordPcInfo call.
However pools should be emitted at the opening of a scope if this
is required to satisfy branch/ldr ranges. This is not done by the
BlockPoolsScope, so proper scopes are now used now:
ExactAssemblyScope and EmissionCheckScope.

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

Bug: 34850123

Change-Id: I30365ad63c644cf9dd85d5a3c2118f9c57be9d20

Merge "Use entrypoint switching on x86 & x86-64 for GC root read barriers."

Merge "Change type initialization entrypoints to kSaveEverything."