Merge "Fix art_quick_instrumentation_exit stub for x86_64"

Fix art_quick_instrumentation_exit stub for x86_64

Restores callee-saved registers.

Bug: 19708384
Change-Id: I1cb47b1cc616af613816c4ee041bdfc975bf9f20

Merge "Run the natural loop recognizer before building SSA."

Run the natural loop recognizer before building SSA.

SSA building checks the consistency of the graph when dealing
with dead phis.

Fixes continuous AOSP builds with optimizing.

Change-Id: Ia9a0f0adc24a8e144e54444e090ad828b9b40040

Merge "Revert "Opt Compiler: Materialise constants that cannot be encoded""

Revert "Opt Compiler: Materialise constants that cannot be encoded"

Fails building the core image.

This reverts commit 758c2f65805564e0c51cccaacf8307e52a9e312b.

Change-Id: Ic3ebd8a08a3d17a513d820035b430f6de4125866

Merge "Give an expected type to phis created for multiple returns."

Give an expected type to phis created for multiple returns.

When inlining, we used to take the type of the inlined method
for the phi in case of multiple returns. I recently changed the
logic of phi types to only be of int/float/double/ref, so we
need to call ToPhiType when creating the phi.

Change-Id: I960067ca8a8814509c2a7c52c08387d892ebf4a3

Merge "Opt Compiler: Materialise constants that cannot be encoded"

Merge "ART: Check that boot image classes are verified"

Merge "Rosalloc thread local allocation path without a cas."

Rosalloc thread local allocation path without a cas.

Speedup on N4:
MemAllocTest 3044 -> 2396 (~21% reduction)
BinaryTrees  4101 -> 2929 (~26% reduction)

Bug: 9986565
Change-Id: Ia1d1a37b9e001f903c3c056e8ec68fc8c623a78b

Merge "Properly compile multidex for tests when using Jack."

Merge "ART: Fix finalizable class in Unstarted Runtime"

ART: Fix finalizable class in Unstarted Runtime

Add a finalizable check for emulation of Class.newInstance, as this
is not allowed in transactional mode.

Change-Id: I9633929bf484ac8807b97209ab4b422c320b04da

Merge "ART: Fix AbortTransaction in Unstarted Runtime"

ART: Fix AbortTransaction in Unstarted Runtime

Can only call AbortTransaction if in a transaction. Should log
something else otherwise and fail.

Change-Id: I31ea3558c38fb6284ee6cacd1eafe01910e0965e

Merge "Fix incompatible class change error for JIT stress mode"

Fix incompatible class change error for JIT stress mode

There was a problem with miranda methods, when we would dequicken to
one of these, it wouldn't resolve as virtual during the method
lowering resolve. The solution is to try resolving as interface if we
fail to resolve as virtual.

Fixed a bug in dequickening where unreachable register lines with
quick invokes would cause CHECK failuers. In this case we punt to the
interpreter (test 435-try-*).

Added test regression test. Example failure:
java.lang.IncompatibleClassChangeError: The method
'void Main$TheInterface.m()' was expected to be of type virtual but
instead was found to be of type interface (declaration of
'java.lang.reflect.ArtMethod' appears in
out/host/linux-x86/framework/core-libart-hostdex.jar)
  at Main.DoStuff(Main.java:37)
  at Main.main(Main.java:44)

Bug: 17950037

Change-Id: I39c32cc8849bf02032a4f61a7ce57462b7fcac75

Properly compile multidex for tests when using Jack.

Bug: 19687200
Change-Id: I8c5771ccd474aa06001c3a48cc82550589ac7055

Merge "Fix a bug in the SSA builder."

Fix a bug in the SSA builder.

The build would leave behind phis with incompatible input types
(for example float and int). We need another dead phi run after
the type propagation to ensure all such phis are dead.

Change-Id: I6ef1da725c7d4a1ebaf6b52dd7eb0c7bacd261b2

Opt Compiler: Materialise constants that cannot be encoded

The VIXL MacroAssembler deals gracefully with any immediate. However
when the constant has multiple uses and cannot be encoded in the
instruction's immediate field we are better off using a register for
the constant and thus sharing the constant generation between multiple
uses.

Eg:
  var += #Const;    // #Const cannot be encoded.
  var += #Const;

Before:                 After:
  mov wip0, #Const        mov w4, #Const
  add w0, w0, wip0        add w0, w0, w4
  mov wip0, #Const        add w0, w0, w4
  add w0, w0, wip0

Change-Id: I8d1f620872d1241cf582fb4f3b45b5091b790146
Signed-off-by: Serban Constantinescu <serban.constantinescu@arm.com>

Merge "Fix sampling profiler race condition"

Fix sampling profiler race condition

Thread 1 is running RunSamplingThread and has just read trace into
the_trace.

Thread 2 is calling Trace::Stop and has just suspended all the
threads. At this point thread 1 is blocked on the SuspendAll.
Thread 2 goes and deletes the trace which Thread 1 still has a
pointer to, calls ResumeAll(). At this point thread 1 suspends the
threads and adds samples to the just deleted trace.

The fix is to join the thread before we delete the trace.

Bug: 18950006
Change-Id: I3090c4dac392a4e5d880c4dc8d9385aef53c7425

Merge "ART: Missing include"

ART: Missing include

Fix missing include.

Change-Id: I439bdf3a5c7dc2a710f3dfa0dca494ac3698de60

Merge "ART: Refactor unstarted runtime"

ART: Refactor unstarted runtime

Refactor and clean up unstarted runtime.

Bug: 19542228
Change-Id: Ib3e4b3517e06e8242d4fed32ca59419fef553a47

Merge "Revert "Revert "[optimizing] Enable x86 long support."""

Revert "Revert "[optimizing] Enable x86 long support.""

This reverts commit 154552e666347d41d95d7619c6ee56249ff4feca.

Change-Id: Idc726551c249a888b7ff5fde8508ae50e81b2e13

Merge "patchoat: DisableAutoClose when patching in place"

patchoat: DisableAutoClose when patching in place

Installd can give the same input and output when
patching directly in the dalvik-cache.
For this particular case, we need to DisableAutoClose
to avoid closing twice the same fd.

Change-Id: Ic1c0f1006f776e6aeb3772c83c52309c78a4bdfd
Signed-off-by: Julien Delayen <julien.delayen@intel.com>
Signed-off-by: Guilhem IMBERTON <guilhem.imberton@intel.com>

Merge remote-tracking branch 'goog/stage-aosp-master' into HEAD

Merge "Fix dex2dex logic"

Fix dex2dex logic

There was a bug in the dex2dex logic where would only dex2dex if
IsCandidateForCompilation was true or method verifier had check casts.

The bug was caused by not adding verification results, which caused
verification_results_->GetVerifiedMethod to not have verified methods
in most cases for interpreter-only and verify-none compilation.

Caffeinemark interpret-only before:
Sieve score = 1184
Loop score = 630
Logic score = 3519
String score = 1532
Float score = 882
Method score = 686
Overall score = 1159

After:
Sieve score = 2820
Loop score = 1942
Logic score = 4694
String score = 1679
Float score = 2288
Method score = 1294
Overall score = 2244

Bug: 17950037

Change-Id: I535ab8784b434b0a8e3f66e3ab4936d10a1b04bf

am d82d5755: Merge "Test uses libart.so, so can\'t be run NDEBUG."

* commit 'd82d5755bbfa272d4bfe0144df1ba713e02f8335':
  Test uses libart.so, so can't be run NDEBUG.

Merge "Test uses libart.so, so can't be run NDEBUG."

Test uses libart.so, so can't be run NDEBUG.

Change-Id: I22c7391e07a9d06a8ce9a4d23a21ba9f60bcfe64

am 0b2bfbd4: Merge "Fix tests now that the SSA builder is pruning more phis."

* commit '0b2bfbd440f86c8c0c7ab41b74d5557edf6e9d49':
  Fix tests now that the SSA builder is pruning more phis.

Merge "Fix tests now that the SSA builder is pruning more phis."

Fix tests now that the SSA builder is pruning more phis.

Change-Id: I503400e7343322e31ad393d51367b404d6ed7bd8

am 21c7906e: Merge "ART: Relax test 703"

* commit '21c7906e08b801f1afcb62bdd1f1763c93e5ea06':
  ART: Relax test 703

Merge "ART: Relax test 703"

am 48e702fe: Merge "Make the SSA builder honor the debuggable flag."

* commit '48e702fe11458b334e6abcc8f562729ccb5b27dd':
  Make the SSA builder honor the debuggable flag.

Merge "Make the SSA builder honor the debuggable flag."

am 39d9fe2e: Merge "Add way to print JIT related info at shutdown"

* commit '39d9fe2eb3552a002c53ed41701c6faffe3cd75a':
  Add way to print JIT related info at shutdown

Merge "Add way to print JIT related info at shutdown"

am cb85ad66: Merge "Use quick compiler to generate odex for oat file assistant tests."

* commit 'cb85ad6673a0b0815fe40100a14984fd187a5f89':
  Use quick compiler to generate odex for oat file assistant tests.

Merge "Use quick compiler to generate odex for oat file assistant tests."

am 9a5342ef: Merge "Revert "Turn on logging for oat file assistant tests.""

* commit '9a5342ef8535b54305dab72e2ac1b3412a053f9d':
  Revert "Turn on logging for oat file assistant tests."

Use quick compiler to generate odex for oat file assistant tests.

This is needed to generate non-PIC odex files, because optimizing
compiler always generates PIC.

Change-Id: Ic77a0faef6b36f13c61607327640e3378a7ee75e

Merge "Revert "Turn on logging for oat file assistant tests.""

Revert "Turn on logging for oat file assistant tests."

This reverts commit f45599d8f92748c5c2d2aec515948932c52444eb.

Change-Id: I33c6c5bf1d2643bdf8ce13708f376d01599c2a98

Make the SSA builder honor the debuggable flag.

This requires to properly type phis that are only
used by environments, and discard phis with incomptable types.
The code generators do not handle these conflicting types. In
the process, ensure a phi has a type that does not depend
on the order of the inputs (for example (char, short) -> short),
and set int for int-like types. We can refine this later.

Change-Id: I60ab601d6d00b1cbf18623ee4ff1795aa28f84a1

ART: Relax test 703

NaNs are a set of numbers. Do not expect one specific one as the
result of a NaN computation.

Change-Id: Ib98d61ea1fe8c2dd80027d1a23b8a77bfb248a27

am f719fdd2: Merge "[MIPS64] Fix GenInlinedPeek and GenInlinedPoke functions."

* commit 'f719fdd20e6282f3579cb99529bb65e852612647':
  [MIPS64] Fix GenInlinedPeek and GenInlinedPoke functions.

Merge "[MIPS64] Fix GenInlinedPeek and GenInlinedPoke functions."

am 7e9c99bd: Merge "Turn on logging for oat file assistant tests."

* commit '7e9c99bd850e22b8a5e2b5419bcf0b1abf0e29e5':
  Turn on logging for oat file assistant tests.

[MIPS64] Fix GenInlinedPeek and GenInlinedPoke functions.

Change-Id: I4994bb2f9075cb3b6488e5a5b88dbed17b80595a

Merge "Turn on logging for oat file assistant tests."

Turn on logging for oat file assistant tests.

Temporary to debug test failure.

Change-Id: Idbfcdbdae5e0aa3544b01f1a1ab0297a81d736d5

am 2af72de8: Merge "Remove ThrowLocation."

* commit '2af72de83957fb52426b8f9144e9814d35b78951':
  Remove ThrowLocation.

Merge "Remove ThrowLocation."

am 072e48ed: Merge "Create OatFileAssistant class for assisting with oat files."

* commit '072e48ed63aa4f99bec1dbbe8a7872a4bcc03e41':
  Create OatFileAssistant class for assisting with oat files.

am e8e42f35: Merge "JDWP: allocate DebugInvokeReq only when requested"

* commit 'e8e42f3548fd894f860912bb1b71ce6fa2d7daf3':
  JDWP: allocate DebugInvokeReq only when requested

Remove ThrowLocation.

Note that this is a cleanup change, and has no functionality change.
The ThrowLocation had no use anymore.

Change-Id: I3d2126af1dc673cec3a0453ff3d56a172663a5f6

Merge "Create OatFileAssistant class for assisting with oat files."

Merge "JDWP: allocate DebugInvokeReq only when requested"

am 54a8cc68: Merge "Trim arenas for JIT"

* commit '54a8cc689dc23f346c9aac0d5fc6f47e61df8cad':
  Trim arenas for JIT

Add way to print JIT related info at shutdown

Added a runtime option called -XX:DumpJITInfoOnShutdown with prints
various interesting JIT statistics during the shutdown of the runtime
if enabled.

Example of running EvaluateAndApplyChanges with the option:
I art     : Code cache size=427KB data cache size=84KB num methods=1598
I art     : Start Dumping histograms for 1598 iterations for JIT timings
I art     : Compiling: Sum: 727.850ms 99% C.I. 135us-4402us Avg: 455.475us Max: 15567us
I art     : Verifying: Sum: 191.074ms 99% C.I. 19us-1450.500us Avg: 119.570us Max: 5142us
I art     : TrimMaps:  Sum: 62.053ms 99% C.I. 13us-199.374us Avg: 38.831us Max: 10058us
I art     : MakeExecutable:  Sum: 11.153ms 99% C.I. 4us-47us Avg: 6.979us Max: 47us
I art     : Initializing:  Sum: 7.991ms 99% C.I. 1us-49.781us Avg: 5us Max: 4852us
I art     : Done Dumping histograms

Bug: 17950037
Change-Id: I23538b24f947e5f6934f55579b8baa9d806eb3d5

Merge "Trim arenas for JIT"

Create OatFileAssistant class for assisting with oat files.

The oat file assistant is used for determining whether dex2oat or
patchoat is needed, for running dex2oat or patchoat as needed to make
an oat file up to date, and to load dex files associated with a given
dex location.

The introduction of the OatFileAssistant class is meant to clean up and
consolidate code related to the management of oat files that was
duplicated and spread across dalvik_system_DexFile.cc and
class_linker.cc.

Bug: 11301553
Change-Id: I0c16027b9bae4570c2c50faa9c14f581c0cbafb8

Trim arenas for JIT

Moved arena pool into the runtime. Added arena trimming to arena
pool. When called, this madvises the used memory.

Changed the JIT compiler to trim arenas after compilation. Changed
the arena mmap name to dalvik-LinearAlloc.

Native PSS before:
  80353 kB: Native
  80775 kB: Native
  78116 kB: Native
After:
  73357 kB: Native
  70181 kB: Native
  70306 kB: Native

Bug: 17950037
Bug: 17643507
Bug: 19264997

Change-Id: I63e7a898fd6e909c2c677fa57b5917a7b1398930

am b3226684: Merge "ART: Fix init-failure graph tool"

* commit 'b3226684f0ee69094f896cb99a9d1f12585eac42':
  ART: Fix init-failure graph tool

Merge "ART: Fix init-failure graph tool"

am 81d38f74: Merge "Use the current Java stack instead of the ThrowLocation."

* commit '81d38f74219774d87389f83f23595ffc8fe814b1':
  Use the current Java stack instead of the ThrowLocation.

ART: Fix init-failure graph tool

Fix missing entries when the failed class doesn't appear in the
stack trace.

Add coloring for loadLibrary and getProperty.

Change-Id: I35df917ac8b1ada1f6eb2e418fa6ff9ca6b17048

Merge "Use the current Java stack instead of the ThrowLocation."

Use the current Java stack instead of the ThrowLocation.

The ThrowLocation contains information we can actually retrieve
from the current stack.

Change-Id: I7e5b2f519f95830c457cb31ea10a581aa9c973ee

JDWP: allocate DebugInvokeReq only when requested

Only allocates thread-local DebugInvokeReq when the debugger requests
a thread to invoke a method. The JDWP thread allocates that structure
then attaches it to the target thread. When the thread is resumed, it
executes the method. Once the invocation completes, the thread
detaches the DebugInvokeReq, signals the JDWP thread then suspends.
Finally, the JDWP thread wakes up, prepares the reply with the invoke
result (or exception) and deallocates the DebugInvokeReq.

Also ensures GC safety for object returned by the invoke. We add the
object to the JDWP object registry right after the invoke. We now
reference that object with a JDWP ObjectID instead of an Object* in
the DebugInvokeReq struct. This prevent from accessing a stale
reference if the GC runs and moves the Object*.

This CL includes the following changes:
- Move former DebugInvokeReq::ready flag to
  Thread::tls_32bit_sized_values::ready_for_debug_invoke. It's needed
  to know whether a thread has been suspended by an event, thus ready
  to invoke a method from the debugger.
- Remove DebugInvokeReq::invoke_needed: we now test if we attached a
  DebugInvokeReq* to the thread.
- Rename misleading FinishMethod function to RequestMethod.

Bug: 19142632
Bug: 18166750
Change-Id: I351fb4eb94bfe69fcafb544d21d55ff35a033000

am 2cfdabd2: Merge "Fix run-test with JIT enabled"

* commit '2cfdabd2bb4833d7092819d27ef08a9e1cdffead':
  Fix run-test with JIT enabled

Merge "Fix run-test with JIT enabled"

Fix run-test with JIT enabled

Uses -Xusejit:true instead of -Xjit.

Bug: 17950037
Change-Id: I8280245e3e2aeced5927ade1632b7a110b040136

am 504b1731: Merge "Revert "Revert "More Makefile fun for --debuggable."""

* commit '504b1731dc8f115d7a6d6d54da6fa6a0e3f4b0c9':
  Revert "Revert "More Makefile fun for --debuggable.""

Merge "Revert "Revert "More Makefile fun for --debuggable."""

Revert "Revert "More Makefile fun for --debuggable.""

This reverts commit 9728a930fdf717cca60b48d9c2b715d0eed497fd.

Change-Id: Ieca964a793e58f11931f78f50be47a4a9f0dc21f

am bf40b5ba: Merge "Revert "More Makefile fun for --debuggable.""

* commit 'bf40b5ba88dff872f252c975dd6570571c7d5a90':
  Revert "More Makefile fun for --debuggable."

am d633384a: Merge "Revert "Fix braino in Makefile.""

* commit 'd633384abccd91224b93a3ff2f970f95611280d1':
  Revert "Fix braino in Makefile."

Merge "Revert "More Makefile fun for --debuggable.""

Merge "Revert "Fix braino in Makefile.""

Revert "Fix braino in Makefile."

This reverts commit 64657d2d840e5d4e6b4b71d5df6ee0fcc4a8afab.

Change-Id: I5b778027538a4247cf51eff338220328e741074f

Revert "More Makefile fun for --debuggable."

This reverts commit bd648cd3aef1ce82aaaa85924b1178a7f499cc29.

Change-Id: I9be508730dc5c25b9d08f914c5628e88bc4aa15b

am c4ab6b04: Merge "Fix braino in Makefile."

* commit 'c4ab6b04a96e0f4dc6a86ca869b545bfa3ec9391':
  Fix braino in Makefile.

Merge "Fix braino in Makefile."

Fix braino in Makefile.

They are such a joy to work with.

Change-Id: I1a982c97fb9e2e9f34f921e7f6b481988a57f858

am 5c9d8f0d: Merge "Compute the right catch location for the debugger."

* commit '5c9d8f0d2948c2b4fe4258f96e3598ad5b8fa23b':
  Compute the right catch location for the debugger.

Merge "Compute the right catch location for the debugger."

Compute the right catch location for the debugger.

Also remove tls ThrowLocation, it is not needed anymore.

Change-Id: I78fddf09ce968ca475e39c17fa76d699c589c8d9

am 3dd536ff: Merge "More Makefile fun for --debuggable."

* commit '3dd536ffd7e8f82e4587964545b272acbd61d68e':
  More Makefile fun for --debuggable.