am 254aa0d3: Merge "ART: Fix SelectKind to work with nullptr"

* commit '254aa0d3f58b56b227077ef32b4606ebc25eaae4':
  ART: Fix SelectKind to work with nullptr

am 1293b6b2: Merge "ART: Fix issues with SplitBlock"

* commit '1293b6b2adba33e8b5eca5632dd90c9835b2ef42':
  ART: Fix issues with SplitBlock

am 893e8558: Merge "Fix to MirGraph::GetSSANameWithConst()"

* commit '893e855864b7535f7da92f3942d3f88aaa584dd8':
  Fix to MirGraph::GetSSANameWithConst()

Merge "ART: Fix SelectKind to work with nullptr"

Merge "ART: Fix issues with SplitBlock"

Merge "Fix to MirGraph::GetSSANameWithConst()"

am 663796fb: Merge "Fix and improve static evaluation of constant expressions."

* commit '663796fb14accb722e9b37c4fe8c3d2588680d7e':
  Fix and improve static evaluation of constant expressions.

Merge "Fix and improve static evaluation of constant expressions."

Fix and improve static evaluation of constant expressions.

- Fix the definition of art::HSub::Evaluate.
- Qualify Evaluate methods as OVERRIDE.
- Evaluate comparisons in a deterministic way: if a
  comparison is true, always return 1 (instead of letting
  the compiler return any non-null value).
- Better exercise static evaluation of constant expressions
  in compiler/optimizing/constant_propagation_test.cc.

Change-Id: I13d0862e5f4eba1275016fb8c3c17e9aff54408b

ART: Fix issues with SplitBlock

The SplitBlock is a public interface from MIRGraph to be able to create two
blocks from one. This interface can be called not just during the control
flow graph building phase, but later. At that later point, extended instructions
may have already been inserted.

For example, SpecialMethodInliner can insert kMirOpNullCheck and kMirOpNop,
and then another pass tries to split block that contains this. The DCHECK
will fail even though the split is semantically correct.

Signed-off-by: Mathew Zaleski <mathew.zaleski@intel.com>
Conflicts:

compiler/dex/mir_graph.cc

Change-Id: I8bb3551407907909cda444a3379da2eb9331b9ff

am 50940ea8: Merge "Implement method calls using relative BL on ARM64."

* commit '50940ea8fe668a80c15920d98cae228851ceb990':
  Implement method calls using relative BL on ARM64.

Merge "Implement method calls using relative BL on ARM64."

am 1000e69b: Merge "Add trivial register hints to the register allocator."

* commit '1000e69b7e11348f2e1d3ba67339616a647f53d7':
  Add trivial register hints to the register allocator.

Merge "Add trivial register hints to the register allocator."

Add trivial register hints to the register allocator.

- Add hints for phis, same as first input, and expected registers.
- Make the if instruction accept non-condition instructions.

Change-Id: I34fa68393f0d0c19c68128f017b7a05be556fbe5

am a9f29042: Merge "Add support for floats and doubles."

* commit 'a9f2904263581f606a5704f2bb74efcecf7e9f97':
  Add support for floats and doubles.

Merge "Add support for floats and doubles."

Add support for floats and doubles.

- Follows Quick conventions.
- Currently only works with baseline register allocator.

Change-Id: Ie4b8e298f4f5e1cd82364da83e4344d4fc3621a3

am 4a1b4679: Merge "Inserting a node must also update its inputs users."

* commit '4a1b4679cda2f0d2893b8e3f910c21231849291c':
  Inserting a node must also update its inputs users.

Merge "Inserting a node must also update its inputs users."

Inserting a node must also update its inputs users.

Change-Id: I55357564b81efcc0cf52fffdf23289696fe27dd1

am 0d6102a8: Merge "Propagate optimization flags to kMirOpCheck."

* commit '0d6102a80ec741d701788decfb701a657d7a98f7':
  Propagate optimization flags to kMirOpCheck.

Merge "Propagate optimization flags to kMirOpCheck."

am 45eaba56: Merge "Fix movw on x86/x86_64 to accept any 16bits immediate."

* commit '45eaba56a9d847fed21f215bd9e9cbc2f8a7b14d':
  Fix movw on x86/x86_64 to accept any 16bits immediate.

Merge "Fix movw on x86/x86_64 to accept any 16bits immediate."

Fix movw on x86/x86_64 to accept any 16bits immediate.

Change-Id: I282eece0cd497431f207cec61852b4585ed3655c

am 41abdb6e: Merge "Do not use kDiesAtEntry when inputs must be in specific reg."

* commit '41abdb6ec97978df7c6d79abce4efb664c994ce8':
  Do not use kDiesAtEntry when inputs must be in specific reg.

Merge "Do not use kDiesAtEntry when inputs must be in specific reg."

Do not use kDiesAtEntry when inputs must be in specific reg.

The way the register allocator blocks registers currently
does not handle these cases. Since it only applies to x86 for now,
just ensure such requests cannot happen.

Change-Id: Idfa25532b9b4996a192d05800f56c6e44edd3a8a

am e9da5d17: Merge "Add a prepare for register allocation pass."

* commit 'e9da5d17fb6e8fde383c943c184905d63ed0c644':
  Add a prepare for register allocation pass.

Merge "Add a prepare for register allocation pass."

am 9800e55b: Merge "Make which dex2oat version to use configurable."

* commit '9800e55b059d4a0fdc0ceebd5652a53f7a8d837a':
  Make which dex2oat version to use configurable.

Merge "Make which dex2oat version to use configurable."

am 92e7b912: Merge "Fix a merge error in cl 108991."

* commit '92e7b912929528f23abef6b33d9fa6e6bf327ccc':
  Fix a merge error in cl 108991.

am 902c989b: Merge "Add way to warn about missing JNI_ABORT"

* commit '902c989bc46da80c3c80e49c7f868b63ba73d2de':
  Add way to warn about missing JNI_ABORT

am da20867f: Merge "Merge host-run-test-jar and push-and-run-test-jar."

* commit 'da20867f1967cab18722b507758e90913410b8e8':
  Merge host-run-test-jar and push-and-run-test-jar.

Merge "Fix a merge error in cl 108991."

Merge "Add way to warn about missing JNI_ABORT"

Fix a merge error in cl 108991.

Bug: 12687968
Change-Id: I37603c154b252321b59a5ea02c6a92f5a9300667

Add way to warn about missing JNI_ABORT

Bug: 16858794
Change-Id: I6794a14ee323ef95569cc7646619e6869771c7c6

Merge "Merge host-run-test-jar and push-and-run-test-jar."

Merge host-run-test-jar and push-and-run-test-jar.

- Too many code duplication between the two files.
- Also fix --zygote and --gdb invocations.

Change-Id: I1f99320e63a7a0f19ad1ad2b12693901b8c6bb43

am 31fa08d8: Merge "Handlerize methods across some GC points."

* commit '31fa08d8db5abae646b15b0c339366232ca25a91':
  Handlerize methods across some GC points.

Merge "Handlerize methods across some GC points."

Add a prepare for register allocation pass.

- Currently the pass just changes the uses of checks to the
  actual values.
- Also optimize array access, now that inputs can be constants.
- And fix another bug in the register allocator reveiled by
  this change.

Change-Id: I43be0dbde9330ee5c8f9d678de11361292d8bd98

Handlerize methods across some GC points.

Bug: 12687968
Change-Id: I0d5b0a78488ba76db4d25991d8db95b24bb624e9

am 17b1c174: Merge "Add support for inputs dying at entry of instructions."

* commit '17b1c174dddb1d83018740c2084ab42daa812fff':
  Add support for inputs dying at entry of instructions.

Merge "Add support for inputs dying at entry of instructions."

Add support for inputs dying at entry of instructions.

- Start using it in places where it makes sense.
- Also improve suspend check on arm to use subs directly.

Change-Id: I09ac0589f5ccb9b850ee757c76dcbcf35ee8cd01

Make which dex2oat version to use configurable.

Change-Id: I86503a51538c0b48ad96e8624b81461c2ba3279a

Fix to MirGraph::GetSSANameWithConst()

Don't call ConstantValueWide() for a wide constant if high_word is set,
as it will DCHECK if you try to get the value for the high word.

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

Propagate optimization flags to kMirOpCheck.

Bug: 17751474
Change-Id: I9561b641d956e21dac686c7cd414fa0c6a176147

am 2d4e89e9: Merge "Respect USE_CLANG_PLATFORM_BUILD for art builds"

* commit '2d4e89e97812aeca16ff058d7286f29b7549c43a':
  Respect USE_CLANG_PLATFORM_BUILD for art builds

Merge "Respect USE_CLANG_PLATFORM_BUILD for art builds"

Respect USE_CLANG_PLATFORM_BUILD for art builds

Without this patch, ART_TARGET_CLANG is set to false unconditionally.
It should be set to whatever USE_CLANG_PLATFORM_BUILD requests, if the
latter is set.

Change-Id: I29c4397c308a1549c2097aad2c997fe732be27ac
Signed-off-by: Bernhard Rosenkraenzer <Bernhard.Rosenkranzer@linaro.org>

am 15a7fc2e: Merge "Make host-run-test-ar take --no-prebuild."

* commit '15a7fc2e25cede6519a5a315cede27ec84566405':
  Make host-run-test-ar take --no-prebuild.

Merge "Make host-run-test-ar take --no-prebuild."

Make host-run-test-ar take --no-prebuild.

Forgot to update this file in last commit.

Change-Id: I7f7c6ecbabecaaabdeffa3fb5a50a9e9e78d03c0

am e6b73ea6: Merge "Merge push-and-run-{prebuilt}-test-jar files."

* commit 'e6b73ea6719a2c91433b560fd3441d79945edc18':
  Merge push-and-run-{prebuilt}-test-jar files.

Merge "Merge push-and-run-{prebuilt}-test-jar files."

am d4155721: Merge "Add some missing TEMP_FAILURE_RETRYs around unlink(2)"

* commit 'd415572117cbfc12f78c7d1c5844a7b62e68a2ff':
  Add some missing TEMP_FAILURE_RETRYs around unlink(2)

Merge "Add some missing TEMP_FAILURE_RETRYs around unlink(2)"

am fc951768: Merge "ART: Prepare for ELF64."

* commit 'fc95176845e1db99115202d7f64c03d521ca0aab':
  ART: Prepare for ELF64.

Add some missing TEMP_FAILURE_RETRYs around unlink(2)

Change-Id: Ibf446d37092933dce165d7a95c0ff2e52b94ce46

Merge "ART: Prepare for ELF64."

am c226f141: Merge "Add test for getDeclaredClasses of with missing class"

* commit 'c226f1417cab34097280e4501ad67b07a23d214a':
  Add test for getDeclaredClasses of with missing class

Merge "Add test for getDeclaredClasses of with missing class"

Add test for getDeclaredClasses of with missing class

Bug: 17782530
Change-Id: I399621344c3835226c9df9678217fca62cbd5c92

am b9001abf: Add extra logging for lock contention

* commit 'b9001abff3a45f1ae90536da7dd1ec28a6ae0174':
  Add extra logging for lock contention

am dfaf4c39: Merge "Quick compiler: Fix ambiguous LoadValue()"

* commit 'dfaf4c39809035bca7af85d2c51a8bd2f381e58e':
  Quick compiler: Fix ambiguous LoadValue()

Merge "Quick compiler: Fix ambiguous LoadValue()"

Add extra logging for lock contention

When we get contention which is longer than kLongWaitMs (= 100ms) we
log to logcat.

Example output:
W/art     ( 9960): Long monitor contention event with owner method=void com.android.server.am.BroadcastQueue.processNextBroadcast(boolean) from BroadcastQueue.java:542 waiters=1 for 243ms

Bug: 17787391
Change-Id: Id1f92d9c41ec909d221871ecc2ff6181dcec2ed4

Quick compiler: Fix ambiguous LoadValue()

Internal b/17790197 & hat tip to Stephen Kyle

The following custom-edited dex program demonstrated
incorrect code generation caused by type confusion.
In the example, the constant held in v0 is used in both
float and int contexts, and the register class gets
confused at the if-eq.

.method private static getInt()I
    .registers 4
    const/16 v0, 100
    const/4 v1, 1
    const/4 v2, 7
    :loop
    if-eq v2, v0, :done
    add-int v2, v2, v1
    goto :loop
    :done
    add-float v3, v0, v1
    return v2
.end method

The bug was introduced in c/96499, "Quick compiler: reference cleanup"
That CL created a convenience variant of LoadValue which selected the
target register type based on the type of the RegLocation.  It should
not have done so.  The type of a RegLocation is the compiler's best
guess of the Dalvik type - and Dalvik allows constants to be used
in multiple type contexts.  All code generation utilities must specify
desired register class based on the capabilities of the instructions
to be emitted.  In the failing case, OpCmpImmBranch (and
GenCompareZeroAndBranch) will be using core registers, so the
LoadValue must specify  either kCoreReg or kRefReg.

The CL deletes the dangerous LoadValue() variant.

Change-Id: Ie4ec6e51b19676dbbb9628c72c8b3473a419e7ec

ART: Prepare for ELF64.

Only expose necessary interface in ElfFile, and move all details into template class ElfFileImpl.

Change-Id: I9df2bbc55f32ba0ba91f4f3d5d0009e84a2ddf74

Implement method calls using relative BL on ARM64.

Change-Id: I9e5d0b6c100b6cddd6bbb7ab07cff77ab104ea31

am 63462448: Merge "AArch64: oat patches should be 32-bit ints."

* commit '63462448ca4e377074a10a4720aa22f71154dbe9':
  AArch64: oat patches should be 32-bit ints.

Merge "AArch64: oat patches should be 32-bit ints."

am b5325e24: Merge "Revert "Fix OOM throwing if it happens in finalizer reference""

* commit 'b5325e24ca58299b2b011e57e784b2584f99d687':
  Revert "Fix OOM throwing if it happens in finalizer reference"

am 04fe6524: Merge "Revert "Rename registers in arm64.""

* commit '04fe6524edd927278626084db73c27219a213319':
  Revert "Rename registers in arm64."

am 20810995: Merge "Remove all uses of MIR_INLINED."

* commit '20810995a11f29b8220658ab829791db0810eb9d':
  Remove all uses of MIR_INLINED.

am 3eae0839: Merge "Rename registers in arm64."

* commit '3eae0839c28469a00030b967b998e9c8a694c1a5':
  Rename registers in arm64.

am 72f96181: Merge "Fix OOM throwing if it happens in finalizer reference"

* commit '72f961813dcb915542b9ae957aed040082d3e40a':
  Fix OOM throwing if it happens in finalizer reference

am 674b5b95: Merge "Fix test-art-(host|target)-run-test(32|64) targets."

* commit '674b5b951a92e295c8942f56c571801f6d6b8d41':
  Fix test-art-(host|target)-run-test(32|64) targets.

am 53ce7d4f: Merge "Fix x86_64 assembler LoadRef to use movl."

* commit '53ce7d4f97da490d74b0695c83a7f165aacf5ab6':
  Fix x86_64 assembler LoadRef to use movl.

am 013bf973: Merge "Colorize warnings when building with clang."

* commit '013bf973fe1faf98f17ccfc4fbdae67e4bfdb696':
  Colorize warnings when building with clang.

Merge push-and-run-{prebuilt}-test-jar files.

Can't find a reason why those two files forked...

Change-Id: I5ef3dd76aa3b2d843c8b4b57365edec3bf2c3f3b

Merge "Revert "Fix OOM throwing if it happens in finalizer reference""

Revert "Fix OOM throwing if it happens in finalizer reference"

This reverts commit 6d7729d6ae8b2ac3800e92092d61390ce4e3b6d7.

Change-Id: I3e863b7372657ee85b0e48029c3a3e2b4ba75a7c

AArch64: oat patches should be 32-bit ints.

This makes the arm64 backend consistent with the behaviour of the code
in oat_writer.cc and in the patchoat tool.
It also reduces the size of boot.oat by 1.6% (aosp_arm64-eng build).

Change-Id: Ia0b96737159c08955cd7b776ee396ff578cd58f6

Merge "Revert "Rename registers in arm64.""

Revert "Rename registers in arm64."

This reverts commit 11daa0adbb30f341ccbdec64a2d43d8eeb111288.

Change-Id: I2ce0879dce8fbf5512649d0d9c3a2c189c23dd5f

Merge "Remove all uses of MIR_INLINED."

Merge "Rename registers in arm64."

Merge "Fix OOM throwing if it happens in finalizer reference"

Merge "Fix test-art-(host|target)-run-test(32|64) targets."

Fix test-art-(host|target)-run-test(32|64) targets.

Fix extra hyphen and missing target.

Change-Id: Ib7f87172693507ff693bcebe2629e5dfd64b1b3a

Merge "Fix x86_64 assembler LoadRef to use movl."

Merge "Colorize warnings when building with clang."

am 54e53789: Merge "More native bridge tests"

* commit '54e53789549816a67484d8b5f493158ac72d05f8':
  More native bridge tests

Merge "More native bridge tests"