am 01b7c431: Merge "ART: Print maps directly to log"

* commit '01b7c431ed85e0eae4554f6fc0f79a78bc82f31f':
  ART: Print maps directly to log

Merge "ART: Print maps directly to log"

am ec27dc12: Merge "ART: Nano optimization of LiveInterval"

* commit 'ec27dc12124d57283d8f86e5386c95896b01eb23':
  ART: Nano optimization of LiveInterval

Merge "ART: Nano optimization of LiveInterval"

ART: Nano optimization of LiveInterval

Shuffling the order of conditions in the FirstIntersectionWith method
of LiveInterval class can save a couple of comparisons. Even though
this is a tiny optimization, it can save some compilation time since
the method is heavily used during register allocation.

Change-Id: I1817bd95db2d0eb96cc06fb2e9e06ac1cea784fe

am 2cfa9f27: Merge "ART: Update description of failing libcore tests"

* commit '2cfa9f2743f4a120070e83b1e34cbec22742780d':
  ART: Update description of failing libcore tests

Merge "ART: Update description of failing libcore tests"

ART: Update description of failing libcore tests

One of the tests that was described as failing on volantis fails on
hammerhead too. It was moved to a new category.

Change-Id: Ie58266d51d3aa180a36a460035ea7f016c0b7e3d

ART: Print maps directly to log

Do not read proc maps into a string before printing them later back
to the log. In low-memory situations this can cause a bad_alloc.

External bug: http://b.android.com/153990
Bug: 19494774

Change-Id: Ie63d8788afe8c9da65b30b2f89c50d3dbb820755

am 42f8c1c3: Merge "ART: Fix "unused parameters""

* commit '42f8c1c39b74aae50714488525d14de30f9722c9':
  ART: Fix "unused parameters"

Merge "ART: Fix "unused parameters""

ART: Fix "unused parameters"

GCC 4.8 decides that parameters for functions implemented with
"= default" are unused. This currently only impacts x86, but remove
the parameter names anyways.

Change-Id: I01865faa81af68c4c0e0b1cb1fb19e88ef548769

am 8fa836e1: Merge "Print thread name for attempting to unregister unattached thread"

* commit '8fa836e16e65aca3aeaa1c4784157207188e6373':
  Print thread name for attempting to unregister unattached thread

Merge "Print thread name for attempting to unregister unattached thread"

am 79a3082b: Merge "Delete bad DCHECK"

* commit '79a3082b1cf1f1e84d599f3c2d39178b8787db3f':
  Delete bad DCHECK

Merge "Delete bad DCHECK"

Delete bad DCHECK

The declaring class of a super field could be in a different dex file.
Fixes sdk build.

Change-Id: I8258d64eeff6539afb52448595f5a6ec4c71a6bc

Print thread name for attempting to unregister unattached thread

Bug: 18713034
Change-Id: Ic0654c6569277e5bbd6d2582140595faaa36cded

am 510be430: Merge "Revert "Revert "Add JIT"""

* commit '510be430dc4e68c09b56e5636e97a76380927e1d':
  Revert "Revert "Add JIT""

Merge "Revert "Revert "Add JIT"""

am 17b01497: Merge "ART: Add -Wunused"

* commit '17b01497b29f980a3d515a01adcb5f47df67ac63':
  ART: Add -Wunused

Merge "ART: Add -Wunused"

ART: Add -Wunused

Until the global CFLAGS are fixed, add Wunused. Fix declarations
in the optimizing compiler.

Change-Id: Ic4553f08e809dc54f3d82af57ac592622c98e000

Revert "Revert "Add JIT""

Added missing EntryPointToCodePointer.

This reverts commit a5ca888d715cd0c6c421313211caa1928be3e399.

Change-Id: Ia74df0ef3a7babbdcb0466fd24da28e304e3f5af

am 354d58ba: Merge "JDWP: assert no pending exception when using JNI"

* commit '354d58ba776866ea7b1c71f0d0848d5aaa013ae3':
  JDWP: assert no pending exception when using JNI

Merge "JDWP: assert no pending exception when using JNI"

am 91d67430: Merge "ART: Disable libcore crypto test failing on target"

* commit '91d6743016a513bbe8b288923aaa46a075aed133':
  ART: Disable libcore crypto test failing on target

Merge "ART: Disable libcore crypto test failing on target"

ART: Disable libcore crypto test failing on target

Another test failing due to missing loop network device on hammerhead
and volantis bots. We disable it until the device configuration issue
is fixed.

Change-Id: I5b4610d706a8e4bdf178764f64f2d4359a83682d

JDWP: assert no pending exception when using JNI

Checks there is no pending exception when creating new JDWP ids using
JNI routines. We can create JDWP ids when executing code and report
an event like a breakpoint.

Bug: 17491155
Change-Id: I4e5851bf2261510b7776a709a2388e7ff586a77c

am 2cb551b1: Merge "not-int can also take non-int (byte and short) instructions."

* commit '2cb551b1f9ef581b0eb1ef7cbee6bfdb6d9db4b3':
  not-int can also take non-int (byte and short) instructions.

am 6726d200: Merge "ART: Fix run-test 118"

* commit '6726d200144a1c1be0058c57853f9831223991e6':
  ART: Fix run-test 118

Merge "not-int can also take non-int (byte and short) instructions."

Merge "ART: Fix run-test 118"

ART: Fix run-test 118

Need to strip error logging for no-prebuild config.

Change-Id: I870bbbf3b6c7e92996e40e45b1bfa58c7e80f709

not-int can also take non-int (byte and short) instructions.

So we should use the result-type instead if the input type
for knowning what instruction to use.

Bug: 19454010
Change-Id: I88782ad27ae8c8e1b7868afede5057d26f14685a

am a48c573d: Merge "ART: Enable more libcore tests (crypto)"

* commit 'a48c573d2351177d878e36e003f0cdf4d7f9328f':
  ART: Enable more libcore tests (crypto)

Merge "ART: Enable more libcore tests (crypto)"

am 995e7d49: Merge "Fix inlining in the presence of multiple returns."

* commit '995e7d495eb898b2812e14ad82d54597676e93fd':
  Fix inlining in the presence of multiple returns.

Merge "Fix inlining in the presence of multiple returns."

Fix inlining in the presence of multiple returns.

One return could actually return a phi, so doing a phi check for
knowing if the result phi was already created was bogus.

Bug: 19454010

Change-Id: Iee703a2d1071ae263092354465eda368e5d6770d

am b679039d: Merge "ART: Disable one more libcore test failing on hammerhead"

* commit 'b679039d4fdc739959623c2cb1dcc2efa9a1ebe6':
  ART: Disable one more libcore test failing on hammerhead

Merge "ART: Disable one more libcore test failing on hammerhead"

ART: Disable one more libcore test failing on hammerhead

One more test from the harmony package fails due to the locale issue
on hammerhead. This patch disables it.

Change-Id: I1f3941bfcc20d5b2e23fe5ae963676138eecccb8

ART: Enable more libcore tests (crypto)

This patch enables additional libcore tests run by the ART buildbots.
These are tests from the org.apache.harmony.crypto and
libcore.javax.crypto packages and take extra ~8 minutes to run on
target and ~2 minutes on host.

Change-Id: I7cd314eca654320104fee26d8c34437450814dc2

am 27a3e615: Merge "ART: Disable libcore tests failing on bots"

* commit '27a3e6153df81dd8adbc9ace70fdb3e7922b72f8':
  ART: Disable libcore tests failing on bots

Merge "ART: Disable libcore tests failing on bots"

am 2d043b35: Merge "[optimizing] Add if-context sensitivity for null popagation."

* commit '2d043b356d2e5c911892badf78aa56b843c5c3a6':
  [optimizing] Add if-context sensitivity for null popagation.

Merge "[optimizing] Add if-context sensitivity for null popagation."

ART: Disable libcore tests failing on bots

Some tests in previous commits are failing on the testing devices.
Tests dependent on DateTimeFormatter fail on hammerhead due to
misconfigured locales and tests with a server running on localhost
fail due to the loop device being switched off on volantisg. These
problems will be further investigated but this patch disables them
for the time being.

Change-Id: I2dde178925de555f26a4d532be15487f499571b4

[optimizing] Add if-context sensitivity for null popagation.

Change-Id: I3725b6c6a6cf44440c34a1bfb67e623531e665d6

am 0b6daeb0: Merge "ART: Enable more libcore tests (harmony package)"

* commit '0b6daeb0f0014474b542cbba1f713eb0dbefb7f9':
  ART: Enable more libcore tests (harmony package)

Merge "ART: Enable more libcore tests (harmony package)"

ART: Enable more libcore tests (harmony package)

This patch enables additional libcore tests run by the ART buildbots.
These are tests from the org.apache.harmony.* packages and take extra
~5 minutes to run on target and ~2 minutes on host.

Change-Id: Idb9fc09ac7df460bd4337bc5151301adddea4b97

am 69f46525: Merge "ART: Enable more libcore tests (libcore.* package)"

* commit '69f46525c7238f43eaefdb844106c2e97181c498':
  ART: Enable more libcore tests (libcore.* package)

Merge "ART: Enable more libcore tests (libcore.* package)"

ART: Enable more libcore tests (libcore.* package)

This patch enables additional libcore tests run by the ART buildbots.
These are tests from the libcore.* packages and take extra ~3 minutes
to run on both host and target.

Change-Id: I89512836adb87b20ed5f9446d3b3ee245f9162dc

am 893bee6c: Merge "JDWP: update thread synchronization"

* commit '893bee6c8f05fdf7345bf348a8f9d24e9300bf8e':
  JDWP: update thread synchronization

Merge "JDWP: update thread synchronization"

am f548c54c: Merge "Revert "Add JIT""

* commit 'f548c54c69443ba94fcee89daca6f5f499d71e9a':
  Revert "Add JIT"

Merge "Revert "Add JIT""

Revert "Add JIT"

Sorry, run-test crashes on target:
0-05 12:15:51.633 I/DEBUG   (27995): Abort message: 'art/runtime/mirror/art_method.cc:349] Check failed: PcIsWithinQuickCode(reinterpret_cast<uintptr_t>(code), pc) java.lang.Throwable java.lang.Throwable.fillInStackTrace() pc=71e3366b code=0x71e3362d size=ad000000'
10-05 12:15:51.633 I/DEBUG   (27995):     r0 00000000  r1 0000542b  r2 00000006  r3 00000000
10-05 12:15:51.633 I/DEBUG   (27995):     r4 00000006  r5 b6f9addc  r6 00000002  r7 0000010c
10-05 12:15:51.633 I/DEBUG   (27995):     r8 b63fe1e8  r9 be8e1418  sl b6427400  fp b63fcce0
10-05 12:15:51.633 I/DEBUG   (27995):     ip 0000542b  sp be8e1358  lr b6e9a27b  pc b6e9c280  cpsr 40070010
10-05 12:15:51.633 I/DEBUG   (27995):

Bug: 17950037
This reverts commit 2535abe7d1fcdd0e6aca782b1f1932a703ed50a4.

Change-Id: I6f88849bc6f2befed0c0aaa0b7b2a08c967a83c3

am 9e87baff: Merge "Delete bad DCHECK"

* commit '9e87baff2eee8e8ae0700c7391ca4447dbe142b0':
  Delete bad DCHECK

Merge "Delete bad DCHECK"

Delete bad DCHECK

The declaring class of a super field could be in a different dex file.
Fixes sdk build.

Change-Id: I6e1738603e71c3c673fa12d9aaa25c9eb84db406

am 4fa1a274: Merge "Add JIT"

* commit '4fa1a274024067e160a60d0a77124e95dca980e1':
  Add JIT

Merge "Add JIT"

am 40c99de1: Merge "ART: Strip error message in run-test"

* commit '40c99de1fd26f9ba6fae20a01175240b96fc7443':
  ART: Strip error message in run-test

Merge "ART: Strip error message in run-test"

ART: Strip error message in run-test

It seems the output is flaky - sometimes the LOG(ERROR) makes it
to output.txt, sometimes it doesn't. So instead always strip it.

Change-Id: I409b98eb9a5bf0c2e6c1df16ff3b8013a2a85cc5

Add JIT

Currently disabled by default unless -Xjit is passed in.

The proposed JIT is a method JIT which works by utilizing interpreter
instrumentation to request compilation of hot methods async during
runtime.

JIT options:
-Xjit / -Xnojit
-Xjitcodecachesize:N
-Xjitthreshold:integervalue

The JIT has a shared copy of a compiler driver which is accessed
by worker threads to compile individual methods.

Added JIT code cache and data cache, currently sized at 2 MB
capacity by default. Most apps will only fill a small fraction of
this cache however.

Added support to the compiler for compiling interpreter quickened
byte codes.

Added test target ART_TEST_JIT=TRUE and --jit for run-test.

TODO:
Clean up code cache.
Delete compiled methods after they are added to code cache.
Add more optimizations related to runtime checks e.g. direct pointers
for invokes.
Add method recompilation.
Move instrumentation to DexFile to improve performance and reduce
memory usage.

Bug: 17950037

Change-Id: Ifa5b2684a2d5059ec5a5210733900aafa3c51bca

am 39ce2282: Merge "Only detach shutdown thread if we attached it"

* commit '39ce2282ed2b020e51e8d92f442b419ae9dd6434':
  Only detach shutdown thread if we attached it

Merge "Only detach shutdown thread if we attached it"

Only detach shutdown thread if we attached it

Speculative fix to:
"Request to unregister unattached thread" bug.

Bug: 18713034
Change-Id: I6e69088dd66970c704010397a11e3758cf107ff9

am 9e80e7fa: Merge "ART: Allow the execution to stop if the compilation fails via an option"

* commit '9e80e7fa5ef60f02f35823bc58969c3d5a03453e':
  ART: Allow the execution to stop if the compilation fails via an option

Merge "ART: Allow the execution to stop if the compilation fails via an option"

ART: Allow the execution to stop if the compilation fails via an option

The current implementation continues the execution of the application if
dex2oat fails by relying on the interpreter.

This patch adds a -Xno-dex-file-fallback option to stop the default behavior.
This can be used two-fold.

First, one can enforce that a runtime only starts with a boot image. A
follow-up patch will ensure that dex2oat (for apps) and patchoat in general
request that mode and close gracefully otherwise.

Second, this can be used for testing and debugging purposes, as it ensures
that compiler failures & aborts are not silently ignored.

Add testing.

Bug: 19100590
Change-Id: Iaf07b5ccf00942ca8a8ec8687599320a3ddbc089
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>

am 84fbead4: Merge "Optimizing: Remove redundant hash set copy in GVN"

* commit '84fbead44b77c5f9f9458d88b42c8925a002dfd4':
  Optimizing: Remove redundant hash set copy in GVN

Merge "Optimizing: Remove redundant hash set copy in GVN"

am 38fee8ef: Merge "Avoid unaligned accesses (SIGBUG/BUS_ADRALN) in IRT."

* commit '38fee8ef4bc0f4dbe2c6d1f5585895f0c4d16984':
  Avoid unaligned accesses (SIGBUG/BUS_ADRALN) in IRT.

Merge "Avoid unaligned accesses (SIGBUG/BUS_ADRALN) in IRT."

am bad712e2: Merge "More checker tests for BCE."

* commit 'bad712e2576631529381280e52e36905dda5f048':
  More checker tests for BCE.

Avoid unaligned accesses (SIGBUG/BUS_ADRALN) in IRT.

Pointers in IrtEntry aren't currently aligned under 64 bit builds. But
unaligned atomic stores (store exclusive) do not work on arm64 (causes
SIGBUG/BUS_ADRALN). Fix CC collector crashes caused by this.

Bug: 12687968
Change-Id: I1d2f5376778a9a1e5cfea876f1f57d7a88ad5445

Merge "More checker tests for BCE."

More checker tests for BCE.

Also make sure the check on MonotonicValueRange narrow is more strict.
Plus some handling on array length division such as array.length/2.
Added checker tests for each case.

Change-Id: I9f32fc5f6ca1f3da8edec576de66b44d85a50bc6

am 96ac49a2: Merge "ART: Introduce NO_RETURN, Mark DoLongJump noreturn"

* commit '96ac49a260223dc00291116daa63d7489c52bb0f':
  ART: Introduce NO_RETURN, Mark DoLongJump noreturn

Merge "ART: Introduce NO_RETURN, Mark DoLongJump noreturn"

ART: Introduce NO_RETURN, Mark DoLongJump noreturn

Add NO_RETURN macro that adds C++11 noreturn attribute. Mark
DoLongJump methods as noreturn.

Change-Id: Ifde4318e370493237050d4c1349285a0382df23f

Optimizing: Remove redundant hash set copy in GVN

During the GVN analysis, a basic block inherits the set of movable
instructions from its dominator. If the block is the only successor
of the dominating block, there is no need for cloning of the parent
set (a very expensive operation).

Change-Id: I59e033b9e9e093984dc8e903e3a7be1cb3645cc2

am 304e4746: Merge "[optimizing] Fix float addition on x86."

* commit '304e47467579c909a6312ef3cca5cdad1433de9c':
  [optimizing] Fix float addition on x86.

Merge "[optimizing] Fix float addition on x86."

[optimizing] Fix float addition on x86.

Change-Id: Ic39aaae89b8e5184b98001ea67221a3564e9334a

am f9fe845f: Merge "Each primitive kind now spills to different locations."

* commit 'f9fe845f587e856acf4b3f1b19e4b7bab828b142':
  Each primitive kind now spills to different locations.

am 762374d0: Merge "Optimizing: Speed up HInstruction use removal"

* commit '762374d084cd2d0a564a4bd410b91794af6853a4':
  Optimizing: Speed up HInstruction use removal

Merge "Each primitive kind now spills to different locations."

Merge "Optimizing: Speed up HInstruction use removal"

Optimizing: Speed up HInstruction use removal

Similarly to a previous commit on HEnvironment use removal, this patch
adds links from instructions to their respective inputs' use lists for
contant-time removal at the cost of doubling the size of input lists
(from one pointer per entry to two). Manual testing shows that this
significantly reduces the time required to transform HGraph to SSA
form for some huge methods.

Change-Id: I8dc3e4b0c48a50ac1481eb55c31093b99f4dc29f

Each primitive kind now spills to different locations.

Having different slots depending on the types greatly simplifies
the parallel move resolver. It also avoids doing FPU <-> Core
register swaps, and force backends to implement such a swap.

Change-Id: Ide9f0452e7ccf9efb8adddbcc246d44b937b253c

am 73596913: Merge "Implement a simple count based boot marker."

* commit '735969139b162f9d45a3c0e47dc24a8aec63c736':
  Implement a simple count based boot marker.

Merge "Implement a simple count based boot marker."