Merge "Rename isDexOptNeededInternal and add kSelfPatchOatNeeded"

Merge "Speedup div/rem by constants on x86 and x86_64"

Speedup div/rem by constants on x86 and x86_64

This is done using the algorithms in Hacker's Delight chapter 10.

Change-Id: I7bacefe10067569769ed31a1f7834f796fb41119

Merge "Opt compiler: ARM64: Use TBZ and TBNZ in VisitIf."

Merge "Exercise art::arm::Thumb2Assembler::StoreToOffset for word pairs."

Merge "Quick: Clean up temp use counting."

Merge "Fix JDWP race at runtime shutdown"

Merge "[optimizing] Implement more x86/x86_64 intrinsics"

Quick: Clean up temp use counting.

For the boot image on arm64 and x86-64 we're using true
PC-relative addressing, so pc_rel_temp_ is nullptr and
CanUsePcRelDexCacheArrayLoad() returns true, but we're not
actually using the ArtMethod* so fix the AnalyzeMIR() to
take it into account.

Also don't count intrinsic invokes towards ArtMethod* uses.
To avoid repeated method inliner inquiries about whether a
method is intrinsic or special (requiring lock acquisition),
cache that information in MirMethodLoweringInfo. As part of
that cleanup, take quickened invokes into account for
suspend check elimination.

Change-Id: I5b4ec124221c0db1314c8e72675976c110ebe7ca

Merge "Remove the old CFI infrastructure."

Opt compiler: ARM64: Use TBZ and TBNZ in VisitIf.

TBZ and TBNZ have a short range. Now that VIXL supports
veneers, we can use them safely without the danger of
running out of range.

Change-Id: Iaf77a441ccf86282c1793a2213a69a2091ca829a

Merge "Exercise art::arm::Thumb2Assembler::StoreToOffset for words."

Merge "Ignore not yet loaded classes during hprof"

Merge "ART: Remove LLVM cruft"

ART: Remove LLVM cruft

Change-Id: I133ebed6101bf12a0642ed71e13f332c0c4f14e7

Merge "ART: Fix 64-bit ELF file support"

ART: Fix 64-bit ELF file support

The API wasn't cross-compile-safe, 32-bit patchoat would fail for
negative delta applied to a 64-bit ELF file.

Add 64-bit ELF file output to the compilers, behind a flag, currently
off by default (preserving current behavior).

Bug: 20095017
Change-Id: I2cde7b4c7cc83413c76692d7b745868d644a604c

Merge "Promote pointer to dex cache arrays on arm."

Merge "PC-relative loads from dex cache arrays for arm."

Remove the old CFI infrastructure.

Change-Id: I12a17a8a1c39ffccaa499c328ebac36e4d74dc4e

Merge "Quick: Use PC-relative dex cache array loads for SGET/SPUT."

Merge "Fix CC root visiting bug"

Promote pointer to dex cache arrays on arm.

Do the use-count analysis on temps (ArtMethod* and the new
PC-relative temp) in Mir2Lir, rather than MIRGraph. MIRGraph
isn't really supposed to know how the ArtMethod* is used by
the backend.

Change-Id: Iaf56a46ae203eca86281b02b54f39a80fe5cc2dd

PC-relative loads from dex cache arrays for arm.

Change-Id: Ic25df4b51a901ff1d2ca356b5eec71d4acc5d9b7

Quick: Use PC-relative dex cache array loads for SGET/SPUT.

Change-Id: I890284b73f69120ada5cf9b9ef4a717af3273cd2

Fix CC root visiting bug

Also some cleanup.

Change-Id: Ia3de8f2d409770be3619ec116e8b06ecd82338fe

Merge "ART: Print C1vis header only if visualizer enabled"

Merge "ART: Remove unused variables."

ART: Print C1vis header only if visualizer enabled

Setting kStringFilter currently suppresses graph dumps of non-matching
methods but their headers are still printed. This fixes the issue.

Change-Id: Ib33fb20fcca2bf409534a824e7f76f1feb85724d

ART: Remove unused variables.

Curious that neither Clang nor GCC flagged those.

Change-Id: Ie1866d8171651c768e378d829a669cc57cab1f09

Merge "Remove fuzzer build-time dependency on core image."

Remove fuzzer build-time dependency on core image.

Test if the image exists at runtime.

Forcing this dependency slows down development. For example, if after a
change dex2oat does not compile anymore you need to first revert to a
stable state and rebuild the core image to be able to move foreward.
Since this is a test tool, it is ok to assume that the core image has
already been built if host execution was selected.

Change-Id: Id61a960ba62bb8d1d21acb07114f72208fbc9720

Merge "JDWP: clear exception when allocation fails"

Exercise art::arm::Thumb2Assembler::StoreToOffset for word pairs.

- Augment compiler/utils/arm/assembler_thumb2_test.cc.
- Ensure art::arm::Thumb2Assembler::StoreToOffset properly
  handles IP as (implicit) second source register.
- Remove the parity constraint on the first source register
  for art::arm::Thumb2Assembler::ldrd and
  art::arm::Thumb2Assembler::strd (as they are not required
  by the Thumb-2 encoding).
- Introduce additional versions of
  art::arm::Thumb2Assembler::ldrd and
  art::arm::Thumb2Assembler::strd accepting a second source
  register, which is not necessarily the one following the
  first source register, as it is allowed by the Thumb-2
  encoding.

Change-Id: I7dba168437a96a5cbb117058e9c547fb1ff5c295

Merge "Fix address formatting in Mips64 disassembler."

Merge "Build 32-bit version of the disassembler as well."

Merge "Proper fix for issue 20051601 - arm actually means thumb2."

Ignore not yet loaded classes during hprof

Skips classes that are no yet loaded because we cannot access its
fields nor its superclass. This CL fixes a crash observed during the
run-test 130-hprof.

Bug: 20040824
Change-Id: Ia618b6c1579d35dfd726726ca6d2ff4258d8a841

[optimizing] Implement more x86/x86_64 intrinsics

Implement CAS and bit reverse and byte reverse intrinsics that were
missing from x86 and x86_64 implementations.

Add assembler tests and compareAndSwapLong test.

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

Merge "Revert "ART: Fix noreturn for Mac""

Revert "ART: Fix noreturn for Mac"

missing-noreturn was turned off for Mac for all the
other similar warnings. Restore the cleaner header
file.

This reverts commit 79dda4251d7e3a7888e326bc7a3f069d6190d194.

Change-Id: Ia25a47533ff7848ad96e0d76e7f0ea768a48f946

Merge "ART: Turn off missing-noreturn for Mac"

ART: Turn off missing-noreturn for Mac

Too much stuff unimplemented and causing warnings.

Change-Id: I43bae875f1eca3cc24e03b0e7d6fdfd2adae0cbd

Merge "ART: Add support for "silvermont" variant"

Merge "ART: Fix noreturn for Mac"

ART: Fix noreturn for Mac

Change-Id: I59b60340a90672a4de4d3bdf092fa504394d5892

Merge "Add more logging for debugging possible deadlock"

Add more logging for debugging possible deadlock

Change-Id: I4b2db54504d3ed8c2c33a110191488cab1fcb2d9

Merge "Prevent verified method from being deleted by compiler driver."

Merge "Fix JNI compiler for mips64"

Prevent verified method from being deleted by compiler driver.

The results of it will be needed by future optimizing compiler changes.

Change-Id: Ifbdf47e7564906b74193f80a4ba0459eed87c0f7

Fix JNI compiler for mips64

Fix assembly instructions in JNI stub to store the right size. Fix
callee-saves in JNI calling convention to reflect return registers.
Fix pointer size in mips64 assembler.

Change-Id: I4eccd1f56f62fa9119918192ec424e9073398bc7

Merge "ART: Fix gtests for missing-noreturn"

ART: Fix gtests for missing-noreturn

gtest.h isn't sufficiently annotated.

Change-Id: I3d5fac8bcdd645227bfd7886c3aa781b225fcc9c

Merge "ART: Make unknown variant "mips64r6" go away"

ART: Make unknown variant "mips64r6" go away

It's the standard variant, accept it just like "default."

Change-Id: I9b600345e3e1ebfb508ffe22ddde17757be2172e

Merge "ART: Fix no-return in sigchainlib"

ART: Fix no-return in sigchainlib

Ignore warning in sigchain_dummy.

Change-Id: I2c9ed93c0125d06b6a3d4011a794b062cf1c4093

Merge "ART: Enable more Clang warnings"

ART: Enable more Clang warnings

Change-Id: Ie6aba02f4223b1de02530e1515c63505f37e184c

Merge "ART: Enable Clang's -Wdeprecated"

ART: Enable Clang's -Wdeprecated

Replace throw() with noexcept.

Add default copy constructors and copy assignment constructors for
cases with destructors, as the implicit definition is deprecated.

Change-Id: Ice306a3f510b072b00bec4d4360f7c8055135c9d

Merge "ART: Add Clang's -Wused-but-marked-unused"

Merge "Refactor and improve GC root handling"

Fix address formatting in Mips64 disassembler.

Use FormatInstructionPointer like all the other disassemblers.
This ensures that the 'absolute_addresses' option is honoured.

Change-Id: I5580319cc4fad40e00f3fbbde25b142f7c689390

Refactor and improve GC root handling

Changed GcRoot to use compressed references. Changed root visiting to
use virtual functions instead of function pointers. Changed root visting
interface to be an array of roots instead of a single root at a time.
Added buffered root marking helper to avoid dispatch overhead.

Root marking seems a bit faster on EvaluateAndApplyChanges due to batch
marking. Pause times unaffected.

Mips64 is untested but might work, maybe.

Before:
MarkConcurrentRoots: Sum: 67.678ms 99% C.I. 2us-664.999us Avg: 161.138us Max: 671us

After:
MarkConcurrentRoots: Sum: 54.806ms 99% C.I. 2us-499.986us Avg: 136.333us Max: 602us

Bug: 19264997

Change-Id: I0a71ebb5928f205b9b3f7945b25db6489d5657ca

Build 32-bit version of the disassembler as well.

Change-Id: I22ecc2611c3b05b1031b42abdb5bf8c245220e03

Merge "Pass linker patches around as const."

Merge "Delete missed ClampGrowthLimit CHECK"

Proper fix for issue 20051601 - arm actually means thumb2.

Change-Id: I38e81868cbebdccec6ecbba40494612f7c1d8092

ART: Add Clang's -Wused-but-marked-unused

Add detection of wrong unused annotations. Fix our codebase.

Change-Id: I85cc20f2eac71c1ec6c5c7cd6efb08454a629634

Delete missed ClampGrowthLimit CHECK

Bug: 20043461
Change-Id: If4a0bb91c64571945756151be162f3626534781c

Merge "Add missing RecordPcInfo's for intrinsics_x86."

Add missing RecordPcInfo's for intrinsics_x86.

Change-Id: I5ad856e57f63e2bd350f62e7023911c276427edd

Merge "ART: Turn of native stack dump on x86 on-device"

ART: Turn of native stack dump on x86 on-device

Temporary workaround for bug.

Bug: 20040863
Change-Id: Ifdde68c66b94ebc2f13c7983f38700c8bcf41559

ART: Add support for "silvermont" variant

The variant is already supported in the build system and wired up
for the Nexus Player (fugu). The values are copied from the build
system and correspond to documentation in, e.g.,
http://www.intel.com/content/dam/www/public/us/en/documents/manuals/64-ia-32-architectures-optimization-manual.pdf
chapter 15, section 1.1.

Change-Id: I9facaa9d5b8bae3d98cf48f2969d02d11ebfe0e4

Merge "Fix the read barrier builds."

Fix the read barrier builds.

A check failure (!UseJit() in ArtMethod::Invoke()) in forced interpret
only mode in the eng build with JIT enabled by default on target.

A build failure (inl header file issues) in the USE_BAKER_READ_BARRIER
build.

Change-Id: Ib13632b10864ad8b4691b66971c0ab5d2ae1e675

Rename isDexOptNeededInternal and add kSelfPatchOatNeeded

This change renames isDexOptNeededInternal to getDexOptNeeded and adds
another possible result: kSelfPatchOatNeeded. kSelfPatchOatNeeded is
returned when there is no odex file to relocate from but there is an
oat file that can be relocated in place.

Change-Id: Ib9a6373f98474f1242367b5285086251a9d580e5

Merge "Use va_list argument to abort transaction"

Merge "Delegate SIG_DFL on sigaction to libc."

Merge "Store OatDexFile instead of OatFile in DexFile."

Delegate SIG_DFL on sigaction to libc.

  In the case when user registers SIG_DFL hanlder
  sigchainlib's handler tends to go into infinite loop
  because the function handling signals resets signal
  using by calling to signchain's own implementation.

  This cl fixes the problem by passing sigcations with
  SIG_DFL to the next sigaction (usually libc's).

Bug: 19594886
Change-Id: I9eecf9afd1c7e6d1fe3cd1d4fc506383ecbebe04

Merge "Fix a bug when creating a HDeoptimization instruction."

Merge "ART: Avoid obvious segfault in arm & x86 fault handler"

Merge "Explicitly specify the C libraries that have mallinfo(3)."

Use va_list argument to abort transaction

Creates AbortTransactionV taking a va_list argument and renames
AbortTransaction to AbortTransactionF which calls AbortTransactionV.

This fixes the compiler_driver_test under valgrind.

Change-Id: Ia1c57330091c055ae9e46585a944ce0b78864920

Fix a bug when creating a HDeoptimization instruction.

We need to copy the environment, instead of just pointing
to an existing one. Otherwise, if the instruction that initially
holds the environemnt gets removed from the graph, any update
to an instruction in that environment will not be reflected in it.

bug:20058506

Change-Id: I2a62476d0851ecbc3707c0da395d8502ee437422

ART: Avoid obvious segfault in arm & x86 fault handler

These handlers need to read an instruction to determine where to
look for info. Don't try to read from pc=0.

Bug: 20040863
Change-Id: I38b56dc6dd806df22e608ee8d46c4091a738e4bc

Merge "Revert "ART: Workaround for b/20019689""

Revert "ART: Workaround for b/20019689"

This reverts commit 328dd4457632f59e67d08d9d086670b56141a4b1.

Bug: 20019689
Change-Id: Ia1087e45d1a1817b6ff4e45f8d74b84d34f078ee

Merge "Use specific exception class to abort transaction"

Use specific exception class to abort transaction

We used to throw a java.lang.InternalError when aborting a
transaction (when preinitializing image classes at compilation time).

We now use dedicated class dalvik.system.TransactionAbortError that
is only thrown by the compiler to abort a transaction. This class has
constructors taking a java.lang.Throwable "cause" so we can wrap
exceptions causing the transaction to abort (for instance class
java.lang.ClassNotFoundException) and give more information about the
cause of the transaction abort.

Bug: 20019689
Change-Id: I019a72a1c754d8bba6a7ad6bb0f02e4fd6668622

Explicitly specify the C libraries that have mallinfo(3).

Change-Id: Ib6d22134cb246e30452a7ed9acabbde42b3ef091

Merge "ART: Move trace options to a struct"

ART: Move trace options to a struct

Bundles the different options and moves them to a struct in the
implementation file. Will allow for easier change for streaming.

Change-Id: I486e0942614e714964eb233e2d6b4e3e12aa704f

Merge "Revert "ART: Valgrind hotfix for VIXL 1.9""

Merge "[optimizing] Implement x86/x86_64 math intrinsics"

Merge "Fix for issue 20051601 - ARM instruction alignment."