Merge "Add missing SHARED_LOCKS_REQUIRED attibute."

Add missing SHARED_LOCKS_REQUIRED attibute.

BUG: 18093707
Change-Id: I66e5c7650c67bb308472ff0da7faed6dff4ead53

Merge "ART: Add basic tests for materialized conditions."

Merge "Fix concurrent start bytes race"

Merge "Revert "Implement long negate instruction in the optimizing compiler.""

Revert "Implement long negate instruction in the optimizing compiler."

This reverts commit 66ce173a40eff4392e9949ede169ccf3108be2db.

Fix concurrent start bytes race

Previously, we set concurrent start bytes to max int when we
requested a concurrent GC, but there was a race if another thread
was doing another GC and had already completed GrowForUtilization
but had not yet finished the GC. This meant that the thread doing
the GC would update the concurrent start bytes properly, but the
allocating thread would re-update it to max int. Then when the
concurrent GC thread woke up, it would call WaitForGcToComplete
and see that there was a collector running and avoid doing the
concurrent GC, leaving the concurrent start bytes set to max int.

This meant that there would be no more concurrent GC until either
the next explicit GC or the next GC for alloc.

The fix is to only set concurrent start bytes to max int inside of
the CollectGarbageInternal code such that there isn't any way for
two threads to race.

Bug: 17942071

(cherry picked from commit 0133ec454d8dd3fa5ffe35649b5704aa18f15a49)

Change-Id: Ibc23dd2f937a2b5fda51ed064634c034dbb6e668

ART: Add basic tests for materialized conditions.

Change-Id: I4acef30cc6a48b5fe07d55db6b9cf0d093b326ee

Merge "Implement long negate instruction in the optimizing compiler."

Implement long negate instruction in the optimizing compiler.

- Add support for the neg-long (long integer two's
  complement negate) instruction in the optimizing compiler.
- Add a 64-bit NEG instruction (negq) to the x86-64
  assembler.
- Generate ARM, x86 and x86-64 code for integer HNeg nodes.
- Put neg-related tests into test/415-optimizing-arith-neg.

Change-Id: I1fbe9611e134408a6b8745d1df20ab6ffa5e50f2

Merge "[optimizing compiler] Handle SUB_FLOAT/DOUBLE nodes"

[optimizing compiler] Handle SUB_FLOAT/DOUBLE nodes

Also add 414-optimizing-arith-sub to
TEST_ART_BROKEN_OPTIMIZING_ARM64_RUN_TESTS.

Change-Id: Ia1b7ff7857bc0e488b9b8ed8a36efb1a3a9bad36

Merge "Use the shared smali test infrastructure for 800-smali."

Merge "[optimizing compiler] Add float/double subtraction"

[optimizing compiler] Add float/double subtraction

- for arm, x86, x86_64
- add tests
- a bit of clean up

Change-Id: I3761b0d908aca3e3c5d60da481fafb423ff7c9b9

Merge "ART: optimizing compiler: initial support for ARM64."

ART: optimizing compiler: initial support for ARM64.

The ARM64 port uses VIXL for code generation, to which it defers work
like label binding and branch resolving, register type coherency
checking, and immediate values handling.

Change-Id: I0a44508c0c991f472a63e67b3469cdd878fe1a68
Signed-off-by: Serban Constantinescu <serban.constantinescu@arm.com>
Signed-off-by: Alexandre Rames <alexandre.rames@arm.com>

Merge "Remove obsolete TODOs from codegen tests"

Remove obsolete TODOs from codegen tests

The features are already exercised by the art test 411-optimizing-arith.

Change-Id: Id008931e0ed8206ced11ecc85a80a7e4aef3e68e

Merge "[native bridge] Make sure we always unload the native bridge"

[native bridge] Make sure we always unload the native bridge

libnativebridge may allocate some resources during loading and
intialization and de-allocate them when unloading. This makes sure that
we don't leak anything.

Bug: 18097480
Change-Id: I901f2d3c2ab1efb2875388f99e8c5c111ce82d5a

Use the shared smali test infrastructure for 800-smali.

No need for a special build file now.

Change-Id: I7f6813553548b48fb5f54640506d1a361d9db3c1

Merge "Implement int bit-wise not operation in the optimizing compiler."

Implement int bit-wise not operation in the optimizing compiler.

- Add support for the not-int (integer one's complement
  negate) instruction in the optimizing compiler.
- Extend the HNot control-flow graph node type and make it
  inherit from HUnaryOperation.
- Generate ARM, x86 and x86-64 code for integer HNeg nodes.
- Exercise these additions in the codegen_test gtest, as there
  is not direct way to assess the support of not-int from a
  Java source.  Indeed, compiling a Java expression such as
  `~a' using javac and then dx generates an xor-int/lit8 Dex
  instruction instead of the expected not-int Dex instruction.
  This is probably because the Java bytecode has an `ixor'
  instruction, but there's not instruction directly
  corresponding to a bit-wise not operation.

Change-Id: I223aed75c4dac5785e04d99da0d22e8d699aee2b

Merge "C++11 related clean-up of DISALLOW_.."

C++11 related clean-up of DISALLOW_..

Move DISALLOW_COPY_AND_ASSIGN to delete functions. By no having declarations
with no definitions this prompts better warning messages so deal with these
by correcting the code.
Add a DISALLOW_ALLOCATION and use for ValueObject and mirror::Object.
Make X86 assembly operand types ValueObjects to fix compilation errors.
Tidy the use of iostream and ostream.
Avoid making cutils a dependency via mutex-inl.h for tests that link against
libart. Push tracing dependencies into appropriate files and mutex.cc.
x86 32-bit host symbols size is increased for libarttest, avoid copying this
in run-test 115 by using symlinks and remove this test's higher than normal
ulimit.
Fix the RunningOnValgrind test in RosAllocSpace to not use GetHeap as it
returns NULL when the heap is under construction by Runtime.

Change-Id: Ia246f7ac0c11f73072b30d70566a196e9b78472b

Merge "Fix arm64 bug introduced by refactoring."

Merge "Tidy up logging."

Tidy up logging.

Move gVerboseMethods to CompilerOptions. Now "--verbose-methods=" option to
dex2oat rather than runtime argument "-verbose-methods:".
Move ToStr and Dumpable out of logging.h, move LogMessageData into logging.cc
except for a forward declaration.
Remove ConstDumpable as Dump methods are all const (and make this so if not
currently true).
Make LogSeverity an enum and improve compile time assertions and type checking.
Remove log_severity.h that's only used in logging.h.
With system headers gone from logging.h, go add to .cc files missing system
header includes.
Also, make operator new in ValueObject private for compile time instantiation
checking.

Change-Id: I3228f614500ccc9b14b49c72b9821c8b0db3d641

Merge "X86 Long Min/Max: Avoid calling SRegToVReg with -1"

Merge "Fix make file comment"

Fix make file comment

Change-Id: I2b2261aea00b934411979dbca1e00d5a75030407

Merge "Minor fix in codegen tests."

Minor fix in codegen tests.

Change-Id: I9b843536353d4f820b969895d5f75ee9b679aff0

Fix arm64 bug introduced by refactoring.

- Commit 1d8cdbc5202378 made changes to the parameters for
artSet64InstanceFromCode, but did not change THREE_ARG_DOWNCALL
to THREE_ARG_REF_DOWNCALL for arm64.

Change-Id: If256b0bd32dbd1c95f61e0afcb35975bb09a8b97
Signed-off-by: Stephen Kyle <stephen.kyle@arm.com>

Merge "Add multiplication for floats/doubles in optimizing compiler"

Add multiplication for floats/doubles in optimizing compiler

Change-Id: I61de8ce1d9e37e30db62e776979b3f22dc643894

Merge "Fix typo in Makefile."

Fix typo in Makefile.

Change-Id: I9133804eff8ce8e785f8551488daee1f19036952

Merge "Implement array creation related DEX instructions."

Implement array creation related DEX instructions.

Implement new-array, filled-new-array, and fill-array-data.

Change-Id: I405560d66777a57d881e384265322617ac5d3ce3

X86 Long Min/Max: Avoid calling SRegToVReg with -1

It is possible that the result of a call to min/max can't be combined
with the following move.  In that case, the destination will use the
default long return value (EAX/EDX), with a s_reg_low value of -1.
A debug compiler will assert fail in that case.

Fix: A result with no s_reg_low must be unused.  Just return with no
code generated, like X86 GenInlinedAbsFloat().

Seen compiling GmsCore.apk on the host with a debug backend.

Change-Id: I8006e822e8dcb2112d86e4047bb2e3037ba6fece
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>

Merge "Fixed debugger for threads attached from JNI"

Merge "Fix bug in UnionIfNotIn."

Fix bug in UnionIfNotIn.

Bug: 18066207

Change-Id: Ib9b24802546403b3d5a4da19996034eb45601f53

Merge "Various fixes related to integer negate operations."

Various fixes related to integer negate operations.

- Emit an RSB instruction for HNeg nodes in the ARM code
  generator instead of RSBS, as we do not need to update the
  condition code flags in this case.
- Simply punt when trying to statically evaluate a long
  unary operation, instead of aborting.
- Move a test case to the right place.

Change-Id: I35eb8dea58ed35258d4d8df77181159c3ab07b6f

Merge "ART: Print args in case of run-test build failure"

ART: Print args in case of run-test build failure

Bug: 17959926
Change-Id: If1a6bf0341248818bef6c2a11e0677803c07ad3f

Merge "ART: Fix common_build CFLAGS"

ART: Fix common_build CFLAGS

Actually use the additional art_clang_cflags.

Change-Id: I2b5c5dacfdbffaa9b9c1e7a6fdb8ad79d8536fe8

Merge "Fix Mac build."

Fix Mac build.

Why can't Apple implement named macro arguments!

Change-Id: I5e09e04449c3a5d9f6430b587c004d6b16000f09

Merge "Fix a DCHECK failure IsResolved() || IsErroneous()."

Fix a DCHECK failure IsResolved() || IsErroneous().

Bug: 17914035
Change-Id: I90877d31fdce3d3bf7b5f22f7759f9536ab14d03

Merge "Avoid strict-aliasing problems with Handles."

Avoid strict-aliasing problems with Handles.

Replace use of reinterpret_cast with down_cast.

Bug: 18074773

Change-Id: Id42d462f2798f69a2210e5912f441c868b8b5812

Merge "Const-ify functions/variables in trampoline entrypoints."

Const-ify functions/variables in trampoline entrypoints.

Change-Id: I777600c7362ab3c1dd65a695e36a4633b43033fe

Merge "ART: Fix test 089 to ignore debug lines"

ART: Fix test 089 to ignore debug lines

Bug: 17959926
Change-Id: Icc9d0fc5233937255b172b3ba4754c4e8d64a103

Merge "ART: Fix run-test script"

ART: Fix run-test script

Correctly pipe error information to the output file.

Bug: 17959926
Change-Id: Ifccf2a2a22517b2523228718be6d2a57250847c2

Merge "Fix register_allocator_test after reg alloc changes."

Fix register_allocator_test after reg alloc changes.

Change-Id: Ieaf5daf35efaff6685720a93a442cd7a152f1567

Merge "Fix off by one errors in linear scan register allocator."

Fix off by one errors in linear scan register allocator.

Change-Id: I65eea3cc125e12106a7160d30cb91c5d173bd405

Merge "Fix m test-art-host-run-test-001-HelloWorld."

Fix m test-art-host-run-test-001-HelloWorld.

Noticed by Calin, bug spotted by Andreas.

Change-Id: Id10004597a7d9605bd80d68c998add611f242510

Merge "Implement register allocator for floating point registers."

Implement register allocator for floating point registers.

Also:
- Fix misuses of emitting the rex prefix in the x86_64 assembler.
- Fix movaps code generation in the x86_64 assembler.

Change-Id: Ib6dcf6e7c4a9c43368cfc46b02ba50f69ae69cbe

Merge "Enable generic JNI for x86 and ARM when interpret-only."

Enable generic JNI for x86 and ARM when interpret-only.

Change-Id: I006ce1ce74acd0f0d53d380e28e409d24d772ea3

Merge "Clarify a confusing error message."

Clarify a confusing error message.

We didn't run out of stack; we ran out of heap so there's no stack
trace in the pre-allocated exception we're throwing.

Bug: 17975886
Change-Id: I406f84d7580a28451b232eba323f9c9978e678e6

Merge "Constant folding on unary operations in the optimizing compiler."

Merge "Implement int negate instruction in the optimizing compiler."

Constant folding on unary operations in the optimizing compiler.

Change-Id: I4b77afa2a89f5ad2eedd4d6c0c6c382585419349

Implement int negate instruction in the optimizing compiler.

- Add support for the neg-int (integer two's complement
  negate) instruction in the optimizing compiler.
- Add a HNeg node type for control-flow graphs and an
  intermediate HUnaryOperation base class.
- Generate ARM, x86 and x86-64 code for integer HNeg nodes.

Change-Id: I72fd3e1e5311a75c38a8cb665a9211a20325a42e

Merge "Remove the notion of dies at entry."

Remove the notion of dies at entry.

- Instead, explicitly say that the output does not overlap.
- Inputs that must be in a fixed register do die at entry,
  as we know they have a location that others can not take.
- There is also no need to differentiate between an input move
  and a connecting sibling move - those can be put in the
  same parallel move instruction.

Change-Id: I1b2b2827906601f822b59fb9d6a21d48e43bae27

Merge "Shut down FaultManager after shutting down threads."

Merge "Support running run-test in ndebug mode through make."

Support running run-test in ndebug mode through make.

Change-Id: I514f44f356ecd0bf64d3f5b1295a99ff52a5d7b3

Merge "Link libsigchain.a instead of sigchain.o"

Link libsigchain.a instead of sigchain.o

  Relative paths may sometimes lead to failed builds
  if sources are built in particular order.

  Also it is possible to export symbols from static lib
  with LOCAL_WHOLE_STATIC_LIBRARIES variable.

Change-Id: I6d62594c2e84020d67254747c1fd465e1144920e

Merge "Refactor quick entrypoints"

Refactor quick entrypoints

Remove FinishCalleeSaveFrameSetup.
Assembly routines write down anchor into TLS as well as placing runtime
method in callee save frame.
Simplify artSet64InstanceFromCode by not computing the referrer from the
stack in the C++ code.
Move assembly offset tests next to constant declaration and tidy arch_test.

Change-Id: Iededeebc05e54a1e2bb7bb3572b8ba012cffa1c8

Merge "Rosalloc should print unreachable page map type"

Shut down FaultManager after shutting down threads.

Bug: 17703163
Change-Id: I7a8454b61e887376d019af601f3a416d18bc0760

Merge "Add perf option to art tool"

Add perf option to art tool

'art --perf' will record user space cycle information
'art --perf-report' will also report the saved data.

When using perf, the temporary data directory is not deleted.

Change-Id: Iadceeb3883f815cac8ed2b8a96b9b8363661d156

Fixed debugger for threads attached from JNI

Dbg::PostThreadStart was not invoked in Thread::Attach,
thus when attaching existing native thread to JNI, JDWP
events from that thread are not handled correctly by the
debugger, because the thread keeps being unannounced.

The issue was reproduced on Eclipse 4.3 and 4.4.

Change-Id: I9a330f5cb96857862b18f37047ef87646a1f2aa2
Signed-off-by: Daniel Mihalyi <daniel.mihalyi@mattakis.com>

Merge "Have HInstruction::StrictlyDominates compute strict dominance."

Merge "Revert "Revert "Introduce a class to implement optimization passes."""

Merge "Just depend on TARGET_CORE_DEX_FILES."

Merge "Rename arm64 `Register` to `XRegister`."

Merge "Rework arm64 register codes and fix Arm64ManagedRegister tests."

Merge "Quick: In GVN, apply modifications early if outside loop."

Just depend on TARGET_CORE_DEX_FILES.

There is no such thing as a 2NDTARGET_CORE_DEX_FILES.

Change-Id: Idccf31a7b9d7f2329678352805b260a48851d568