am 486728d8: Merge "Address small comments in HGraphBuilder"

* commit '486728d82e5b36a5ff3d40d282d9c32f7305bc3a':
  Address small comments in HGraphBuilder

am 4880fd56: Merge "Revert "Revert "ART: Ignore try blocks with no throwing instructions"""

* commit '4880fd5695ea1726dde27bb448dae1338d0a0973':
  Revert "Revert "ART: Ignore try blocks with no throwing instructions""

Merge "Address small comments in HGraphBuilder"

Address small comments in HGraphBuilder

Change-Id: Ife0a19f6e07146c8e03922f1330527d092f5e236

Merge "Revert "Revert "ART: Ignore try blocks with no throwing instructions"""

am d9acfe21: Merge "Migrated dexlist from Dalvik (libdex) into Art (libart)"

* commit 'd9acfe21f6eb09a22db69958dfb05c1f48f779bd':
  Migrated dexlist from Dalvik (libdex) into Art (libart)

Merge "Migrated dexlist from Dalvik (libdex) into Art (libart)"

Revert "Revert "ART: Ignore try blocks with no throwing instructions""

The original CL broke libcore tests because monitor-exit instructions
did not have any side-effects and got removed by DCE once not labelled
throwing any more.

This reverts commit efe374d7c25c1d48945a9198d96469de99e0c1bd.

Change-Id: I624c0f91676d9baaada6f33be9d7091f68d57535

am 98bea9fd: Merge "Revert "ART: Ignore try blocks with no throwing instructions""

* commit '98bea9fdab2c5f964a95f9e76620b0b35c050a8a':
  Revert "ART: Ignore try blocks with no throwing instructions"

Merge "Revert "ART: Ignore try blocks with no throwing instructions""

Revert "ART: Ignore try blocks with no throwing instructions"

Turns out monitor-exit *can* throw... Need to investigate

This reverts commit 8f8ee680bec71a28d9d7b7538e8c7ca100a18184.

Change-Id: I8b42690918833c917b6a7fc3ceea932b7c1a6f15

am 761f281e: Merge "Revert "Use the object class as top in reference type propagation""

* commit '761f281e6f78b1c48103cbd45385a5c410e2be07':
  Revert "Use the object class as top in reference type propagation"

am c7d5ae36: Merge "ART: Release inputs in Long.reverse intrinsic in x86"

* commit 'c7d5ae363bc43ebba0f21c6ee4a2164dfbb209a9':
  ART: Release inputs in Long.reverse intrinsic in x86

Merge "Revert "Use the object class as top in reference type propagation""

Revert "Use the object class as top in reference type propagation"

failing on the build bot on some targets but not locally. needs more investigation.

This reverts commit 20e6071362b84a9782b633a893c29ebde458205e.

Change-Id: I6965483f569fb862f9bdb66d459b747ded54de71

Merge "ART: Release inputs in Long.reverse intrinsic in x86"

ART: Release inputs in Long.reverse intrinsic in x86

In the worst case we are using two temps each for input and output.
Then we do not have a temp left over for the swap operations. The
input is dead, however, after the first swap. So try to release it
(a no-op if it isn't actually a temp).

Bug: 22324327
Change-Id: I1fc50159afdad14160e34abeaf4670958171d6b2

am 7d5e4d05: Merge "ART: Ignore try blocks with no throwing instructions"

* commit '7d5e4d056d6b8f54b2a7b054136e53ecbf97d314':
  ART: Ignore try blocks with no throwing instructions

Merge "ART: Ignore try blocks with no throwing instructions"

ART: Ignore try blocks with no throwing instructions

In order to avoid complex removal of redundant exceptional edges in
the SSA builder, this patch modified the graph builder to consider
blocks without throwing instructions as not in a try block, even if
covered by a TryItem.

In some corner cases, this may generate more TryBoundaries than
necessary, but those can be removed once the SSA form is built.

Change-Id: I158c4542b2c1964a8dd532f82e921b9cb1997e1e

am c7432f3b: Merge "Use the object class as top in reference type propagation"

* commit 'c7432f3b8cd7dd6fd198fe5cf7238650921cb306':
  Use the object class as top in reference type propagation

am c21dc873: Merge "Fix oatdump after ArtMethod change."

* commit 'c21dc873f5a6ce31d73299ee71805031ddb9b65a':
  Fix oatdump after ArtMethod change.

Merge "Use the object class as top in reference type propagation"

Merge "Fix oatdump after ArtMethod change."

Fix oatdump after ArtMethod change.

ArtMethod is now word aligned, and not necessarily 8 byte aligned.

Change-Id: I8bd6b2f156d7f2ab191d6f2675b013713bbf258d

am 30502f1f: Merge "Revert "Revert "Remove interpreter entrypoint in ArtMethod."""

* commit '30502f1fc8357bdaf11b70e6fc8f5311c6237194':
  Revert "Revert "Remove interpreter entrypoint in ArtMethod.""

Merge "Revert "Revert "Remove interpreter entrypoint in ArtMethod."""

Revert "Revert "Remove interpreter entrypoint in ArtMethod.""

The start of the interned strings in the image was not aligned
properly, now that ArtMethods just need to be word aligned.

This reverts commit 7070ccd8b6439477eafeea7ed3736645d78e003f.

bug:22242193

Change-Id: I580c23310c33c239fe0e5d15c72f23a936f58ed1

Use the object class as top in reference type propagation

This properly types all instructions, making it safe to query the type
at any time.

Change-Id: I3ee2f0f79253cdf45b10ddab37ecb473345ca53a

am a4e43032: Merge "Accept 0xFFFFFFFF as dex debug info offset."

* commit 'a4e4303253a31aa2a03d25c48a6228d30286d579':
  Accept 0xFFFFFFFF as dex debug info offset.

Merge "Accept 0xFFFFFFFF as dex debug info offset."

am 3e18a895: Merge "Fuse long and FP compare & condition on x86/x86-64 in Optimizing."

* commit '3e18a8958d2caa4980c4cee4e537313a61adf3d8':
  Fuse long and FP compare & condition on x86/x86-64 in Optimizing.

Merge "Fuse long and FP compare & condition on x86/x86-64 in Optimizing."

am 569e81e5: Merge "Revert "Remove interpreter entrypoint in ArtMethod.""

* commit '569e81e500725f52116b7d0342ec80a6d1e0089b':
  Revert "Remove interpreter entrypoint in ArtMethod."

am d5bd4fdd: Merge "Remove interpreter entrypoint in ArtMethod."

* commit 'd5bd4fdd46dce3c33abf17376fbceadcc1aff547':
  Remove interpreter entrypoint in ArtMethod.

Merge "Revert "Remove interpreter entrypoint in ArtMethod.""

Revert "Remove interpreter entrypoint in ArtMethod."

Build failures on bots. Investigating.

This reverts commit fa2c054b28d4b540c1b3651401a7a091282a015f.

Change-Id: Id65b2009aa66cb291fb8c39758a58e0b0d22616c

Merge "Remove interpreter entrypoint in ArtMethod."

Remove interpreter entrypoint in ArtMethod.

Saves 4/8 bytes for each ArtMethod.

Change-Id: I110ecdddf8516b0759a31fa157609643e6d60b15

Migrated dexlist from Dalvik (libdex) into Art (libart)

Rationale:
The new dexlist (temporarily called dexlist2 until we are
satisfied with the migration) is a re-implementation of the
original dexlist utility that was based on Dalvik functions
in libdex into a new dexlist that is now based on Art functions
in libart instead.

Bug: 22322814
Change-Id: I0f034d66c1a1f3c5450423f57fcdc23e2f5a1e29

Accept 0xFFFFFFFF as dex debug info offset.

Bug: 22023362
Change-Id: I03e5c74f5ece4dbe1fab37d8b5f857582b04f57c

am a100332e: Merge "Build rule for dexdump regression test on host."

* commit 'a100332ee0f4b842c28ec2448f97e40a882cd837':
  Build rule for dexdump regression test on host.

Merge "Build rule for dexdump regression test on host."

am 53ed3c5f: Merge "ART: Fix opsize in LoadArgDirect"

* commit '53ed3c5f71c983f01cd10a805255c1301438ab8c':
  ART: Fix opsize in LoadArgDirect

Merge "ART: Fix opsize in LoadArgDirect"

ART: Fix opsize in LoadArgDirect

If the destination register is a reference, use kReference for the
op size.

Bug: 22244733
Change-Id: Idf52f2ee4c65b5dc41cb66257d95281dc6f32255

Fuse long and FP compare & condition on x86/x86-64 in Optimizing.

This is a preliminary implementation of fusing long/float/double
compares with conditions to avoid materializing the result from the
compare and condition.

The information from a HCompare is transferred to the HCondition if it
is legal.  There must be only a single use of the HCompare, the HCompare
and HCondition must be in the same block, the HCondition must not need
materialization.

Added GetOppositeCondition() to HCondition to return the flipped
condition.

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

am ee35ff80: Merge "Improve code generation for ARM64 VisitArrayGet/Set."

* commit 'ee35ff809616324cbada38cbc0610eb09da09b35':
  Improve code generation for ARM64 VisitArrayGet/Set.

Merge "Improve code generation for ARM64 VisitArrayGet/Set."

Improve code generation for ARM64 VisitArrayGet/Set.

We prefer the code sequence
   add temp, obj, #offset
   ldr out, [temp, index LSL #shift_amount]
to
   add temp, obj, index LSL #shift_amount
   ldr out, [temp, #offset]

Change-Id: I98f51a1b5a5ecd84c677d6dbd4c4bfc0f157f5e2

am 00dec6a9: Merge "Make dexdump2 compile for 64bit devices."

* commit '00dec6a9506364806227f21f6a053176a6723fe9':
  Make dexdump2 compile for 64bit devices.

am 399426d9: Merge "Added next condition check to branch instruction"

* commit '399426d95de2fd4f317527319a72f4abfa4cc0a0':
  Added next condition check to branch instruction

am 4e869c81: Merge "Make oatdump work with .art files and code generated by Optimizing."

* commit '4e869c818c44f9e65c9807ff5df4cd5de155d260':
  Make oatdump work with .art files and code generated by Optimizing.

Merge "Make dexdump2 compile for 64bit devices."

Make dexdump2 compile for 64bit devices.

Change-Id: I4c990272b20062632d236671415cfb3dbb26c867

Merge "Added next condition check to branch instruction"

Merge "Make oatdump work with .art files and code generated by Optimizing."

Build rule for dexdump regression test on host.

Bug: 17442393

Change-Id: I7795ba014e57080572769286cb3f8e3d4931e706

am bb5965c8: Merge "Disable sanitization of dex2oat on device."

* commit 'bb5965c8891c748a30120ff20e873bbfe2dbf5d5':
  Disable sanitization of dex2oat on device.

Merge "Disable sanitization of dex2oat on device."

Disable sanitization of dex2oat on device.

Bug: 22233158
Change-Id: I54990bb167a65275ed28e0cbbab56b936eb70ca4

am 288b3001: Merge "List dependence of dexdump gtest on dexdump binary."

* commit '288b30015d136a81e5899439c398ca476e332006':
  List dependence of dexdump gtest on dexdump binary.

Merge "List dependence of dexdump gtest on dexdump binary."

List dependence of dexdump gtest on dexdump binary.

Change-Id: I1293009a370eff808b5e31cc89110eb9fe3ce32f

am e7ea17b7: Merge "Migrated dexdump from Dalvik (libdex) into Art (libart)"

* commit 'e7ea17b726ab415a0dc98c0a3901497bbcf3a4c3':
  Migrated dexdump from Dalvik (libdex) into Art (libart)

Merge "Migrated dexdump from Dalvik (libdex) into Art (libart)"

Migrated dexdump from Dalvik (libdex) into Art (libart)

Rationale:
The new dexdump (temporarily called dexdump2 until we are
satisfied with the migration) is a re-implementation of the
original dexdump utility that was based on Dalvik functions
in libdex into a new dexdump that is now based on Art functions
in libart instead.

The output is identical to the original for *correct* DEX files.
Output in error messages and the usage() may differ, however,
since the new utility relies on Art parsing and verification.

NOTE 1:
ODEX files are no longer supported.

NOTE 2:
Where possible, I kept the file as close to the original
as possible, including some archaic C idioms on memory
allocation; those can be improved over time.

NOTE 3:
I used the standard Android.mk format for the new dexdump,
but this probably needs to be Art-i-fied.

NOTE 4:
Some minor issues that need resolution are marked with a TODO.

Bug: 17442393
Change-Id: I753743f64afcf4b84b8d33efbd1cfcb7908f0c3e

am d4e2738f: Merge "Revert "Revert "Fix LSRA bug with explicit register temporaries"""

* commit 'd4e2738fa7a062fee6beb88204c7420e64e5d9a6':
  Revert "Revert "Fix LSRA bug with explicit register temporaries""

am 7fb6ddb8: Merge "ART: Check switch and array data better in the verifier"

* commit '7fb6ddb874af342c5117559044f826ad1270e72b':
  ART: Check switch and array data better in the verifier

am a3c31dfd: Merge "ART: Unlink old file in OS::CreateEmptyFile"

* commit 'a3c31dfd721e54bd00fa7bfe218a69e5483a5a5d':
  ART: Unlink old file in OS::CreateEmptyFile

am d5c5ea39: Merge "Revert "ART: Unlink target oat file before compiling""

* commit 'd5c5ea39775276e38df8b5981a8423777254e098':
  Revert "ART: Unlink target oat file before compiling"

Merge "Revert "Revert "Fix LSRA bug with explicit register temporaries"""

Merge "ART: Check switch and array data better in the verifier"

ART: Check switch and array data better in the verifier

Incorrect padding (or just malformed code) can lead to pseudo
instructions for data payload not being correctly discoverable in
a linear scan. This is problematic for the optimizing compiler,
which uses such a scan to build its graph.

Use the opcode flag in the insn_flags_ to ensure that a pseudo
instruction referenced from a base instruction was actually
correctly discovered. This does not ensure the letter of the dex
specification for now, which calls for nops for padding. However,
it is unclear what the difference between unreachable code and
padding is.

Bug: 22237618
Bug: https://code.google.com/p/android/issues/detail?id=178522
Change-Id: I823b3f541f326e6e77b15f3e13a7608a39d4b1fb

Merge "ART: Unlink old file in OS::CreateEmptyFile"

Merge "Revert "ART: Unlink target oat file before compiling""

am f7aa6c05: Merge "Revert "Fix LSRA bug with explicit register temporaries""

* commit 'f7aa6c05a1c7d70182d43abaf3ff43b6d463eec0':
  Revert "Fix LSRA bug with explicit register temporaries"

Revert "Revert "Fix LSRA bug with explicit register temporaries""

This reverts commit a5fc140ff315dda9bc0a8e59963ed547676cd941.

Change-Id: Ic322484176e55d0c7cd7250d629b9e5046006a4f

Merge "Revert "Fix LSRA bug with explicit register temporaries""

Revert "Fix LSRA bug with explicit register temporaries"

register_allocator_test32 fails.

This reverts commit 283b8541546e7673d33d104241623d07c91cf500.

Change-Id: I2a46f3c68de3e8273e402102065c13797045c481

am 16d6d0ef: Merge "Fix LSRA bug with explicit register temporaries"

* commit '16d6d0effdcdba70e97518be591f409dc8460a03':
  Fix LSRA bug with explicit register temporaries

Merge "Fix LSRA bug with explicit register temporaries"

Fix LSRA bug with explicit register temporaries

A temporary with an explicit RegisterLocation, such as ESI on x86 didn't
have the register marked as allocated.  This caused it to not be
saved/restored in the prologue/epilogue, causing problems in the caller
routine, which expected it to be saved.  Found while implementing
https://android-review.googlesource.com/#/c/157522/.

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

am 51f38e3a: Merge "ART: Fix graph for switch leaving a try block"

* commit '51f38e3adf58ba4e35b5374fb8c4b87cb3112abd':
  ART: Fix graph for switch leaving a try block

am b2409d91: Merge "Supress OsTest#test_xattr on the bots."

* commit 'b2409d91a2bb8d190712c009940659cea080cd65':
  Supress OsTest#test_xattr on the bots.

Merge "ART: Fix graph for switch leaving a try block"

ART: Fix graph for switch leaving a try block

Blocks that GraphBuilder creates for switch-case logic are given
a dex_pc of the branch targets they serve, while in fact they should
be considered part of the switch instruction itself and get its pc.
This caused the try/catch algorithm to either miss try boundaries or
create bogus edges.

This patch fixed the dex_pc of the switch-case blocks and modifies
the try/catch logic to iterate over all blocks as opposed to just
branch targets since multiple blocks can now cover the same dex_pc.

Change-Id: I30fe4f8db0647b869979197a3bc847cf212a7315

Merge "Supress OsTest#test_xattr on the bots."

Supress OsTest#test_xattr on the bots.

bug: 22258911
Change-Id: Iba57eb8660e0f46e2b152ca05f49f64a233e0f44

am 4a305daf: Merge "Add implicit null pointer and stack overflow checks for Mips."

* commit '4a305daf77a9b80d6abb7817a836aa59d2db521d':
  Add implicit null pointer and stack overflow checks for Mips.

Merge "Add implicit null pointer and stack overflow checks for Mips."

Add implicit null pointer and stack overflow checks for Mips.

Bug: 21555893
Change-Id: I2a995be128a5603d08753c14956dd8c8240ac63c

am b447598f: Merge "Disable static builds for mac and asan."

* commit 'b447598f6900f05f0b1940a0731ee374c57c3100':
  Disable static builds for mac and asan.

Merge "Disable static builds for mac and asan."

Disable static builds for mac and asan.

- asan doesn't support static linkage
- we don't need a static build of dex2oat for mac

Change-Id: I7eddd19f1f47e8b2a67ecc0aaf40cb70941febdd

am aefdcd9a: Merge "Do not create a HBoundType when the instruction is non-null."

* commit 'aefdcd9a06bc436a0f0908230856d1054018cd93':
  Do not create a HBoundType when the instruction is non-null.

Merge "Do not create a HBoundType when the instruction is non-null."

Do not create a HBoundType when the instruction is non-null.

We don't need to refine the type after a null check, if the
instruction is known non null or null. As a side effect, this
avoids replacing HLoadClass instructions with HBoundType instructions.

bug:22116987

(cherry picked from commit 3abd437507f8ba30a238a52c273c9944dcb9d5a1)

Change-Id: I5e56de293554534195ade9770b7d1e4b078d685b

am 06b6da5c: Merge "Fix mac build: use -lrt only on linux."

* commit '06b6da5ccbf9246c697b8cf1be0ba58ce616521d':
  Fix mac build: use -lrt only on linux.

Merge "Fix mac build: use -lrt only on linux."