ART: Change temporaries to positive names

Changes compiler temporaries to have positive names. The numbering now
puts them above the code VRs (locals + ins, in that order). The patch also
introduces APIs to query the number of temporaries, locals and ins.

The compiler temp infrastructure suffered from several issues
which are also addressed by this patch:
-There is no longer a queue of compiler temps. This would be polluted
with Method* when post opts were called multiple times.
-Sanity checks have been added to allow requesting of temps from BE
and to prevent temps after frame is committed.
-None of the structures holding temps can overflow because they are
allocated to allow holding maximum temps. Thus temps can be requested
by BE with no problem.
-Since the queue of compiler temps is no longer maintained, it is no
longer possible to refer to a temp that has invalid ssa (because it
was requested before ssa was run).
-The BE can now request temps after all ME allocations and it is guaranteed
to actually receive them.
-ME temps are now treated like normal VRs in all cases with no special
handling. Only the BE temps are handled specially because there are no
references to them from MIRs.
-Deprecated and removed several fields in CompilationUnit that saved
register information and updated callsites to call the new interface from
MIRGraph.

Change-Id: Ia8b1fec9384a1a83017800a59e5b0498dfb2698c
Signed-off-by: Razvan A Lupusoru <razvan.a.lupusoru@intel.com>
Signed-off-by: Udayan Banerji <udayan.banerji@intel.com>

Merge "Reduce interpret-only compile time."

Merge "ART: Added dirty flag to optimizations"

Merge "ART: Add capability for a pass to have options"

ART: Add capability for a pass to have options

This patch adds capability to have pass options. These are needed when a pass
has multiple flags that can be tweaked. The user is now allowed to pass those
options via command line.

Since passes are treated as singletons and they are immutable, the overridden
options provided by user are set on the compilation unit. Doing this way also
allows a selectivity system to tweak the option per compilation instead of
doing it globally (due to the single pass existing).

The following command line flags have been added:
--print-pass-options - This prints all passes that have options along with
their defaults.
--pass-options= - This is used to pass the overridden options in format of
PassName:PassOption:PassOptionSetting

Change-Id: Ib5156f5d2ff51a0c64c4ea0fa050bd2170663417
Signed-off-by: Razvan A Lupusoru <razvan.a.lupusoru@intel.com>
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>

ART: Added dirty flag to optimizations

- Added a dirty flag to the PassMEDataHolder and use it to determine
    if we want to perform the post-optimization passes or not.

Change-Id: I59a85f5198f9224d5fe8893397bfaf68a978d8f1
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>

Merge "Use loops to build combinations of run-tests."

Use loops to build combinations of run-tests.

Add a JNI force copy test.
Split tracing from GC, relocation from prebuilding.
As the combinations of tests is sufficiently large to cause make serious work,
only build rules for tests that will be executed. Default to small number of
tests rather than full. Use ART_TEST_FULL=true to override.

Change-Id: Ic25431f29071b840d2e2295be6853511c0b52cfa

Merge "ART: Add non-temporal store support"

Reduce interpret-only compile time.

Before:
39.04user 5.18system 0:29.24elapsed 151%CPU (0avgtext+0avgdata 164176maxresident)k
38.87user 5.16system 0:29.14elapsed 151%CPU (0avgtext+0avgdata 164144maxresident)k

After:
36.26user 3.25system 0:27.00elapsed 146%CPU (0avgtext+0avgdata 162592maxresident)k
36.25user 3.28system 0:26.28elapsed 150%CPU (0avgtext+0avgdata 162688maxresident)k

Disabled implicit stack protection for the compiler, this reduces page faults.

Added support for not timing every method compilation and verification. NanoTime is
slow and adds ~2 seconds of real time. This is currently enabled since people want
to know which methods are slow to compile.

Bug: 16853450

Change-Id: I349ffb3f36db8c437137387aa6914dc17d743f09

ART: Add non-temporal store support

Added non-temporal store support as a hint from the ME.
Added the implementation of the memory barrier
extended instruction that supports non-temporal stores
by explicitly serializing all previous store-to-memory instructions.

Change-Id: I8205a92083f9725253d8ce893671a133a0b6849d
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>
Signed-off-by: Chao-ying Fu <chao-ying.fu@intel.com>

Merge "Fix race condition in TransitionCollector."

Fix race condition in TransitionCollector.

There was a minor race condition that could happen if multiple
threads called TransitionCollector at the same time. Starting out
in CMS:

Thread 1: TransitionCollector(SS).
Thread 2: TransitionCollector(CMS) sees that it is already CMS so
has copying_transition == false. But then thread 2 changes to SS.
Thread 1 resumes but incorrectly doesn't check the
disable_moving_gc_count_ possibly resulting in errors if
disable_moving_gc_count_ != 0 due to JNI since we are going from
SS -> CMS and are goign to move objects.

(cherry picked from commit de2233bbf64c7dde96513a729f6a97aebc2f0424)

Change-Id: I2fdda85f7cb7cbb0df3d1466a3c74853c5304c1b

Merge "ART: Improve error message on oat writer failure"

ART: Improve error message on oat writer failure

Try to log the pending exception to help resolve issues.

Bug: 16406811
Change-Id: I035ae9e59a5ee02b9e90c35c0644ec088d3c7d12

Merge "Fix usage of literals within a macro."

Fix usage of literals within a macro.

Macs handle arguments in macros as $num meaning literals must be $$num.

Change-Id: Ia30c7ef4e089561e9f23f353d7080e800b117d70

Merge "Map heap maps at a lower address."

Map heap maps at a lower address.

We now map the heap mmaps at a lower address in case the app needs
larger continuous address space.

Bug: 16502380

(cherry picked from commit 18bacb60e09fa8aa20ae0a1d7a4a8968d4913fb0)

Change-Id: I66447f853e439448c72923e2d1c657c94c5f1543

Merge "ART: kSuper also has IncompatibleClassChangeError"

ART: kSuper also has IncompatibleClassChangeError

When resolving methods, kSuper can also have
IncompatibleClassChangeError.

Bug: 17266767

(cherry picked from commit 9cb8d7a9a5013dd1e6734d9643573a4750d869e2)

Change-Id: I5fafe03ad578f605825a3d0c89f1254a4a385b76

Merge "ART: Fix IncompatibleClassChange check for ArtMethod"

ART: Fix IncompatibleClassChange check for ArtMethod

In case of kSuper we should complain sometimes.

Bug: 17265201

(cherry picked from commit 249907fe09afd9243c9675a67b157b98378fa3cf)

Change-Id: Ifad903a750495963c7128296713fbd7c4add71cd

Merge "Quick compiler: fix DCHECK"

Quick compiler: fix DCHECK

Cherry-pick from internal 532582

A DCHECK intended to verify the soundness of the mapping between
a wide Dalvik value and a pair of temp registers incorrectly fired
when the temp register pair was associated with a wide temporary
(i.e. - when (SReg() == INVALID_SREG)).

In this particular situation, the wideness flag is meaningful only when
there is an associated valid vreg.  Code rearranged to only perform
the DCHECK and reset the partner's wideness bit when we're dealing with
a live wide value.

b/16484538

Change-Id: I3b8a7bebc5d918c65ea56ae8db0a8a5cd8003386

Merge "CheckAndClearResolveException should allow LinkageError"

CheckAndClearResolveException should allow LinkageError

Bug: 16565964

(cherry picked from commit 118785c6f82ac1ec7074b5efe367b3dafd904940)

Change-Id: I7bc170e7a9ee9bd19defdd20187ee36a2381bf5d

Merge "Reduce GC log spam."

Reduce GC log spam.

Homogeneous compaction and collector transitions now print to
VLOG(gc) instead of LOG(INFO).

Bug: 17166556

(cherry picked from commit 0ab56e95e203806dae347056d3331a3e6037457d)

Change-Id: I33157ca27da86cbb741759bb0bda6d38d1f5bfcd

Merge "Fixed typo in entrypoint stub."

Fixed typo in entrypoint stub.

Fixed typo for x86 stub causing mac build failure.

Change-Id: I5e063fb87c598c2d9beb22f6d5b97b7fd4df9b84

Merge "Remove an intermediate field array in ClassLinker::InitializeClass."

Remove an intermediate field array in ClassLinker::InitializeClass.

This improves on CL 105090 by removing the need for the intermediate
field array by iterating two iterators in parallel.

Bug: 12687968
Change-Id: I32d4570948a9d39ccb1c20a02e18b48f15ed0738

Merge "ART: Extending FlagsOf"

Merge "Reduced memory usage of primitive fields smaller than 4-bytes"

ART: Extending FlagsOf

Modified FlagsOf to handle extended flags.

Change-Id: I9e47e0c42816136b2b53512c914200dd9dd11376
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>

Reduced memory usage of primitive fields smaller than 4-bytes

Reduced memory used by byte and boolean fields from 4 bytes down to a
single byte and shorts and chars down to two bytes. Fields are now
arranged as Reference followed by decreasing component sizes, with
fields shuffled forward as needed.

Bug: 8135266
Change-Id: I65eaf31ed27e5bd5ba0c7d4606454b720b074752

Merge "Reduce log spam by changing a LOG(INFO) to VLOG(startup)"

Reduce log spam by changing a LOG(INFO) to VLOG(startup)

Bug: 17166556
Change-Id: I9975ba07ad8f119a6e367e2770a60040a462149b

Merge "Support running without a boot image."

Merge "Fix GC memory overhead accounting."

Fix GC memory overhead accounting.

There was some missing null checks.

Bug: 16238192

(cherry picked from commit 2e290fb35ba1959e5a0ac85e87591ab9623808c1)

Change-Id: I4220272ac9c194e30fc307fca9918a4bb725e261

Support running without a boot image.

Bug: 17000769

Change-Id: I6404d5050c8a2f4ee6e70d58532eb25ee9de248e

Merge "Clean up Handle usage."

Clean up Handle usage.

Prefer using ConstHandle instead of Handle as function arguments
since you can't assign new references to ConstHandle which helps
prevent bugs.

Changed NullHandle to be a ConstHandle so that you can never modify
it to be a non null reference.

Change-Id: I81cb979f6f8d5b49e5614966a2caf28c3701dd4f

Merge "ART: Clean up compiler"

ART: Clean up compiler

Clean up the compiler: less extern functions, dis-entangle
compilers, hide some compiler specifics, lower global includes.

Change-Id: Ibaf88d02505d86994d7845cf0075be5041cc8438

Merge "Add fallback for non moving space being full."

Add fallback for non moving space being full.

When the non moving space becomes full, we disable moving GC and
make the main space the new non moving space.

Also added a runtime option for changing the non moving space size:
-XX:NonMovingSpaceCapacity.

Bug: 17189964

(cherry picked from commit 4c5a469683e433f126c9863cd393747d2e7c4a29)

Change-Id: If82e3c6f8a0f389e37e14a0b1e6d5126d571fd7a

Merge "ART: Support MIRGraph constant interface"

ART: Support MIRGraph constant interface

-Adds a helper to be able to ask for a wide constant.
-Allows MIRGraph to provide interface to set constants.

Change-Id: Id282ee1604a0bd0bce6f495176d6bca35dcd5a00
Signed-off-by: Razvan A Lupusoru <razvan.a.lupusoru@intel.com>

Merge "Using the correct time for wait_start_ms when lock contention logging is enabled"

Using the correct time for wait_start_ms when lock contention logging is enabled

Bug: 17208457
Change-Id: I35555e0acf3b37bb3776a62c64d59ae93403b237
(cherry picked from commit b894a19dfd668b6779de939cf5265b7e409d8809)

Merge "ART: Reject field accesses to non-reference vregs."

ART: Reject field accesses to non-reference vregs.

The verifier will now reject any iget/iput insts
that do not have a reference object in vB.

Bug: 17207857
Change-Id: I2826f0f7abfc88561f4da9486588592f4b296d14
Signed-off-by: Stuart Monteith <stuart.monteith@arm.com>

Merge "Handlerize MethodVerifier::mirror_method_."

Handlerize MethodVerifier::mirror_method_.

The method (mirror_method_) wasn't handlerized across some
allocation/GC points such as the ResolveType() call in
ScanTryCatchBlocks() and the GetReturnType() calls in
CodeFlowVerifyInstruction().

Bug: 12687968
Change-Id: I95323de14459eb5a7c4abfcf44f882f86d59be64

Merge "Fix testing wrong variable for non zygote."

Fix testing wrong variable for non zygote.

For non zygote we want to only have a single non moving space to
let dex2oat use RosAlloc for all allocations.

Fixed stale low memory logic in parsed options.

Bug: 17189964

(cherry picked from commit a12c2a945fe3f10d3ed31aa5db3e422cc1ad7f98)

Change-Id: I4fc00b277db6780ed410eab4cd969e8f004d5348

Merge "Always setup the native bridge library"

Always setup the native bridge library

... even if the string is empty. This will initialize the native bridge
library but mark it as unavailable.

- also, rename native_bridge_library_path to
native_bridge_library_filename to be closer to the actual meaning (it's
just the filename without any path).

Bug: 16404669

(cherry picked from commit I94628639691459d48d1fbf0841f36b68d51818e7)

Change-Id: I94628639691459d48d1fbf0841f36b68d51818e7

Merge "GetDalvikDisassembly should work even without SSA info"

Merge "ART: Fix frontend to allow possibility of multiple code units"

Merge "ARM64: Avoid the duplication of condition codes."

Merge "AArch64: Improve MIR to LIR translation for abs"

ARM64: Avoid the duplication of condition codes.

Merge "ART: Add dex dependency for gtest"

Merge "Fix signal test build"

Fix signal test build

Bug: 17188805

(cherry picked from commit b9d6a55eb4eb5872739ed26ac535f0c6342e813b)

Change-Id: I0a271aa214eebe11a8442ebf68811e0d8f077d36

Merge "Add supported architectures to signal test."

Merge "Fix fault handler to unregister on shutdown"

Merge "Remove profiler log line"

ART: Add dex dependency for gtest

Change-Id: I22d5e9b69e40585c6ee57381ee758b4b32e07f96

Merge "ART: Remove a BasicBlock's fall_through pointer in Hide()"

Merge "Avoid handle-less fields in ClassLinker::InitializeClass()"

Merge "ART: Implement kMirOpNullCheck"

Avoid handle-less fields in ClassLinker::InitializeClass()

There were some handle-less fields in a SafeMap across GC points.

Bug: 12687968
Change-Id: Ib8c6527d4e23031f1d0074fa11d8f85499b68340

Merge "Make a couple of map checks debug only."

Merge "Fix host 64-bit ISA string"

Fix host 64-bit ISA string

Bug: 17185619, 15358152

Change-Id: I5b80e36890abf659634c38ec7dd6fa336bf8a75a

Make a couple of map checks debug only.

This cost us close to 80ms in app startup times.

The checks that a reused region was within an already
existent map has been demoted to a debug check.

A couple of other negative checks have been removed
outright because one of them was superflous and the other
wasn't guaranteed to be correct.

bug: 16828525

(cherry picked from commit bddaea2b88b0a19d9cc7a4dea772af8e829323b3)

Change-Id: Ia6f3e69692bb9cb5b4ff6f47946ea38a56d4cdb6

Merge "ART: Remove GOT setup from x86 assembly where not needed"

ART: Remove GOT setup from x86 assembly where not needed

Change-Id: Id839d6d6427da7b7faa4340e04085cdab388e9a5

Fix fault handler to unregister on shutdown

This fixes a problem with the fault handler where it wasn't
unregistering itself during shutdown of the runtime.

Bug: 17133266

(cherry picked from commit e8b9afcd0cd86b8808af29a97332038aab70c604)

Change-Id: I1a4ec4292ec049046dda30769265680201729efb

Merge "ART: A couple of checks were missed in class LockWord"

Add supported architectures to signal test.

This adds ARM, AARCH64, i386 and x86_64 to the supported architectures
in 004-SignalTest

Bug: 16948053

(cherry picked from commit d4af31aa69fe8786a291c566c375bbac04da9ced)

Change-Id: I17a992b2cf47f8744f867b8e3f1c360aa345093d

Remove profiler log line

This removes a LOG(INFO) saying that the profile is disabled.

Bug: 17166556

(cherry picked from commit b9c3888380666a7b44718f04f787693787cd57c6)

Change-Id: I9c9c470f554605a1183bcf84c5d864f9c4f24f72

Merge "x86: A couple of minor changes for String.indexOf() inlining"

Merge "ART: Fix verifier fail message."

Merge "Fix OOM test case to handle more optimizations"

Merge "Check pending exception result in AllocObjectWithAllocator."

Check pending exception result in AllocObjectWithAllocator.

Possible previous bug:
Allocation fails due to OOM and the collector transitions.
This caused us to incorrectly retry the allocation with a pending
exception. We now return null if there is a pending exception.

Bug: 17164348

(cherry picked from commit 27f5ae830c5418fa92094608a6e9f693ea88bb69)

Change-Id: I8e7ecafbade7f0b955812df3dd7c94fdc696c856

Merge "Fixed wrong field usage in Instrumentation::RemoveListener"

Fixed wrong field usage in Instrumentation::RemoveListener

Change-Id: I574fcacf754c1da1dc03459d5859b04bcebeac11
Signed-off-by: Daniel Mihalyi <daniel.mihalyi@mattakis.com>

GetDalvikDisassembly should work even without SSA info

It is good if GetDalvikDisassembly can dump MIR even if
SSA register info is not available. Without this patch it
crashes.

Change-Id: I704c28c891cd2580a7819f7fd972167c3bf67ddc
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

Merge "Kill HAVE_ANDROID_PTHREAD_SETNAME_NP."

Kill HAVE_ANDROID_PTHREAD_SETNAME_NP.

All it means is __BIONIC__ anyway.

Change-Id: I3ef4ef1a14d0a77c75539f2ab533ee2f3964e1a5

Merge "ART: Relax GetInstructionSetFromString"

ART: Remove a BasicBlock's fall_through pointer in Hide()

Remove the fall_through link that a BasicBlock has to its child when
calling its Hide function.  Useful for removing the visible link that is
there when we dump the CFG.

Change-Id: Ie82c48b195d1d62e2984d284dcc29ed7d7444db2
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>

ART: Relax GetInstructionSetFromString

Do not abort on an unknown instruction set string. Instead return
kNone and let the caller handle this.

Also simplify the patchoat tool to use this.

Bug: 17136416

(cherry picked from commit aabbb2066a715b3fd8e752291f74c6d77b970450)

Change-Id: I24131914bcf91c04ae93179bf809a2907f1f2b7a