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

Temporary workaround for bug.

Bug: 20040863
Change-Id: Ifdde68c66b94ebc2f13c7983f38700c8bcf41559

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

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

Merge "Remove version-script option"

Remove version-script option

  Version script was used to limit number of symbols
  exposed by main executable because of --export-dynamic
  option. This was overthinking on my part which can
  lead to unwanted side-effects like not working asan.

Change-Id: Iaab4d38f86ddf21357d26d21d35c9ed6e7ff989e

Merge "Split RelativePatcher::ReserveSpace() into two."

Fix for issue 20051601 - ARM instruction alignment.

The instructions do not have to be 4 byte aligned.

Change-Id: I58078a15190b835601094611ed01a599e2d99563

Split RelativePatcher::ReserveSpace() into two.

Instead of passing nullptr and MethodReference(nullptr, 0u)
to ReserveSpace() at the end, call a newly created function
ReserveSpaceEnd().

Change-Id: I38815fe9464b4e1a919878b6e8577614f1058d00

Merge "Add tests for Arm64RelativePatcher."

Merge "Fix race with Heap::ClampGrowthLimit and GC"

Fix race with Heap::ClampGrowthLimit and GC

Aded logic for handling the temp bitmap if the GC is running and
the live bitmap is clamped to the mark bitmap. This fixes the SIGABRT
from ClampGrowthLimit if the GC clamped the bitmaps at this point.

Also added locking of the heap_bitmap_lock_ so that added a lock so
that the temp bitmap doesn't change from underneath us.

Bug: 20043461

Change-Id: Ib427e40bcdf149de0408b4b53e6524f51463f0af

Add tests for Arm64RelativePatcher.

Change-Id: I9d2c21d323137ac143eabb8fdf6ca075bae45c51

Merge "Update dexfuzz after optimizing default selection."

Revert "ART: Valgrind hotfix for VIXL 1.9"

Fixed in external/vixl and upstream VIXL.

Change-Id: I7d1901da06a7c1517c88a2583bc668a3e23ef852

Merge "Add tests for Thumb2RelativePatcher."

Add tests for Thumb2RelativePatcher.

Also make the thumb2/arm64 thunk allocation precise instead
of eagerly allocating thunk space. This allows the calls to
use the maximum positive offset.

Change-Id: Ifa95b0bb00bd73eeab0c2905d21e2f3078f4b0a8

Merge "Write .debug_line section using the new DWARF library."

Merge "Refactor RelativePatcher out of OatWriter."

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

Update dexfuzz after optimizing default selection.

Now that the optimizing compiler is the default backend, the quick
compiler must be explicitly selected as an argument.

Change-Id: I5e6b8e6a79432118e0a91c9aea85bdd9cd7305dc

Refactor RelativePatcher out of OatWriter.

Move the relative patcher classes to compiler/linker/ and
compiler/linker/<arch>/ . Refactor them to avoid OatWriter
dependency so that they can be unit tested. Add tests for
x86 and x86-64.

Change-Id: I1b42baa9fc431378e4cce1399bec590c5b5a409f

Merge "Move the "huge method" test into its own test case."

Move the "huge method" test into its own test case.

Change-Id: Icd7c95d2a7a0cacf15f079eed38c5d1a8f138b3f

PC-relative loads from dex cache arrays for x86-64.

Change-Id: I6cfe22c7e69512b3c0f95b073aaa572db74ec189

Merge "Switch to shared libcutils on host to fix odr-violation"

Switch to shared libcutils on host to fix odr-violation

Change-Id: Ia8eb950806a09e12fec2956d51eff99852920bee
SUMMARY: AddressSanitizer: odr-violation: global atrace_is_ready at system/core/libcutils/trace-host.c:23:25

Merge "Revert "Revert "Deoptimization-based bce."""

Merge "CanThrow() of HNewArray should return true."

Merge "Clean up some RecordPcInfo in x86 slow paths."

CanThrow() of HNewArray should return true.

Change-Id: I9950f1c391dfeb26cf59cee769705d01d8e283d5

Merge "ART: Fix unstarted runtime"

ART: Fix unstarted runtime

Correctly delete local refs for dex/direct-byte-buffer.

Change-Id: Ia74dfafb959d69aa281af7316f393997e053a3b3

Merge "ART: Workaround for b/20019689"

ART: Workaround for b/20019689

Don't throw InternalError as wrapped for now, swallowing the
original exception.

Bug: 20019689
Change-Id: I77a56262304b09b326232bee0c7916bb73401d74

Merge "Opt compiler: Fix move from constant."

Revert "Revert "Deoptimization-based bce.""

This reverts commit 0ba627337274ccfb8c9cb9bf23fffb1e1b9d1430.

Change-Id: I1ca10d15bbb49897a0cf541ab160431ec180a006

Merge "ART: Set default prune counter to 10"

Merge "Adds host execution to dexfuzz."

Merge "Fix test 421-large-frame."

Fix test 421-large-frame.

- Don't force the optimizing compiler to compile a huge
  method by reducing the number of local variables and
  instructions in Main.$opt$LargeFrame.
- Prevent the optimizing compiler from "optimizing too much"
  (through constant folding and inlining), so as to preserve
  a large stack frame in Main.$opt$LargeFrame.
- Keep the original huge method (without the $opt$ marker)
  as Main.HugeFrame.

Bug: 20020727
Change-Id: I49bf8f03146020bd94bd392684e605d7bb6daf1e

Merge "Fix formatting"

Fix formatting

Change-Id: Id64aad5de1f911ac7e747788d586bac397f34f18

Merge "ART: Regression test for condition materialization bug"

ART: Regression test for condition materialization bug

This adds a regression test previously omitted in a hotfix.

Bug: 19999189
Change-Id: I02acb935960f22028bf877cd270b04e17515068f

Merge "ART: Materialize constants on environment use"

ART: Materialize constants on environment use

The decision whether to materialize constants assumed that they cannot
be used by an environment. This changed with introduction of the
boolean simplifier and manifested itself when environment uses became
included in liveness analysis. This patch fixes the issue.

This is a quick fix to unblock the build, test will follow.

Bug: 19999189
Change-Id: Iedb1e3a32073a75b9c0d041a88b8ae640e064d13

[optimizing] Implement x86/x86_64 math intrinsics

Implement floor/ceil/round/RoundFloat on x86 and x86_64.
Implement RoundDouble on x86_64.

Add support for roundss and roundsd on both architectures.  Support them
in the disassembler as well.

Add the instruction set features for x86, as the 'round' instruction is
only supported if SSE4.1 is supported.

Fix the tests to handle the addition of passing the instruction set
features to x86 and x86_64.

Add assembler tests for roundsd and roundss to x86_64 assembler tests.

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

Write .debug_line section using the new DWARF library.

Also simplify dex to java mapping and handle mapping
in prologues and epilogues.

Change-Id: I410f06024580f2a8788f2c93fe9bca132805029a

Merge "ART: Same skip-compilation heuristic in both backends"

Merge "ART: Valgrind hotfix for VIXL 1.9"

ART: Valgrind hotfix for VIXL 1.9

Make sure recommended_checkpoint_ is initialized in the VIXL macro
assembler by calling EmitLiteralPool with an empty pool.

Change-Id: I08589b8fb092a33a8f4aad824e91b5c16ff761b6

Merge "ARM64: Update to VIXL 1.9."

ART: Set default prune counter to 10

By default, prune cache at boot after ten failed boots.

Bug: 19983101
Change-Id: Ib88f807e0082d71292c14c7af38e02cca5a5602c

Clean up some RecordPcInfo in x86 slow paths.

Calling codegen->RecordPcInfo() is likely to miss the third argument which
is the slow path. And can cause deopt bugs later.

The change calls the slow path's version of RecordPcInfo() in slow paths
consistently.

Change-Id: I41605f1b06ee2c6d3d7ffd5aa0c1366940b5a029

Merge "Only GC the first time we call PreZygoteFork"

Only GC the first time we call PreZygoteFork

PreZygoteFork is called for every app launch, it is overkill to GC
each time since it takes 20-30ms and blocks app launch for that long.

Change-Id: I647c8ccca767ceca67a006c1d80a739c7860f0d0

Merge "Visit image roots for hprof"

Visit image roots for hprof

Bug: 19995360

(cherry picked from commit 0cab5e68f14ee403380664146db6dc7ddfc32064)

Change-Id: I17868bff2a701dc25291d41b9732c6b86f92be08

Merge "Fix lint error."

Fix lint error.

Change-Id: I39dda83acd755b2c95453b3b0f7774bc9d18546e

Merge "Fix locations at environment uses."

Fix locations at environment uses.

We were too agressive in not recording environment uses
when the instruction was not of type object. We have to
record the use to the use list of an interval, but it should
not affect the live ranges of that interval.

Change-Id: Id16fb7cc06f14083766d408a345837793583b6ea

Merge "Cortex-A53 Erratum 843419 workaround in OatWriter."

Cortex-A53 Erratum 843419 workaround in OatWriter.

Bug: 19983916
Change-Id: I572248d035200bb04f74eb3ae065e862572628a0

Merge "Fixed layout for dex caches in boot image."

Merge "Fix wrong assumptions about ParallelMove."

Fix wrong assumptions about ParallelMove.

Registers involved in single and double operations can
drag stack locations as well, so it is possible to update
a single stack location with a slot from a double stack location.

bug:19999189

Change-Id: Ibeec7d6f1b3126c4ae226fca56e84dccf798d367

Merge "Revert "Revert "Add small library for writing of DWARF data."""

Merge "Host dex2oat and patchoat require libcutils."

Merge "[optimizing] trivial x86 explicit null check fix"

ART: Same skip-compilation heuristic in both backends

When deciding if a method should be compiled, Optimizing counts the
DEX instructions while Quick approximates this figure as the size of
the CodeItem. This patch modifies Optimizing to use the same logic
so that the two backends reject the same set of methods.

Change-Id: Icb12bf2977f948198df08d59f243532ec8c61783

[optimizing] trivial x86 explicit null check fix

Change a cmp reg,0 to test reg,reg.  I don't know that this code
is even invoked.

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

Merge "Fix oat_file_assistant_test's ReserveImageSpace()."

Revert "Revert "Add small library for writing of DWARF data.""

Use objdump from prebuilts.

This reverts commit 1ded5beffa4bae6203e1fa03903c6ffa2766bf4c.

Change-Id: I91f61a91827406930039ccf8691359b1c2636640

ARM64: Update to VIXL 1.9.

Update VIXL's interface to VIXL 1.9.

Change-Id: Iebae947539cbad65488b7195aaf01de284b71cbb
Signed-off-by: Serban Constantinescu <serban.constantinescu@arm.com>

Merge "Use variable encoding for StackMap."

Use variable encoding for StackMap.

dex_pc, native_pc_offset, dex_register_map_offset, inline_info_offset,
and register_mask can now be encoded in 1, 2, 3, or 4 bytes.

Change-Id: I15f93e8226ce374204d44c5a80a9fd89bda2687c

Merge "Revert "Add small library for writing of DWARF data.""

Revert "Add small library for writing of DWARF data."

objdump differences on the bots

This reverts commit 75c3d61011a06c1253e773c42e53ee6f137dbe53.

Change-Id: Ie5947a27b9fe5de9d5f0838a024b5cf646ec701e

Merge "ART: Inline MemoryRegion operations"

ART: Inline MemoryRegion operations

Functions of the MemoryRegion class method are sufficiently simple
yet do not get inlined. Since these methods are called frequently
during stack map generation, inlining them can save a non-negligible
amount of compile time. This patch inserts attributes to force this.

Change-Id: I8ab4f480103fd536f61f52b805864d91bf58428e

Merge "Use unique location catalogs to encode Dex register maps."

Use unique location catalogs to encode Dex register maps.

- For each CodeInfo object (i.e. for each method), compute a
  variable index size location catalog  of unique Dex
  register locations.  In Dex register maps, instead of
  storing the actual location of a (live) Dex register,
  store the index of the location catalog entry containing
  that Dex register location.
- Adjust art::StackMapStream,
  art::CheckReferenceMapVisitor::CheckOptimizedMethod,
  art::StackVisitor::GetVRegFromOptimizedCode, and
  art::StackVisitor::SetVRegFromOptimizedCode.
- Add LoadBits and StoreBits methods to art::MemoryRegion
  to load and store a block of adjacent bits in a memory
  region.
- Update compiler/optimizing/stack_map_test.cc.
- Remove the default value for parameter EmptyFn of
  art::HashMap.  This default value did not seem to make
  sense, as it would create an "empty function" for type Key
  by default, whereas art::HashMap expects an "empty
  function" for type std::pair<Key, Value>.

Change-Id: Id9e49d7756c253ce41c36630cd832208d06c2e28

Merge "Ensure MemoryRegion::Load & MemoryRegion::Store are word-aligned."

Ensure MemoryRegion::Load & MemoryRegion::Store are word-aligned.

Change-Id: Ib19fb93abec4789a111dfd48fcac6065e2d3ec82

Merge "Add small library for writing of DWARF data."

Merge "Tests still failing on buildbot's devices."