Merge "Revert "Run type propagation after inliner only when needed.""

Revert "Run type propagation after inliner only when needed."

This reverts commit 4e5dd521063beae1706410419f19c7e224db50fe.

Change-Id: I0de261d14dd3f71abe05f9bc71744820cf23b937

Merge "Run type propagation after inliner only when needed."

Merge "Fix dependencies when not building with Jack."

Merge "Disable store elimination on singleton object."

Disable store elimination on singleton object.

Stores into singleton objects that are finalizable need to be kept.
Disable those store eliminations before we have the finalizable info available.

Change-Id: I281e0c442588ec291b1002fdc9b59a144c15dc8e

Merge "Add ArenaUniquePtr"

Add ArenaUniquePtr

Motivation is to use it for adding arenas in the verifier.
Also added arena tags to prevent double free errors.

Bug: 10921004
Change-Id: I545f3374eceb9a6a35e738cb899d1493098fb760

Merge "enum operator<<() script fails with do { } while cond;"

Merge "Move ArenaBitVector into the runtime"

Move ArenaBitVector into the runtime

Motivation is using arenas in the verifier.

Bug: 10921004
Change-Id: I3c7ed369194b2309a47b12a621e897e0f2f65fcf

Merge "ART: Speed up thread stack tracing for ANRs"

ART: Speed up thread stack tracing for ANRs

Share a BacktraceMap for all threads being dumped, which speeds up
ANR dumping. Results from Nexus 9 (average of five):

Before: 0.587s
After:  0.206s

Change-Id: Ia70e0dbd39049318c02de561e7b95258d4849467

Merge "Revert "Revert "load store elimination."""

Fix dependencies when not building with Jack.

Change-Id: I6528a835974c4915872974265f2d135f754169b5

Run type propagation after inliner only when needed.

Currently we run a type propagation pass unconditionally after the
inliner. This change looks at the returned value (if any) and runs a
minimal type propagation only if its type has changed.

Change-Id: I0dd72bd481219081e8a978d2632426afc980d73a

Merge "Disable two tests that require IPv6."

Disable two tests that require IPv6.

bug:25178637
Change-Id: I4615cb4d2f2d9cbb050d02cb77b36215c24fd43b

Merge "ARM64: Instruction simplification for array accesses."

Merge "Disable the ARM & ARM64 UnsafeCASObject intrinsic with heap poisoning."

Disable the ARM & ARM64 UnsafeCASObject intrinsic with heap poisoning.

The current heap poisoning instrumentation of this intrinsic
does not always work properly when heap poisoning in
enabled, hence this quick fix to let the build & test
infrastructure turn green again.

Bug: 12687968
Change-Id: I546a392a61e429cd13209261f806d0aed8d1cd86

Merge "Fix heap poisoning in UnsafeCASObject x86/x86-64 intrinsic."

Merge "MIPS64: Add tests for rotation, and shift instructions"

Revert "Revert "load store elimination.""

This reverts commit 8030c4100d2586fac39ed4007c61ee91d4ea4f25.

Change-Id: I79558d85484be5f5d04e4a44bea7201fece440f0

Merge "Release ahat version 0.2."

Merge "MIPS: Initial version of optimizing compiler for MIPS32"

MIPS: Initial version of optimizing compiler for MIPS32

Change-Id: I370388e8d5de52c7001552b513877ef5833aa621

Merge "Do not run the verifier when deopting optimized frames."

Do not run the verifier when deopting optimized frames.

The stack maps contain all the required information.

Change-Id: I709e72b3de0d76a4203a3a004f093071e7865642

Merge "Fix art-run-tests dependencies"

Fix heap poisoning in UnsafeCASObject x86/x86-64 intrinsic.

Properly handle the case when the same object is passed to
sun.misc.Unsafe.compareAndSwapObject for the `obj` and
`newValue` arguments (named `base` and `value` in the
intrinsic implementation) and re-enable this intrinsic.

Also convert some reinterpret_casts to down_casts.

Bug: 12687968
Change-Id: I82167cfa77840ae2cdb45b9f19f5f530858fe7e8

Merge "Use binary search for FindDeclaredInstance/StaticField"

Merge "Fix destruction order in Dex2Oat."

Merge "JDWP: fix possible JdwpEvent memory leak"

Merge "Block signals before invoking special or user handlers"

Use binary search for FindDeclaredInstance/StaticField

Before:
real  1m18.157s
user  1m8.167s
sys 0m8.071s

After:
real  1m12.943s
user  1m3.223s
sys 0m7.881s

Perf results:
FindDeclaredStaticField: 1.78% -> 0.11%
__GI___strncmp_ssse3: 2.45% -> 0.87%

Bug: 10921004

Change-Id: Ice7d3ce2635d6cd2de5574055375d9e20712d241

Fix art-run-tests dependencies

Their classpath dependency was missing

(cherry picked from commit 6c8d5962573e2df393594bbcee80d07f16edafe3)

Change-Id: Id4f489c140585c62b6c105533b97837612c6199c

JDWP: fix possible JdwpEvent memory leak

Fixes possible memory leak when registering new JdwpEvent but the
Count modifier is invalid. This is done by using a std::unique_ptr
with a custom delete function. It also allows to easily introduce
more modifier checks in the future.

Bug: 14076832
Change-Id: I339ffb1297165ff85ab924e5e7618d336df5612b

Merge "Mark 2 libcore tests as failing."

Mark 2 libcore tests as failing.

bug:25136848
Change-Id: If1cde2f3a0140a5eb9c138a462ecc9a30e73b8eb

Merge "Remove the magic 32 constant and ensure alignment."

Remove the magic 32 constant and ensure alignment.

Change-Id: I383315bf3cd5f0b8634e2982da55d5d864009a91

enum operator<<() script fails with do { } while cond;

The script assumes that a line containing a '}' and a ';' will only
and always signal the end of a scope.
Unfortunately, this also matches '} while (myCond);',
thus incorrectly thinking it has reached the end of a scope.

This can result in enums being emitted without scope resolution
(or with insufficient scope resolution), cause compile errors.

Change-Id: I6e1b3453b941bff93ddd0b2a9fd3950182668728
Signed-off-by: randy.jeong <randy.jeong@samsung.com>

Merge "Inliner: make sure the returned value is in the outer graph."

Merge "Use DlMallocSpace for the JIT code cache."

Use DlMallocSpace for the JIT code cache.

- Also tidy up some code in the JIT compiler.
- And mprotect code space to be writable only when allocating.

Change-Id: I46ea5c029aec489f2af63452de31db3736aebc20

Merge "ART: Disable Clang for arm"

ART: Disable Clang for arm

Disable Clang for arm32 until a fix for the issue is found.

Bug: 25130937
Change-Id: Ia33f387b89ac8d1e8eac9a2782c45f038b9e8a1a

Merge "ART: Fix left-shift of negative numbers"

ART: Fix left-shift of negative numbers

Shifting negative numbers is undefined.

Change-Id: I0c32a3fcf372eae74507a5f2383edbfaf5c6830c

Merge "Fix typos."

Merge "Change dex_location_ from const char* to std::string."

Merge "Switch ahat test to use core-optimizing-pic.art."

Inliner: make sure the returned value is in the outer graph.

The returned value may be a constant or a parameter value. If so, it
will be in the inlined entry_block and (before this CL) we would not
update its block or graph. This CL fixes this and makes sure that the
returned value belongs to the outer graph.

Change-Id: Ie296f0d5a320c33f39eb187df6d328371ccf6500

MIPS64: Add tests for rotation, and shift instructions

Change-Id: I5611cb5c638c6be193739cbe859cb4ff4074a5fa

Fix typos.

Change-Id: Ie2198e3bb4b978c638de12db34547b4311d85eb5

Merge "Disable the x86 & x86-64 UnsafeCASObject intrinsic with heap poisoning."

Merge "Disable test 961-default-iface-resolution-generated with gcstress"

Merge "MIPS64: Disassembler support for rotate instructions."

Disable test 961-default-iface-resolution-generated with gcstress

The test takes too long with gcstress and can cause timeouts.

Change-Id: I99d16e882650ea73d07f4292fcfba0869f5b7ed1

Release ahat version 0.2.

* Update ahat to use perflib studio-1.4
* Add some help tips.

Change-Id: I5aeafb27ff9e55f2ad1a6316184b8d3838e1df7c

Disable the x86 & x86-64 UnsafeCASObject intrinsic with heap poisoning.

The current heap poisoning instrumentation of this intrinsic
does not always work properly when heap poisoning in
enabled, hence this quick fix to let the build & test
infrastructure turn green again.

Bug: 12687968
Change-Id: I03702a057fb6f07134e926e2c1c2780f47e3a50a

Fix destruction order in Dex2Oat.

In case of a failure (say, --image= pointing to non-existent
directory), the image_writer_ may not be destroyed in
CreateImageFile() and it will be destroyed in Dex2Oat's
destructor instead. However, ImageWriter owns a MemMap, so
it must be destroyed before MemMap::Shutdown() called from
Runtime's destructor. In release builds without leak checks,
we intentionally leak the runtime, so we don't hit the
destruction order issue. However, debug build was crashing
on the DCHECK(maps_ != nullptr) in MemMap destructor.

This change cleans up the destruction order and deliberately
leaks even more owned data in release builds without leak
checks.

Change-Id: Ib7f305bb676589d0379ecaafe6a53eeae208beaa

Merge "New attempt at fixing mac build."

New attempt at fixing mac build.

Change-Id: I00e3df55e65eb5edb4e8dd244bb7f8918dd942d4

Merge "Fix mac build."

Fix mac build.

Change-Id: I7229a628a619164eea22735bcaed507428ab054a

Merge "Remove ArtCode."

Remove ArtCode.

- Instead use OatQuickMethodHeader.
- Various cleanups now that we don't have all those
  ArtMethod -> ArtCode -> OatQuickMethodHeader indirections.

As a consequence of this cleanup, exception handling got a bit
faster.

ParserCombinators benchmark (exception intensive) on x64: (lower is better)
Before:
ParserCombinators(RunTime): 1062500.0 us.
After:
ParserCombinators(RunTime): 833000.0 us.

Change-Id: Idac917b6f1b0dc254ad68fb3781cd61bccadb0f3

Merge "Fix induction_var_range_test."

Fix induction_var_range_test.

Change-Id: I43101c5e35f4c516ea4ba3137631508f12703412

Merge "Revert "Revert "optimizing: propagate type information of arguments"""

MIPS64: Disassembler support for rotate instructions.

Also, tighten the tests for recognizing the various shift commands. The
tests, previously, would be unable to distinguish between "shift right
logical" and "rotate right" commands. In particular:

- SRLV vs. ROTRV
- DSRLV vs. DROTRV,
- DSRL vs. DROTR, and
- DSRL32 vs. DROTR32

Change-Id: I7a6df8ab0d76fd3d34b1207da9915369ad84fa97

Merge "MIPS64: Add intrinsic support for bit rotation"

MIPS64: Add intrinsic support for bit rotation

- int java.lang.Integer.rotateLeft(int i, int distance)
- int java.lang.Long.rotateLeft(long i, int distance)

Change-Id: I048ebf310265c6b20a80108673a7931cbeee0513

Merge "Added ability to generate induction range code."

Merge "Keep dex files live in class table"

Added ability to generate induction range code.

Rationale: used by dynamic BCE (done in another CL).

Change-Id: Ia6ce75da57b5298fba74622822ae0bae69c74188

Keep dex files live in class table

The DexFile.loadClass API allows callers to load classes using a
dex file without having that dex file owned by the specified class
loader. We now add the dex file to the class table to make sure it
stays live until the class loader is unreachable.

Fixes interpreter gcstress test 087 with 64 bit.

Bug: 22720414
Change-Id: Ia4341149f45b6293312f8b275c7a68cea179f718

Merge "Generalize codegen and simplification of deopt."

Generalize codegen and simplification of deopt.

Rationale: the de-opt instruction is very similar to an if,
           so the existing assumption that it always has a
           conditional "under the hood" is very unsafe, since
           optimizations may have replaced conditionals with
           actual values; this CL generalizes handling of deopt.

Change-Id: I1c6cb71fdad2af869fa4714b38417dceed676459

Merge "ART: Replace an "ALWAYS_INLINE" with "inline" to fix build."

ART: Replace an "ALWAYS_INLINE" with "inline" to fix build.

Change-Id: I61e6806737589266b36ab7a33561da7e1f8c02ca

Merge "Clean up OatFile."

Merge "MIPS64: Add intrinsic support for bit rotation"

Clean up OatFile.

In Setup(), avoid reading beyond the end and use the %zu
format specifier instead of %zd for size_t output.

Make the .bss section pointers non-const.

Change-Id: Ic8f066effe8037b552d8e911c6a5d17370d79ff4

MIPS64: Add intrinsic support for bit rotation

- int java.lang.Integer.rotateRight(int i, int distance)
- int java.lang.Long.rotateRight(long i, int distance)

Assembler tests for new MIPS instructions will be provided in a
separate patch.

Change-Id: I6dd4786e2d5f674bf56ff3d5afd321bb1bef589e

Merge "MIPS64: Add intrinsics support for trailing zeros"

MIPS64: Add intrinsics support for trailing zeros

- int java.lang.Integer.numberOfTrailingZeros(int i)
- int java.lang.Long.numberOfTrailingZeros(long i)

Change-Id: I0fc1c2629738de9047313b6511b490639eef68f3

ARM64: Instruction simplification for array accesses.

HArrayGet and HArraySet with variable indexes generate two
instructions on arm64, like

    add temp, obj, #data_offset
    ldr out, [temp, index LSL #shift_amount]

When we have multiple accesses to the same array, the initial `add`
instruction is redundant.

This patch introduces the first instruction simplification in the
arm64-specific instruction simplification pass. It splits HArrayGet
and HArraySet using the new arm64-specific IR HIntermediateAddress.
After that we run GVN again to squash the multiple occurrences of
HIntermediateAddress.

Change-Id: I2e3d12fbb07fed07b2cb2f3f47f99f5a032f8312

Block signals before invoking special or user handlers

1. Block signals marked in sa_mask before invoking user handler
   if not in old style.
2. Block all signals before invoking special signal handler.

Change-Id: Ie1283e97ed28ee3421df584352cd62cb107264bc
Signed-off-by: jgu21 <jinghui.gu@intel.com>

Merge "MIPS64: Implement intrinsics from sun.misc.Unsafe:"

MIPS64: Implement intrinsics from sun.misc.Unsafe:

- int getInt(Object, long)
- int getIntVolatile(Object, long)
- long getLong(Object, long)
- long getLongVolatile(Object, long)
- Object getObject(Object, long)
- Object getObjectVolatile(Object, long)
- void putInt(Object, long, int)
- void putOrderedInt(Object, long, int)
- void putIntVolatile(Object, long, int)
- void putObject(Object, long, Object)
- void putOrderedObject(Object, long, Object)
- void putObjectVolatile(Object, long, Object)
- void putLong(Object, long, long)
- void putOrderedLong(Object, long, long)
- void putLongVolatile(Object, long, long)

The compareAndSwap*() functions will be delivered in a separate patch.

Change-Id: I94238254b2f9df017d58fa9a4bb38632f6479472

Merge "Allow changing the optimization level of a build with env variable."

Merge "MIPS64: Additional assember tests:"

Merge "MIPS64: Implement intrinsics from java.lang.String:"

Change dex_location_ from const char* to std::string.

Because we tend to use the dex_location_ as an std::string fairly
often.

Change-Id: Ida9624c9cb12c8ad30577146f1e6e97e25edc155

Merge "Revert "Revert "Implement rosalloc fast path in assembly for 32 bit arm."""

Merge "Fix mips64 cfi test"

Fix mips64 cfi test

Change-Id: I93224f29b23473fb9f2f6631a62949fdf2e8f888