Fix setting FdFile::ReadOnlyMode() flag

The Unix flag O_RDONLY is defined as zero and hence its presence
cannot be tested with '(flags & O_RDONLY) != 0'. This used to be
broken in FdFlag when setting its internal `read_only_mode_` flag.

Test: m test-art-host-gtest-fd_file_test
Change-Id: Ib48abfc908c7032f031450a1574130e06f6c3bab

Merge "Enable IntermediateAddress for primitive arrays with read barriers."

Enable IntermediateAddress for primitive arrays with read barriers.

Test: ART host and target (ARM, ARM64) tests.
Bug: 26601270
Bug: 12687968
Change-Id: I6736ba7b1809bece1bf3cd82c69e4f42a0d3c4a7

Merge "ART: Factor out native stack dumping"

Merge "Add a register allocation strategy compiler option"

Merge "ART: Extract macro assembler"

Merge "Test all register allocators in regalloc gtest"

Add a register allocation strategy compiler option

Test: manually, on device.

Change-Id: If007a1657dd5769ddef03691e0a19dbbe6ba1a29

ART: Extract macro assembler

Extract macro assembler functionality used by the JNI compiler from
the assembler interface. Templatize the new interface so that
type safety ensures correct usage.

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

Merge "Implement a graph coloring register allocator"

Test all register allocators in regalloc gtest

Previously, the gtest only exercised the default register allocator.
Note that the line count is high due mostly to whitespace changes.

Test: m test-art-host-gtest-register_allocator_test

Change-Id: I783edf98ae11d605d4f69834866c387abb71d34f

Implement a graph coloring register allocator

Test: m test-art-host

Change-Id: I8c0d77f339ab02b33588a54b96ecce5c8322cfce

Merge "ARM64 asm for region space array allocation"

ARM64 asm for region space array allocation

Wrote region space tlab array and array resolved allocators in
assembly code. The speedup is a combined increase from checking the
mark bit and having an assembly fast path.

Added resolved, initialized entrypoints for object region TLAB
allocator.

N6P (960000 mhz) EEAC benchmark (average of 50 samples):
CC 1442.309524 -> 1314 (10% improvement)
CMS: 1382.32

Read barrier slow paths reaching C++ code go from 5M to 2.5M.

Bug: 30162165
Bug: 12687968

Test: With CC: N6P boot, run EAAC, test-art-target

Change-Id: I51515b11ef3f795f57eb72fe0f5759618fef5084

Merge "ART: Fix type parameter in tests"

ART: Factor out native stack dumping

Move DumpNativeStack and DumpKernelStack into a new file to keep
concerns separated. Clean up the file.

In preparation for modifications to how we use addr2line.

Bug: 30351473
Test: m test-art-host
Test: manual host testing with a test that times out
Change-Id: I02d5645952fe94fd8e73bcf0522721547f6cef66

Merge "Temporary disable new round implementation on x86/x86_64"

Merge "Add support for CallKind::kCallOnMainAndSlowPath"

Temporary disable new round implementation on x86/x86_64

Rationale:
FUGU is not happy

Test: 580-checker-round

BUG=26327751

Change-Id: If0ddea47a88e14b86d37080b8a18a6f8defcc8e6

Merge "ART: Fix Mac-specific things in x86-64 assembly"

Merge "Implement single-/double-precision round intrinsic in x86_64"

Merge "Clean up use of art::Exec versus execv."

Implement single-/double-precision round intrinsic in x86_64

Rationale:
X86_64 does not provide a direct instruction for the
required rounding and NaN and large positive numbers
must be dealt with too. This CL generates code that
correctly implements SP and DP round.

Test: 580-checker-round

BUG=26327751

Change-Id: Ia7518e2c30afafba4e037e2d0c21e0ce926f0425

ART: Fix Mac-specific things in x86-64 assembly

We incorrectly used double-underscore on x86-64 as VAR (used by
trampoline macros) introduced an additional underscore. Add CALLVAR
just as on x86.

Bug: 30212852
Change-Id: I2a95dd736453160769ea1a8b24c66f1738049ccb
Test: m test-art-host (Linux)
Test: builds on Mac

Clean up use of art::Exec versus execv.

Bug: b/30143290
Change-Id: I41dcaad39beea857bb606d39c1f15ba1da963e46
Test: test-art-host-gtest-oatdump_test

Merge "Implement single-precision round intrinsic in x86"

Implement single-precision round intrinsic in x86

Rationale:
X86 does not provide a direct instruction for the
required rounding and NaN and large positive numbers
must be dealt with too. This CL generates code that
correctly implements SP round in a reasonably
efficient manner (I hope....)

Test: 580-checker-round

BUG=26327751

Change-Id: Ic5f4d9cff9c27c855a8ad577c51ed3ed37fb60cd

Merge "Thumb2: Clean up 16-bit LDR/STR detection."

Merge "x86/x86-64: Fix suspend entrypoint assembly for Mac build."

Thumb2: Clean up 16-bit LDR/STR detection.

The 16-bit LDR/STR sp, #N can be used for N = (imm8 << 2)
but it was previously used only for N < 128. This cleans up
the 16-bit instruction availability calculation for this
case as well as for byte and half word loads and stores.
However, only the LDR/STR sp, #N has impact on the code we
currently generate in Optimizing.

Reduces aosp_hammerhead-userdebug boot.oat by 108KiB.

Test: Run ART test suite on Nexus 5.
Change-Id: Ie06d130a48efad10c23f8ff2596a51efd98d0761

x86/x86-64: Fix suspend entrypoint assembly for Mac build.

Bug: 30212852
Change-Id: I89f1003e73ac42c0e8d63d0fe2f2af4f24c0733d

Merge "Change suspend entrypoint to save all registers."

Change suspend entrypoint to save all registers.

We avoid the need to save/restore registers in slow paths
and get significant code size savings. On Nexus 9, AOSP:
  - 32-bit boot.oat: -1.4MiB (-1.9%)
  - 64-bit boot.oat: -2.0MiB (-2.3%)
  - other 32-bit oat files in dalvik-cache: -200KiB (-1.7%)
  - other 64-bit oat files in dalvik-cache: -2.3MiB (-2.1%)

Test: Run ART test suite on host and Nexus 9 with gc stress.
Bug: 30212852
Change-Id: I7015afc1e7d30341618c9200a3dc9ae277afd134

Merge "Update known buildbot failures for ProcessBuilderTest"

Update known buildbot failures for ProcessBuilderTest

Also updated the comment string based on our latest understanding.
This is a native crash. Both of the affected tests as well as the
other tests in ProcessBuilderTest run 'sh' as a child process.
Only one of the tests that are affected to far accesses /proc/,
which had previously been suspected as related to the cause.

The statement that this is a native crash is based on there not
being a relevant stacktrace in logcat.

A 'sh' related native crash failure occurs in the logs but that
is not specific to these tests so it's not clear if it is related.
If not, it's not clear why the additional native crash doesn't
show up in the logs.

Test: Nothing

Bug: 30657148

Change-Id: I8347edab3cacd060baf1087a1939926f70fdd349

Merge "Note known failures for TCP_USER_TIMEOUT tests on host"

Merge "libcore_failures: Remove obsolete entries."

libcore_failures: Remove obsolete entries.

These tests were failing with the OpenJDK switch, but have since
been fixed.

bug: 26326992
test: art/tools/run-libcore-tests.sh --mode=host --variant=X32
Change-Id: I9210d3fd428e0e462d95c2a620fe014e26bb942c

Merge "Fix CC table lookup tests"

Fix CC table lookup tests

Only set the mark bits for baker barrier, not sure why this
doesn't work for table lookup barrier yet.

Bug: 30162165

Test: test-art-host with CC table lookup

Change-Id: I4309ac22bfc0f8a5c9bf081365e3e38da43eee30

Merge "Change one read barrier bit to mark bit"

Change one read barrier bit to mark bit

Optimization to help slow path performance. When the GC marks an
object through the read barrier slow path. The GC sets the mark bit
in the lock word of that reference. This bit is checked from the
assembly entrypoint the common case is that it is set. If the bit is
set, the read barrier knows the object is already marked and there is
no work to do.

To prevent dirty pages in zygote and image, the bit is set by the
image writer and zygote space creation.

EAAC score (lower is better):
N9: 777 -> 700 (average 31 of runs)
N6P (960000 mhz): 1737.48 -> 1442.31 (average of 25 runs)

Bug: 30162165
Bug: 12687968

Test: N9, N6P booting, test-art-host, test-art-target all with CC

Change-Id: Iae0cacfae221e33151d3c0ab65338d1c822ab63d

Merge "Use ArrayInstance.getLength method."

Merge "ARM64: Use the zero register in the parallel-move resolver."

Note known failures for TCP_USER_TIMEOUT tests on host

TCP_USER_TIMEOUT is not currently defined on the prebuilt tcp.h (at least
not on x86-64), so its value at the non-native level remains 0, breaking
tests that rely on it.

Devices kernels and bionic support it, therefore these tests are only
expected to break on host.

Test: Nothing. Buildbot will let us know soon enough if the expected
      failures do not match reality.

Bug: 30402085
Change-Id: Iacaf63ab341c6e4a41c7d21fb3064d58ada94800

Merge "Fix constant type"

Merge "Fix an assertion in the non-Baker read barrier ARM64 slow path."

Use ArrayInstance.getLength method.

Now that perflib provides it as a better way to get the length of an
array instance.

Change-Id: I6bf8bb5541229e16f36fa8ede0a07cd0eb7b931d
Test: m ahat-test
Bug: 30481345

Fix constant type

Remove the unnecessary cast to long for DCHECK, a nit that was omitted
from the last CL, https://android-review.googlesource.com/#/c/248243/.

Bug: b/28856653
Test: test-art-host-gtest-dex_file_test
Change-Id: I8ca18d19cbac61e40ddfbe32729aef5c20347174

Merge "ART: Extend run-test timeout logging time"

Merge "Scope the compilation and Create the ProfilingInfo in the loop."

Fix an assertion in the non-Baker read barrier ARM64 slow path.

Test: m ART_USE_READ_BARRIER=true ART_READ_BARRIER_TYPE=TABLELOOKUP test-art-target (ARM64).
Bug: 30563701
Change-Id: Ib7969bd78e9c14cc2be9d5527d80385486b4f409

ARM64: Use the zero register in the parallel-move resolver.

When moving zero to a stack slot, this will change

    mov temp, #0
    str temp, [...]

into

    str zr, [...]

Change-Id: I2211b00d70f3fa0a02e781c90198757290b2bf89

Merge "Return error directly when uncompressed length is 0"

Return error directly when uncompressed length is 0

If uncompressed length is 0, in function MemMap::MapAnonymous, it will generate a MemMap instance whose base_begin is nullptr.
Then, when OpenMemory, it will create a DexFile instance.
At DexFile::DexFile constructor, header_ is asigned to 0, then header_->string_ids_off_ will cause crash.

Bug: b/28856653
Test: test-art-host-gtest-dex_file_test
Signed-off-by: ganxiaolin <ganxiaolin@xiaomi.com>
Change-Id: Id37f7629f4646cbc385ef054cb83b15be4c59b00

Scope the compilation and Create the ProfilingInfo in the loop.

We need to release the mutator lock to avoid being in a deadlock.

Also allocating the profiling just before compiling in the unlickely
event that it got collected after releasing the mutator lock.

bug:30351473
Change-Id: I98dba646536d857152949b92f6c69ff9a03cf783

Merge "Improve the graph visualizer's output for constant locations."

Merge "ARM64: Make the VIXL macro assembler part of ART ARM64's assembler."

ART: Extend run-test timeout logging time

Double the time a test is allowed to attempt logging after it
timed out to two minutes.

Also change from using nested timeouts to using "-k" in general,
not just for host dex2oat.

Bug: 30351473
Test: m test-art-host
Change-Id: I134c7b220f69eaab7ea430a5c7ce45d77b091996

Merge "Split art-run-tests between building and packaging"

Merge "Align method code rather than method header in oat files."

Merge "ART: Fix Mips Thread offset checks"

ARM64: Make the VIXL macro assembler part of ART ARM64's assembler.

This avoids a dynamic allocation of the VIXL macro assembler.

Change-Id: I4cd62678d0978f1ad6f32ea0ce7279e09152be38

ART: Fix Mips Thread offset checks

Follow-up to commit 542451cc546779f5c67840e105c51205a1b0a8fd.

Bug: 30373134
Test: m test-art-host
Change-Id: I1d0202896e683cb20db30e8aac1c5ebcacc96467

Improve the graph visualizer's output for constant locations.

Change-Id: I423fb378ee61fb53c3b328fc74f4e95cdef0992a

Merge "DexFile: Add basic support for v38 dex files."

Merge "ARM: Embed 0.0 in VCMP."

ARM: Embed 0.0 in VCMP.

Test: Run ART test suite on Nexus 5.
Change-Id: I5cbbd98c4d64a4d9213e27adcae929ead5099a39

Align method code rather than method header in oat files.

This has always been the intent. For example, the function
name CompiledMethod::AlignCode() shows this intent.

Test: Run ART test suite on host and Nexus 9.
Change-Id: I11ae8963fc537fee853fe82e3aca22e77907eae7

DexFile: Add basic support for v38 dex files.

Makes sure they aren't rejected, essentially. Also adds positive
and negative test cases in dex_file_test.

bug: 30550796
Change-Id: I8c95055a22f8a1435868c8bf3497f9641b7bb900
test: make test-art-host

Merge "dex_instruction: remove unused parameter from instruction definitions."

dex_instruction: remove unused parameter from instruction definitions.

bug: 30550796
test: make test-art-host
Change-Id: Ib80a2d5c548578cefb0552b597e1d50b813ce8ef

Split art-run-tests between building and packaging

We remove the 'obj/PACKAGING' directory during an installclean. Instead
of forcing a rebuild of all of these tests after an installclean, build
them in a different obj directory, then copy them into the PACKAGING
directory to be gathered together by LOCAL_PICKUP_FILES.

The built directory isn't used directly for LOCAL_PICKUP_FILES, since we
don't clean up tests that are no longer defined, so an installclean can
fix that. This matches the behavior of modules installed into the system
image, etc.

This brings my build time for "m -j55 tests" from 3m30s to 35s after
"m installclean".

The total size of these binaries are small (~10MB), so the extra copy is
insignificant.

Change-Id: I20f1c09df25b70df8c7f86167480b3a39e99642a

Merge "ART: Convert pointer size to enum"

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

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

Reverting to unbreak golem builds.

This reverts commit 3c92b129047b12f0887a2068b3800d7a7dcb7d78.

Change-Id: I78ef481f57fbcd3b3a9f4802dd6c21dabb186856

ART: Convert pointer size to enum

Move away from size_t to dedicated enum (class).

Bug: 30373134
Bug: 30419309
Test: m test-art-host
Change-Id: Id453c330f1065012e7d4f9fc24ac477cc9bb9269

Merge "Fixed bug in disassembly of roundss/roundsd"

Fixed bug in disassembly of roundss/roundsd

Rationale:
These instructions should be marked as load, so that, using
Intel syntax, destination (xmm0) appears at left hand side, as in
   roundss xmm0, xmm1
and not the other way around. First I suspected a bug in the
encoding (hence the test) and even the register allocator, but
since the code behaved correctly, only disassembly was really wrong.

Test: disassembler_x86_test (but nothing for actual disassembly)

BUG=26327751

Change-Id: I060ef57f4d5a64cdc04b97ae8a799d1c0d22da05

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."

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

Merge "ART: remove gcc cruft from the makefiles"

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

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"

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"

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""

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"