am a1d1fc2b: Merge "Restore valgrind-test-art-host"

* commit 'a1d1fc2bed0d11a380bc9bdad5f7f3c69d75bd6d':
  Restore valgrind-test-art-host

Merge "Restore valgrind-test-art-host"

Restore valgrind-test-art-host

Change-Id: I2586e16664dabfc543bc9d0dab471ec8d3810f34

am 88dd36d2: Merge "Makefile cleanups."

* commit '88dd36d2d1f0df6d8263ada619497742a4dd21a8':
  Makefile cleanups.

Merge "Makefile cleanups."

Makefile cleanups.

Also add the DEX2OAT_FLAGS to the generation of core.oat/core.art.

Change-Id: I5d113fad460b2e14fc652816069adba29ca744b8

am 9e878d50: Merge "Tidy ELF builder."

* commit '9e878d50567f624094f3c4940ac3aedbc5eff3b9':
  Tidy ELF builder.

Merge "Tidy ELF builder."

am 51205042: Merge "Quick: Reduce memory usage and improve compile time."

* commit '51205042343fa4852e01919de01fdcf261af510b':
  Quick: Reduce memory usage and improve compile time.

am 2da74f2d: Merge "Quick: Separate null check elimination and type inference."

* commit '2da74f2de211b7ecb39464e641fa9794dc1b67f4':
  Quick: Separate null check elimination and type inference.

Merge "Quick: Reduce memory usage and improve compile time."

Merge "Quick: Separate null check elimination and type inference."

Tidy ELF builder.

Don't do "if (ptr)". Use const. Use DISALLOW_COPY_AND_ASSIGN. Avoid public
member variables.
Move ValueObject to base and use in ELF builder.
Tidy VectorOutputStream to not use non-const reference arguments.

Change-Id: I2c727c3fc61769c3726de7cfb68b2d6eb4477e53

Quick: Reduce memory usage and improve compile time.

Move the def-block-matrix from Arena to ScopedArena. Remove
BasicBlockDataFlow::ending_check_v and use a temporary bit
matrix instead. Remove unused BasicBlockDataFlow::phi_v.
Avoid some BitVector::Copy() at the end of null and clinit
check elimination passes when the contents of the source
BitVector is no longer needed.

Change-Id: I8111b2f8a51e63075aa124b528d61b79b6933274

Quick: Separate null check elimination and type inference.

Change-Id: I4566ae9354c91ca935481cb4f5b729bba05c1592

am 2072c465: Merge "Stop converting from Location to ManagedRegister."

* commit '2072c465cfff077da257bdf14f1f1b2690c946c8':
  Stop converting from Location to ManagedRegister.

Merge "Stop converting from Location to ManagedRegister."

Stop converting from Location to ManagedRegister.

Now the source of truth is the Location object that knows
which register (core, pair, fpu) it needs to refer to.

Change-Id: I62401343d7479ecfb24b5ed161ec7829cda5a0b1

am b6404fe0: Disable kEnableJavaStackTraceHandler

* commit 'b6404fe0bb782cbbccda88903901fe0096a741d5':
  Disable kEnableJavaStackTraceHandler

am a3c4d722: Merge "Disable kEnableJavaStackTraceHandler"

* commit 'a3c4d72210de174552f47b2d117b1946f274af1e':
  Disable kEnableJavaStackTraceHandler

Merge "Disable kEnableJavaStackTraceHandler"

Disable kEnableJavaStackTraceHandler

Bug: 17890306

(cherry picked from commit b6404fe0bb782cbbccda88903901fe0096a741d5)

Change-Id: Ie13d6af1d384c6ebf87e8f055120609e2bff3338

am a9b09744: Merge "Don\'t copy fill array data to quick literal pool."

* commit 'a9b097443f26a6df1a322a8fef50fdd1f26a410b':
  Don't copy fill array data to quick literal pool.

Merge "Don't copy fill array data to quick literal pool."

Don't copy fill array data to quick literal pool.

Currently quick copies the fill array data from the dex file to the literal
pool. It then has to go through hoops to pass this PC relative address down
to out-of-line code. Instead, pass the offset of the table to the out-of-line
code and use the CodeItem data associated with the ArtMethod. This reduces
the size of oat code while greatly simplifying it.
Unify the FillArrayData implementation in quick, portable and the interpreters.

Change-Id: I9c6971cf46285fbf197856627368c0185fdc98ca

am 47798949: Merge "Dump mutator lock for thread suspend timeout"

* commit '477989496bfb8d1d9687d7ad8ee5b9ede80dc410':
  Dump mutator lock for thread suspend timeout

Merge "Dump mutator lock for thread suspend timeout"

Dump mutator lock for thread suspend timeout

Should help us see if thread suspend timeouts are due to someone
holding on the mutator lock for too long or if it is a missed
futex wake.

(cherry picked from commit f924d2381a5ea53967ba1e279766d601b9be05ea)

Bug: 17837911
Change-Id: I3417156a78f574076d5da17c86836c5a7c76084a

Disable kEnableJavaStackTraceHandler

Bug: 17890306
Change-Id: I76d1d1b709cfcb0cf8fe02f5d28fb746909d682f

am f1f05d30: Merge "Really fix --gdb when running tests on target."

* commit 'f1f05d303988a5c071c87b760056be8358276c94':
  Really fix --gdb when running tests on target.

Merge "Really fix --gdb when running tests on target."

am 175b2732: Merge "Header file clean up."

* commit '175b2732f7e55a07094c30aa08d946189b4af97e':
  Header file clean up.

Merge "Header file clean up."

Header file clean up.

Remove runtime.h from object.h.
Move TypeStaticIf to its own header file to avoid bringing utils.h into
allocator.h.
Move Array::DataOffset into -inl.h as it now has a utils.h dependency.
Fix include issues arising from this.

Change-Id: I4605b1aa4ff5f8dc15706a0132e15df03c7c8ba0

am edc34c88: Merge "ART: fix NullCheckElimination to preserve MIR_IGNORE_NULL_CHECK"

* commit 'edc34c88b8f8abd04f9c4668787403608cf0b2d4':
  ART: fix NullCheckElimination to preserve MIR_IGNORE_NULL_CHECK

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 NullCheckElimination to preserve MIR_IGNORE_NULL_CHECK"

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."

Really fix --gdb when running tests on target.

Change-Id: Ic96edb066d345839a4af371e7c10327713a045c7

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."

ART: fix NullCheckElimination to preserve MIR_IGNORE_NULL_CHECK

If the MIRGraph::EliminateNullChecksAndInferTypes() function managed
to prove that some regs are non-null then it sets the flag
MIR_IGNORE_NULL_CHECK and resets this flag for all the other regs.
If some previous optimizations have already set MIR_IGNORE_NULL_CHECK
then it can be reset by EliminateNullChecksAndInferTypes. This way
NullCheckElimination discards some optimization efforts.
Optimization passes should not reset MIR_IGNORE_NULL_CHECK unless
they 100% sure NullCheck is needed.

This patch makes the NCE_TypeInference pass be conservative in
resetting MIR_IGNORE_NULL_CHECK.

Change-Id: I4ea74020968b5c5bd8e3af48211ffd4c6afd7f80
Signed-off-by: Yevgeny Rouban <yevgeny.y.rouban@intel.com>

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)"