Merge "Stack support for Optimizing compiler"

Stack support for Optimizing compiler

Allows to read/write DEX registers from physical register or stack
location when the method is compiled with the Optimizing compiler.

Required fixing arm and arm64 JNI compiler by saving floating
point registers.

Bug: 18547544
Change-Id: I401579f251d1c0a130f6cf4a93a960cdcd7518f5

Merge "[optimizing compiler] x86 goodness"

Merge "bce: handle a pattern for circular buffer"

bce: handle a pattern for circular buffer

Change-Id: Ie54bdd7c044af58deea0d0addaaa8186cabf3532

Merge "ART: Cache last returned range in LiveInterval::Covers"

Merge "Fix AddCurrentRuntimeFeaturesAsDex2OatArguments jit handling"

Merge "Fix bad variable expansion"

Fix bad variable expansion

Bug: 19524713
Change-Id: I6537bc650e2690e5902576fd0e5a55142d91182a

Merge "ART: Move DexFile vector to Java array"

Merge "Add support for .bss section in oat files."

Fix AddCurrentRuntimeFeaturesAsDex2OatArguments jit handling

Now jit passes interpret filter to dex2oat.

Bug: 19550422
Change-Id: I932c7d1084acbb9d356ad0f5534a270e2ca2d6ec

Merge "ART: Allow more operations in unstarted Runtime"

ART: Allow more operations in unstarted Runtime

To compile-time initialize more classes, have more dedicated code
for special methods.

Bug: 19542228
Bug: 19548084
Change-Id: Iad37c1c58302b04fa3a5a904a923da388a079cd7

Add support for .bss section in oat files.

Change-Id: I779b80b8139d9afdc28373f8c68edff5df7726ce

Merge "Opt Compiler: ARM64: Enable explicit memory barriers over acquire/release"

Merge "ART: Fix off-by-two whitespace"

[optimizing compiler] x86 goodness

Implement the x86 version of
https://android-review.googlesource.com/#/c/129560/, which made some
enhancements to x86_64 code.
- Use leal to implement 3 operand adds
- Use testl rather than cmpl to 0 for registers
- Use leaq for x86_64 for adds with constant in int32_t range

Note:
- The range and register allocator tests seem quite fragile.  I had to
  change ADD_INT_LIT8 to XOR_INT_LIT8 for the register allocator test to
  get the code to run.  It seems like this is a bit hard-coded to
  expected code generation sequences.  I also changes BuildTwoAdds to
  BuildTwoSubs for the same reason.
- For the live range test, I just changed the expected output, as the
  Locations were different.

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

ART: Cache last returned range in LiveInterval::Covers

Optimizing spends ~10% of compilation time in the register allocator.
One of the frequently called methods is LiveInterval::Covers which
has linear complexity w.r.t. the number of gaps in liveness intervals.
This patch leverages the fact that the register allocator calls Covers
with non-decreasing position values and caches the last returned
result to start the iteration closer to the result the next time the
method is invoked. Stats from compiling the framework show that this
optimization reduces the average number of iterations needed to find
the result by 40%.

Change-Id: I4dd26b900879d5e1d03818ebc1e117cc6a53053c

Merge "Don't require three-character extensions for dex locations."

Opt Compiler: ARM64: Enable explicit memory barriers over acquire/release

Implement remaining explicit memory barrier code paths and temporarily
enable the use of explicit memory barriers for testing.

This CL also enables the use of instruction set features in the ARM64
backend. kUseAcquireRelease has been replaced with PreferAcquireRelease(),
which for now is statically set to false (prefer explicit memory barriers).

Please note that we still prefer acquire-release for the ARM64 Optimizing
Compiler, but we would like to exercise the explicit memory barrier code
path too.

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

ART: Fix off-by-two whitespace

Follow-up to change 134380. Fix the whitespace to be printed. Fix
style. It's surprising the linter did not complain.

Change-Id: I3abcde3f401fa1fd4d7a847ff1f071fba23ce6a7

Merge "Revert "Revert "Re-enable one thread dumping the native stack of another."""

Merge "ART: Allow dex2oat for apps only with image"

ART: Allow dex2oat for apps only with image

Do not allow a runtime without image when compiling an app. This
avoids the current abort when we then try to run out of the (missing)
boot classpath.

Bug: 19100590
Change-Id: Ic269dc2fa807d003215ea134cb42fe4c4d78124e

Revert "Revert "Re-enable one thread dumping the native stack of another.""

This reverts commit edfdaf37d5520a3c3a858b6b3fba4f759c60dadc.

Tests seem to succeed. Might be helpful to understand some deadlock/timeout
situations.

Bug: 15446488
Bug: 18713034
Change-Id: I5be225363f698556221576b5827d9501e26799f3

Merge "ART: Fix Mips disassembler for some floating point instructions."

ART: Fix Mips disassembler for some floating point instructions.

Change-Id: I2b661a8dae4cd924c081df85f570007cf645769c

Merge "Change JIT to not use PIC"

Change JIT to not use PIC

Motivation: PIC is currenly slower and disables a lot of JIT
friendly optimizations.

EvaluateAndApplyChanges N5 eng interpreter + jit:
Before: 2540ms
After: 2350ms

Bug: 17950037

Change-Id: I3de3e7bcccfc53e49cb78370443c514248ec78d7

Merge "Change JIT tests to use interpreter image"

Change JIT tests to use interpreter image

Bug: 19528920
Change-Id: I079cfcca7bb57e492c82a44f104e41228cb33b4f

Merge "Handle another untyped dex CONST issue"

Merge "Fix jit tests"

Fix jit tests

Missing image suffix was missing causing tests to fail.
Also added missing phony targets.

Bug: 19524713
Change-Id: Ib40d39b1fc16e1adfdc01744a199ae9b0b562664

Merge "Fix JIT direct call to interpreter bridge"

Fix JIT direct call to interpreter bridge

FormulaEvaluationActions.EvaluateAndApplyChanges on hammerhead eng:
Before: 3550ms
After: 2540

Bug: 17950037
Change-Id: If7acaea20d4adb5418d00ef13192e2d4c29032af

Merge "Change card cache mod-union table to use bitmaps"

Change card cache mod-union table to use bitmaps

Previously used card sets, using bitmaps saves memory and slightly
increases performance.

Added mod union table test.

Performance EvaluateAndApplyChanges (minimal changes):

Before (card cache image mu table):
UpdateAndMarkImageModUnionTable: Avg: 524.320us
ImageModUnionClearCards: Avg: 54.580us
Native PSS: ~67500kB

After (card cache image mu table):
UpdateAndMarkImageModUnionTable: Avg: 515.600us
ImageModUnionClearCards: Avg: 53.780us
Native PSS: ~66014kB

Native PSS was higher before since the mod_union_table->SetCards()
which happens pre zygote fork was allocating a large amount of
std::nodes.

Bug: 11859910

Change-Id: I956b7e51d5572feec1393ffa618b7b7d8c147b28

Merge "Add standard gc log for transition collections."

Add standard gc log for transition collections.

Factor out the gc log logic into a function.

Note we still don't log in background as before but do for things like
OOME compactions in foreground.

Change-Id: Ifd35a796cf26aba2fba83bf07e72902357d91f8b

Merge "Add missing JIT stuff in make files"

Don't require three-character extensions for dex locations.

Bug: 19437875
Change-Id: Ib62b4c691b04f27c5d499affd5a7fd4d9f0c64f9

Add missing JIT stuff in make files

Was causing core.oat to not be created.

Bug: 19524713
Change-Id: I5da005fd36f9682623fdda9931280997b59c2681

Merge "Update expectations for volantis."

Update expectations for volantis.

Change-Id: Icd4aecd9b7a274febfd99bad6432bb9d46f9e1d3

Merge "ARM: Fix LIR flags in encoding map."

Merge "JDWP: fix thread state on event suspension"

ARM: Fix LIR flags in encoding map.

Also correct memory reference type for PC relative load.

Change-Id: I7a5258f2ed718448dc3e6e7fda6569b3f0c2fe46

Merge "Fix method_verifier_test with the SS collector."

Merge "Change already inline error to warning"

Fix method_verifier_test with the SS collector.

Rename art_default_gc_type to ART_DEFAULT_GC_TYPE.

Bug: 19514492

Change-Id: I3c7920e7a84327c5908462e51e21199b89fb187d

Change already inline error to warning

There is a race condition in the JIT where the JIT and another
running thread can both verify the same method at the same time.
This causes the inline method to be added twice and the LOG(ERROR)
was causing occasional test failures.

Bug: 17950037
Bug: 18713034
Change-Id: I7f94384106ca5c621a5cf7d2ba36910d923d110c

Merge "Fix JIT for vmdebug test 99"

Merge "[MIPS] Fix mips32 trampoline problem."

Merge "Fix argument register allocation for mips32."

[MIPS] Fix mips32 trampoline problem.

Reserve argument slots (for $a0-$a3) on the stack in the
InvokeTrampoline() function, for entrypoints that contain direct
reference to native implementation. Called function may use this
space to store $a0-$a3 regs.

This fixes the Settings app crash on mips32, when navigating to:
Settings->Apps->(swipe) On SD card.

Change-Id: If53ce822fd3ef6ef9839c89b556f76fac0792190

Fix JIT for vmdebug test 99

Test was flaky due to JIT re-compiliation after deoptimization
resulting in some invalid PC offsets.

Bug: 17950037
Change-Id: I276c84c918579259ce47ef873892c3c5dcf0c977

Fix argument register allocation for mips32.

Change-Id: Ic21b0371d7cafb9b67217b92bb2edb735a43d1b0

Merge "ART: oatdump enhancements"

ART: oatdump enhancements

New features list includes:
- Class filter option to limit classes search space
- Method filter is applied only against the method
  name, instead of the entire signature. Can be
  combined with class filter for maximum efficiency.
- Bulk dump of class and method names list only.
  Can be combined with filters to limit results.
- Export embedded dex files from input oat files
  to filesystem (symlinks not supported as utils
  functions are utilized for os & fs operations).
- addr2instr option to locate the in-range method
  implementation and limit disassemble dumps. Input
  relative addr is added to oat executable offset to
  calculate the search offset. If method has been
  successfully located, code is dumped and program
  aborts further analysis of the input file. Methods
  located before the target address just print their
  signature, although skip all disassemble and other
  info. Calculated search offset is also printed as
  part of the initial header info.
- Little-endian dex instructions bytecode is printed
  in the same line before the instruction string.

Some minor re-orders have been also taken place for
more targeted results.

Change-Id: I3116ee3c99c258718f46faea8ea4295da6ae2bf7
Signed-off-by: Anestis Bechtsoudis <anestis@census-labs.com>

Merge "ART: Print maps directly to log"

JDWP: fix thread state on event suspension

Before suspending event thread, we ensure its state is kSuspended
so the debugger sees it as RUNNING instead of WAIT.

Bug: 19103406
Change-Id: I58b4141a958bf6a3c360f6994967fad0078ea373

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

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

ART: Move DexFile vector to Java array

To avoid having native vectors only referenced by Java objects,
which look like leaks to Valgrind, use a Java array to store
references to native DexFile objects.

Change-Id: If3c2b31b9d0914ed1965cfd5e3fdb94ea41b1477

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

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

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

Merge "Revert "Revert "Add JIT"""

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

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

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

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

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

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

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

Merge "ART: Disable libcore tests failing on bots"

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

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

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