Merge "Added direct memory operand support for comiss/sd on x86."
am: e304fc28c4

Change-Id: Iee9114dceeb90feccc670791982815c17136137a

Merge "libsigchain: intercept bsd_signal on lp32"
am: ba001a5577

Change-Id: I197ec9cf35e2c8cff45ca62e9ae501ee87da8701

Merge "Added direct memory operand support for comiss/sd on x86."

Merge "libsigchain: intercept bsd_signal on lp32"

libsigchain: intercept bsd_signal on lp32

bsd_signal is a deprecated synonym for the signal. Nevertheless
there are apps and libraries calling this function.

libsigchain needs to intercept these calls and handle them the same
way it handles signal() call.

Bug: http://b/30562229
Test: readelf --dyn-sym app_process32 and check that bsd_signal is exported
      readelf --dyn-sym app_process64 and check that bsd_signal is not exported
Change-Id: I3aa41632bb015568d8524a82f1c97e4443ec0d6c

Added direct memory operand support for comiss/sd on x86.

Rationale:
Memory operands allows for shorter instruction sequences.
Something that may be used e.g. the upcoming implementation
of the round intrinsic.

Bug=26327751

Change-Id: Ifd976ed86917de1879fa036cbbbff5edf4fb741d
Test: assembler_x86_test

Merge \"Put the deletion of profiling info under a GC critical section.\"
am: 1a7c33704a

Change-Id: I4952fd9f60b7b5c85b2acba7d808f008601029d0

Merge "Put the deletion of profiling info under a GC critical section."

Merge \"Revert \"Revert \"Scan immune spaces using mod union tables.\"\"\"
am: fcc3c6d26a

Change-Id: Id762c75f579d621cb67ca8ea7b7c16024732e23c

Merge "Revert "Revert "Scan immune spaces using mod union tables."""

Merge \"ART: remove gcc cruft from the makefiles\"
am: df638c66d1

Change-Id: I370c6a829207d657b6a67b259773ab28529ec373

Merge "ART: remove gcc cruft from the makefiles"

Merge \"cpp-define-generator: \"make art-update-cpp-defines\" to regen asm_support\"
am: 4130c3360e

Change-Id: I034b4ad64c8298cec6a998cf2d3ddf11c1812d71

Merge "cpp-define-generator: "make art-update-cpp-defines" to regen asm_support"

Merge \"Fix art::ArchTest::FinalizeSetup not being called.\"
am: 6e5e3b2e91

Change-Id: I2791958a7a77d62390805037e789be48fa58e23f

Merge "Fix art::ArchTest::FinalizeSetup not being called."

Fix art::ArchTest::FinalizeSetup not being called.

CommonRuntimeTestImpl::SetUp() is calling FinalizeSetup. Since
CommonRuntimeTestImpl::FinalizeSetup is not virtual (it's virtual
one step higher in class hierarchy in CommonRuntimeTestBase), overridden
version in ArchTest was never used. This is causing ArchTest to
fail with openjdk8u60 version of java.lang.reflect (it's calling
Class#getDeclaredField in clinit, which fails on 32 bit
version because of different pointer sizes [arch_test forces
64-bit ISA])

Bug: 28666126
Test:  make -j 32 test-art-host-gtest
Change-Id: Ief370f5b18ef787575ac0f88ecbe17ebbf037542

Merge \"ART: Fix RegTypeCache for instance field declaring class\"
am: bffecdbb50

Change-Id: I1c4beef69e286097a859d8855296a0c56826dcbb

Merge "ART: Fix RegTypeCache for instance field declaring class"

Put the deletion of profiling info under a GC critical section.

Otherwise the GC could see dangling pointers.

bug:30033802
Change-Id: I839c4f6e537aba98769e99e4d75ff869c9ff13eb

Merge \"Fix for interpreter crash on new instance of class\"
am: 0f290d764d

Change-Id: Ie237c079eee92df36143b32cdb175a4f52719e34

Merge \"Revert \"Revert \"Use try lock to fix class resolution race\"\"\"
am: 0af088d126

Change-Id: If34912dd115472f4fb2a84dd82a0b1d91c488a06

Merge \"Create OatFileAssistant::OatFileInfo inner class.\"
am: 3d733104c9

Change-Id: Iddfadebb0b90ea9ce8376a5a52ec8fc7c2e386f5

Merge "Fix for interpreter crash on new instance of class"

Merge "Revert "Revert "Use try lock to fix class resolution race"""

Revert "Revert "Use try lock to fix class resolution race""

Fix possible deadlock in EnsureResolved caused by interaction with
GC. Since we were sleeping while holding the mutator lock, it could
block thread suspension. This would deadlock if the thread that
had locked h_class is already suspended since we would spin forever
and not make progress.

Bug: 27417671
Bug: 30500547

Test: test-art-host ART_TEST_GC_STRESS=true

This reverts commit 69bf969c055c31a75d17ea92aeee756042678114.

Change-Id: Id8750df065dc3b9ef7dc874f2eb2cc2c58e5d1eb

Merge "Create OatFileAssistant::OatFileInfo inner class."

Fix for interpreter crash on new instance of class

Rationale:
Fuzz testing found divergences between the compiler and interpreter
which turned out to be caused by calling instance on java.lang.Class
(this worked for compiler but crashed interpeter). Since RI does not
allow this construct, solution is to force interpreter in this
unlikely case and throw run time exception like RI. This fixes
two cases found with fuzz testing. CL also includes
fail-before/pass-after test.

Test: 600-verifier-fails

BUG=29758098

Change-Id: Ie80f7758def44e6655d28fec4c10c34ffa0fd60b

Revert "Revert "Scan immune spaces using mod union tables.""

This reverts commit ccf7c8a68972f3b7bfa38af0c6b792f712f83765.

Change-Id: Iee618d9ae78e1dcdea4a7ecbe9253821d4d4d015
Test: N9 boot, art tests, Ritz EAAC.
Bug: 29516968
Bug: 29517059
Bug: 12687968

Merge \"Revert \"Use try lock to fix class resolution race\"\"
am: 7707777f06

Change-Id: I1cd0bea29e785d45fd9b140fc2209826ff316c54

Merge "Revert "Use try lock to fix class resolution race""

Revert "Use try lock to fix class resolution race"

This reverts commit a704eda0078989a73cac111ed309aca50d2e289b.

Bug: 27417671
Bug: 30500547
Change-Id: Ieea05236b9e61c722660cd9497c9d55d13ccd010

Merge \"Add arena stack and priority queue type aliases\"
am: ea159431cd

Change-Id: Ia4db08fe011ee7e1eaadc8f0f221f71f146dbdfb

Merge \"Revert \"Scan immune spaces using mod union tables.\"\"
am: b94ea458e7

Change-Id: I07be24b3ac8a58d041e8bf09857cb1209ab89c2c

Merge "Add arena stack and priority queue type aliases"

Merge "Revert "Scan immune spaces using mod union tables.""

Merge \"ART: remove ART_JIT makefile variable\"
am: 8481f6390a

Change-Id: Id4cf81cfe8547ea03f09a2d1e4d592d58e9725a7

Merge "ART: remove ART_JIT makefile variable"

Revert "Scan immune spaces using mod union tables."

This reverts commit 8256da35feac0ba7589fcd4b692a4b9a936679a4.

Test failing.

Change-Id: I391dcbdba19c6f6313e06075b0a69ea406c7c50e

Add arena stack and priority queue type aliases

These will be used in the graph coloring register allocator.

Test: m test-art-host

Change-Id: I4d3d063ae5a02e697f07fc8212f391b5347270e5

cpp-define-generator: "make art-update-cpp-defines" to regen asm_support

Change-Id: I1a550eda6c77f38e63bb1c8fb943541cbd3f9cdf

Merge \"Pass the right class loader when inlining.\"
am: 4c4d6f42cf

Change-Id: I8568aff8cf2323e43e0d15441827ded3edd417e1

Merge "Pass the right class loader when inlining."

ART: remove gcc cruft from the makefiles

ART only builds with clang now, delete all logic to determine gcc
vs. clang, and move all LOCAL_CLANG_CFLAGS into LOCAL_CFLAGS.

Test: mma, make test-art-host, make test-art-target
Change-Id: I578615233ad3e71af18afb47bb11f3be8417216c

ART: remove ART_JIT makefile variable

The ART_JIT variable isn't used (and was never used?).

Test: mma, make test-art-host, make test-art-target
Change-Id: Ie6463c3c13fab43504f3c60fd5a15f696e2cf481

Create OatFileAssistant::OatFileInfo inner class.

To share code reused for getting info about both the odex and oat files.

Test: oat file assistant tests.
Change-Id: Ifcc48e0ac481899285e39e3b02f3ade180e38728

Merge \"Avoid read barrier for IntArray::GetArrayClass\"
am: 533d684038

Change-Id: I6f2843bd1b2683fa4e120baa8abf80f6a377580c

Merge "Avoid read barrier for IntArray::GetArrayClass"

Avoid read barrier for IntArray::GetArrayClass

Changed the code use Mark instead of read barrier, this showed an
existing lock violation and possible deadlock which was fixed.
Prevent DCHECK failure from the assert.

Bug: 30469265
Test: test-art-host with CC

Change-Id: I275f953f06f6d13262043fc62eb88dca0356465a

Merge \"Use try lock to fix class resolution race\"
am: 85c4a4b8c9

Change-Id: Ie267436199e2bd504b8635f62f368bd84bda1897

Merge "Use try lock to fix class resolution race"

Merge \"Scan immune spaces using mod union tables.\"
am: ccee3363e3

Change-Id: I3930d2d589a750f8b6bbf97503fd7a0a6e06feeb

Merge "Scan immune spaces using mod union tables."

Merge \"Disable CheckSwapUsage test with read barriers on x86.\"
am: 5fc9ddef31

Change-Id: Ib3d6cd1a84663842581f0eb0799cb19df7c41465

Merge "Disable CheckSwapUsage test with read barriers on x86."

Merge \"Remove two ReadBarrierMarkRegX entrypoints.\"
am: d22b69a127

Change-Id: I1e4a311c28af62aac97d699b67bba01de185c9d1

Merge "Remove two ReadBarrierMarkRegX entrypoints."

Pass the right class loader when inlining.

Otherwise, method and type resolution can resolve to the wrong
things and as a side effect update the dex cache with wrong data.

bug:30403437
test:./art/test/run-test --host --jit --dev --no-prebuild 613

(cherry picked from commit 0a210d9b108c87c0e7c1d430a92ce6fc89790c95)

Change-Id: I8fdca96fccae079c8e99b8e86e7b6935acfce89d

ART: Fix RegTypeCache for instance field declaring class

For bytecode instruction:

  iget(-*) ClassB->field:type

where the resolved field is actually declared in ClassA (ClassB
extends ClassA), MethodVerifier would create a RegType representing
ClassA but cache it under the descriptor "ClassB".

This is a bug but does not have any implications on correctness
because earlier resolution of ClassB (part of IGET handling) creates
another cache entry with the "ClassB" descriptor. Because the latter,
wrong entry is always imprecise (ClassA cannot be final because ClassB
extends it), either the earlier entry will be discovered or neither
of them will be a match.

Descriptor was replaced with the correct one and a DCHECK added when
creating a RegType to make sure the descriptor matches.

Bug: 30458218
Change-Id: I19e1bdd5dd79e5eac558122a06b9128d0a5c021f
Test: m test-art-host-run-test
Test: art/test/run-test --host 800-smali

Disable CheckSwapUsage test with read barriers on x86.

This test does not produce the expected results on
some x86 systems when read barriers are enabled.

Also remove unused TEST_DISABLED_FOR_READ_BARRIER macro
definition.

Test: m test-art-host-gtest
Bug: 29259363
Bug: 12687968
Change-Id: If7c124226b32c10425725e62bb684150aba108ab

Merge \"ARM64: Add comment to mterp int-to-long.\"
am: e3051bf607

Change-Id: I50373e031406dad9bc56129983e47d761fde4f26

Merge "ARM64: Add comment to mterp int-to-long."

Merge \"Add ProcessBuilderTest#testRedirectInherit() to known failures\"
am: 93aa03c913

Change-Id: I0bb1d0af6e1a76bf03678dc1d090e53b9d118907

Merge "Add ProcessBuilderTest#testRedirectInherit() to known failures"

ARM64: Add comment to mterp int-to-long.

Test: Build for Nexus 9.
Change-Id: I649e71fc1cdc2a4a103159bf72ceeb005231e66f

Merge \"MIPS32: Improve string and class loads\"
am: f9136b631a

Change-Id: I0d5950875daa51ed819cc62cfdbf3c0c9543f58c

Merge "MIPS32: Improve string and class loads"

Merge \"ARM64: More mterp improvements.\"
am: f59f597690

Change-Id: Iaadad83a4052394e0c9d4d4db8fe0125813eb137

Merge "ARM64: More mterp improvements."

Merge \"ART: Use old operator<< for MemBarrierKind\"
am: 7cef37575c

Change-Id: I352ffba82a4554942ff5b3f2dfb7a8a1e3cce790

Merge "ART: Use old operator<< for MemBarrierKind"

ART: Use old operator<< for MemBarrierKind

To fix assumptions in the tests.

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

MIPS32: Improve string and class loads

Tested:
- MIPS32 Android boots in QEMU
- test-art-host-gtest
- test-art-target-run-test-optimizing in QEMU, on CI20
- test-art-target-gtest on CI20

Change-Id: I70fd5d5267f8594c3b29d5a4ccf66b8ca8b09df3

Merge \"ART: Delete old compiler_enums.h\"
am: 6369394e75

Change-Id: Ibb7e4f1351b8196ffa0b655b0ec91922e263fc20

Merge "ART: Delete old compiler_enums.h"

Merge \"Reduce unnecessary read barriers in GC\"
am: 74d19f13d2

Change-Id: Ic604b9201f4d23c7b230813ba884f1e0fb83320a

Merge "Reduce unnecessary read barriers in GC"

ART: Delete old compiler_enums.h

Holdover from the Quick days. Move the two enums that are still
used closer to the actual users (and prune no longer used cases).

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

Scan immune spaces using mod union tables.

It's faster to scan the immune spaces using the mod union tables (gray
objects on dirty pages) than the live bitmaps (gray objects).

The immune space scan time goes from ~13ms down to ~10ms in the
testWidgetsContainerFling test on angler at 1 GHz.

Change-Id: I0914d7bdb50cbf8a49c859070a3501b48eb79f3e
Test: N9 boot, art tests, Ritz EAAC.
Bug: 29516968
Bug: 29517059
Bug: 12687968

Use try lock to fix class resolution race

There was some possible deadlocks related to EnsureResolved caused by
acquiring an object lock.

Scenario:
Thread 1 acquires lock on obj1
Thread 1 begins to resolve / initialize class1
Thread 1 blocks since it sees that class1 is already being resolved and
gets preempted before it can acquire the object lock on class1
Thread 2 finishes resolving and initializing class1 and locks class1
Thread 2 blocks attempting to lock obj1
Thread 1 blocks attempting to lock class1
Deadlock

Fixed the deadlock by changing EnsureResolved to use a try lock for the
unresolved case.

Added a test.

Test: Device boot, test-art-host, monitor_test

Bug: 27417671
Change-Id: Ic6e1c3ca6f45490cf8a7bf8e137dee71ac83ff64

Merge \"MIPS32: Block callee save fp registers in debuggable\"
am: 1853a70e6e

Change-Id: I6ae521add5fa8186b63306fd41d5f70ca882a967

Merge "MIPS32: Block callee save fp registers in debuggable"

Reduce unnecessary read barriers in GC

Removed read barrier from IsUnprocessed, DequeuePendingReference,
EnqueueReference, and a few other places.

Hard to tell if GC time goes down.

EAAC:
Before GC slow path count: 254857
After GC slow path count: 1005

Bug: 30162165
Bug: 12687968

Test: test-art-host, volantis boot with CC

Change-Id: Ic2add3a9b1e1d7561b0b167f2218b10f8dbff76c

Merge \"Fix profiling output to properly display class info.\"
am: 9755c262df

Change-Id: I417529769b10bd7693461c90f0306881158332a6

Merge "Fix profiling output to properly display class info."

ARM64: More mterp improvements.

Several simple improvements:
  - use EOR for neg-float and neg-double,
  - use SBFX instead of LSL+ASR for const/4,
  - handle long-to-int as move,
  - use LDRSW for int-to-long,
  - remove bogus CMP from iput-quick,
  - use indexed load/store for iget/iput-wide-quick,
  - use CBZ, CBNZ, TBZ, TBNZ for if-eqz, if-nez, if-gez and
    if-ltz, respectively. (The short range of TBZ/TBNZ
    requires emitting the footer before alternate stubs.)

Test: Run ART test suite on Nexus 9 with the interpreter.
Change-Id: I2e65a7cee3d3e2128b870d98cf6157c21f57d607

Fix profiling output to properly display class info.

The profiler stores class def indexes, not type ids.

Change-Id: Ieec9374a2ba381387ed444f376922478183145f8

Add ProcessBuilderTest#testRedirectInherit() to known failures

This test does fairly low level operations that previously exposed
other kernel issues. So far it has been observed failing 1/10 times
on an Android L kernel but never on CTS. Therefore, adding to list
of known failures. testRedirectErrorStream_outputAndErrorAreMerged
has not yet been observed to fail but is similar; it may need to be
added to the list if it fails in future.

Test: cts-tradefed run cts -c libcore.java.lang.ProcessBuilderTest

Bug: 27464570
Change-Id: I5d67f67099e66cba4d75d9bf621609cd2d88d58a

Merge \"Add missing filter cards to zygote mod union table\"
am: 93afc5f20a

Change-Id: I40b355ebdabc4d53661836156b8106fa2c003fbd

Merge "Add missing filter cards to zygote mod union table"

Add missing filter cards to zygote mod union table

Without filtering the cards, every object in the zygote is grayed
during the GC. This was deleted in a PS of previous CL.

GrayAllDirtyImmuneObjects goes from 1.974ms to 376.464us for CC on
N9 maps. This happens during the pause. This CL also fixes regression
in zygote PSS.

Bug: 29516968
Bug: 12687968

Change-Id: I42014e78b1de3ce9b2eefa3bd32f0d24e2ff71c6

MIPS32: Block callee save fp registers in debuggable

This fixes tests 454-get-vreg and 457-regs in debuggable.
Additional changes in LocationsBuilderMIPS::HandleFieldGet/Set
to prevent situations when running out of available fp registers.

Test: mma -j2 ART_TEST_RUN_TEST_DEBUGGABLE=true test-art-target-run-test
Change-Id: Iaad6a6e414ff747b39209780c21aeddc225a04c1

Merge \"Improve CC handling for immune objects\"
am: e92730cbc5

Change-Id: Ie0226a28ebfc44595e614c3c942fba82797ecfa0

Merge "Improve CC handling for immune objects"

Merge \"Remove redundant 601 test, it\'s all in 600.\"
am: 90187c9858

Change-Id: Ibdc01b2f1047bca176c39942e5e80097706ad570

Merge "Remove redundant 601 test, it's all in 600."

Merge \"runtime: Refactor asm_support to be auto-generated (mostly)\"
am: 10656de47c

Change-Id: Ida3192ed82ad2f835eae6f49c46f013ed012a77f

Merge "runtime: Refactor asm_support to be auto-generated (mostly)"

Improve CC handling for immune objects

Currently we reduce ram for immune objects by racing agianst the
mutators to try and finish processing them before the mutators
change many objects to gray. However there is still a window of time
where the mutator can dirty immune pages by changing the lock words
to gray. These pages remain dirty for the lifetime of the app.

This CL changes uses the FlipCallback pause to gray all of the
immune objects that have a dirty card. Once these objects are all
gray we don't to gray any more objects in the immune spaces since
these objects are the only ones that may reference non immune
objects.

Also only scan objects that are gray when scanning immune spaces to
reduce scanning time.

System wide PSS after boot on N9, before:
61668 kB: .art mmap
11249 kB: .Zygote

After:
36013 kB: .art mmap
12251 kB: .Zygote

Results are better than demonstrated since there are more apps
running after.

Maps PSS / Private Dirty, before:
.art mmap     3703     3116
  .Zygote      577      480

After:
.art mmap     1655     1092
  .Zygote      476      392

System server before:
.art mmap     4453     3956
  .Zygote      849      780

After:
.art mmap     2326     1748
  .Zygote      640      564

EAAC:
Before:
ScanImmuneSpaces takes 669.434ms GC time
Scores: 718, 761, 753 average 744
GC time: 4.2s, 4.35s, 4.3s average 4.28s

After:
ScanImmuneSpaces takes 138.328ms GC time
Scores: 731, 730, 704 average 722
GC time: 3.92s, 3.83s, 3.85s average 3.87s

Additional GC pause time is 285us on Maps on N9.
TODO: Reduce this pause time.

Test: N9 booting, test-art-host, EAAC all run with CC

Bug: 29516968
Bug: 12687968

Change-Id: I584b10d017547b321f33eb23fb5d64372af6f69c

Merge \"ART: Add pointer-size template to some reflection functions\"
am: 3023890d94

Change-Id: I4310ad3ef62f7645ff82427512f0ef975f386341