Revert "Revert "Revert "Revert "ART: Enable Jit Profiling in Mterp for arm/arm64""""

This reverts commit 5d03317a834efdf3b5240c401f1bc2ceac7a2f25.

We need to catch all possible cases in which new instrumentation appears
or the debugger is attached, and then switch to the reference interpreter
if necessary.  We may, in a future CL, use the alt-mterp mechanism to accompish
this (as did Dalvik).

Only enables Arm64 for now.  Once it survives extended testing, will enable
arm and update x86.

Updated OSR handling to match other interpreters.

Change-Id: I076f1d752d6f59899876bab26b18e2221cd92f69

Merge "Set bias on != comparison for isNaN."

Set bias on != comparison for isNaN.

Change-Id: I83969ecf7252b5e001bdd501c4ca31e7d0608854

Merge "Clean up register line"

Merge "Implement isNaN intrinsic through HIR equivalent."

Implement isNaN intrinsic through HIR equivalent.

Rationale:
Efficient implementation on all platforms.
Subject to better compiler optimizations.

Change-Id: Ie8876bf5943cbe1138491a25d32ee9fee554043c

Clean up register line

Fixed formatting and style. Also added proper null handling for arena
deleter.

Change-Id: I6a44cb29ca6ad17b35e18dc6be1d12416f211631

Merge "Only visit pointer arrays once during image relocation"

Only visit pointer arrays once during image relocation

Previously they could get visited twice, this caused corruption of
the app image if a pointer was updated twice.

Bug: 22858531
Change-Id: I1f1ba1ba5dc205be07dba51bc6ce7a825c82b33a

Merge "Revert "Allow method references across oat files for multi-image.""

Revert "Allow method references across oat files for multi-image."

Breaks Quick tests.

This reverts commit 6065402316da2b51eed5fc34cffbd991766bd408.

Change-Id: I8a5469ba7cea5f46b85cb489b3e0ef06ed548f03

Merge "Fix uses of art::HCompare::IsGtBias on MIPS32 and MIPS64."

Fix uses of art::HCompare::IsGtBias on MIPS32 and MIPS64.

Do not call this method on non floating-point HCompare
instructions.

Change-Id: I262ab23f48d32295641063f897dca02cb92eb2e2

Merge "Allow method references across oat files for multi-image."

Allow method references across oat files for multi-image.

These were disabled because we didn't have sufficient
information about the multi-image layout when processing
link-time patches in OatWriter. This CL refactors the
ELF file creation so that the information is available.

Change-Id: I6f2e8dc8572d143c72cc2693bb0ba4fd76f889ab

Merge "Build jdwp tests with Jack in vogar"

Build jdwp tests with Jack in vogar

These tests are compiled with Jack in the CTS so we do the same
with vogar now.

Bug: 27218415
Change-Id: I69b86e02a6532c1885c4d40f0768bf2b353c9569

Merge "Extend constant folding to float and double operations."

Merge "Add simple deduplication for .debug_ranges."

Merge "MIPS32: peek*/poke*, and String.charAt intrinsics."

Extend constant folding to float and double operations.

Change-Id: I2837064b2ceea587bc171fc520507f13355292c6

Merge "Be a bit smarter with JIT code triggering deoptimization."

Be a bit smarter with JIT code triggering deoptimization.

Do not re-use an OSR method that triggered deoptimization.
Also add a stack overflow check before doing OSR.

bug:27094810
Change-Id: I6ff6a7fb9b3df9b7c0ff37e3610595efa70ad067

Merge "Support CMOV for x86 Select"

Merge "We also need to delete osr entries when deleting ArtMethod."

Merge "dex2oat: Show memory usage values in bytes"

We also need to delete osr entries when deleting ArtMethod.

In the unfortunate event an ArtMethod gets allocated at the
same location as an old (deleted) ArtMethod, the osr_code_map_
lookup will succeed and return garbage. So we need to delete
entries in the osr_code_map_ when an ArtMethod gets deleted.

This should finally fix:
dalvik.system.DexClassLoaderTest#test_twoJar_diff_getResourceAsStream

Change-Id: I7c8b775c3376a6cfcb907f09b783e393967ad82d

Merge "Add custom arena deleter for RegisterLine"

Add custom arena deleter for RegisterLine

Previously it protected by using sizeof(RegisterLine) but this was
not accurate due to register lines being variable sized.

Bug: 27156726
Change-Id: Ia4b134b85a2e61993d17bd4f0eff60c89c164dc3

Merge "ahat: annotate char[] objects with their string values."

Merge "Refactor the inliner."

Merge "X86: Allow long compares to stack operands"

ahat: annotate char[] objects with their string values.

Bug: 26987597
Change-Id: I1b0bf711c642f4b8a7fe56f0cd27496761db4147

Refactor the inliner.

In preparation for more polymorphic inlining, refactor the inliner
a bit.

Change-Id: Ie3fd6c1ef205f1089989c67a527e6f57ff3c8b5d

Merge "HInvokeStaticOrDirect may not have a special input."

X86: Allow long compares to stack operands

There is no need to force the RHS stack operand into a register for a
long comparison.

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

Merge "Remove some outdated comments in art tests"

HInvokeStaticOrDirect may not have a special input.

For irreducible loops, we disable the generation of
HX86ComputeBaseMethodAddress, so intrinsics code should not assume
it's there.

bug:27149923

Change-Id: I78ba0ca7aefa4033227c77ba438b6eaca53dadd9

Remove some outdated comments in art tests

Change-Id: I5604bd8d9a433eb513a516b1839b55c88c329329

Support CMOV for x86 Select

If possible, generate CMOV to implement HSelect.  Tricky cases are a
long or FP condition (no single CC generated), FP inputs (no FP CMOV)
and when the condition is a boolean or not emitted at the use site.
In these cases, keep using the existing HSelect code.

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

Merge "Remove usages of jill.jar, use jack instead"

Remove usages of jill.jar, use jack instead

Change-Id: Icb403ee6fb5bf287d2b72bb253b683b1d95f4e07

Merge "Follow-up to OSR change."

Follow-up to OSR change.

Change-Id: I99677697178273490201a22aba21dfe5b8e2c8ba

Merge "Remove code cache check from native bridge run-test"

Remove code cache check from native bridge run-test

Every now and them the native bridge run-test failed because the code
cache cannot be found. I suspect a temporary I/O issue when stat-ing the
directory. However I could not verify yet that's actually the cause.

Native bridge can work without a code_cache so we can relax the test
until we figure out the flakiness.

Bug: 27189715
Change-Id: I2ad3f93e80c0e2df89adf1d9d2afe1087c10af08

Merge "Change how DWARF detects end of prologue."

Merge "Fix test by checking if we're using JIT."

Fix test by checking if we're using JIT.

Change-Id: Ifa98815ee2f8af4f338c9469f5afd74fc01022eb

Change how DWARF detects end of prologue.

Instead of ignoring stack map without dex register maps altogether,
explicitly calculate end of prologue as first stack map which has
dex register maps. This gives us more line number mappings.

Change-Id: I643cf0f9daf9762d894fbc94f5ff981ef6c0583b

Add simple deduplication for .debug_ranges.

Variables with same scope can usually share .debug_ranges entries.

Change-Id: I855f456782afdcc8ac5f622365d62ba8950a5c95

Merge "Revert "Revert "Make it possible to enable native debugging through debug flags"""

Merge "Fill some gaps in .debug_loc."

Merge "Temporarily disable compressed CFI tests."

Merge "Re-enable OSR."

Re-enable OSR.

1) Fix flaky test by doing explicit ProfilingInfo allocation,
   and OSR compilation instead of relying on the system. Also
   make sure the method $noinline$inlineCache always starts
   with the interpreter, as otherwise we would infinite loop.

2) Ensure the thread doing the OSR cannot be suspended once it
   holds an OSR method, as otherwise the jit code cache could
   delete the method before the thread had a chance to jump to it.

Change-Id: Ic0dee181a7b23260419a94a7d7405f150433d31a

Temporarily disable compressed CFI tests.

Since the support in libunwind has been temporarily disabled.

The non-compressed CFI test still works.

Bug: 27172087
Change-Id: Ic0e088827122014f13ae2086c766fc7c01b21ea1

Merge "Optimizing: Use range-based loops in BCE."

Merge "Let the CFI test communicate via stdout rather than arbitrary sleep."

Optimizing: Use range-based loops in BCE.

Change-Id: Ib7cbc6dcbdf61d0b115e6b872914cff3687ad6e4

Merge "Optimizing: Remove unnecessary And before TypeConversion."

Revert "Revert "Make it possible to enable native debugging through debug flags""

This reverts commit 3a98aae1b9b20bc78dc5e05d2e60cb1d0072db02.

* Add support for a new debug flag disabling the optimizations in
  the compiler and enable the generation of some additional debug
  info (--native-debuggable).
* Ignore the content of the oat files if force JIT is enabled so
  the runtime ignores the AOT-ed code what doesn't contain any
  debug info.

Time measurements on a Nexus 5 with running:
am start -n com.facebook.katana/com.facebook.katana.LoginActivity -W

Before change:             | AVG | DEV
--------------------------------------
ThisTime:  549 492 512 511 | 516 | 24
TotalTime: 549 492 512 511 | 516 | 24
WaitTime:  662 511 528 526 | 557 | 71

After change:              | AVG | DEV
--------------------------------------
ThisTime:  530 467 503 544 | 511 | 34
TotalTime: 530 467 503 544 | 511 | 34
WaitTime:  551 497 536 583 | 541 | 36

Based on the numbers the speed impact of the change is less then the
accuracy of the measurement and it is also negligible.

The minor speed improvement displayed in the measurements are just
the cause of the variance of the measurement and not caused by this
change.

Change-Id: Ibf7294bfd14f8632a0fdeb27a5b90bfe5037d085

Merge "Fix broken tests"

Fix broken tests

Change-Id: I9d0f1064e8cd63045283445946c46594a50d4b84

Merge "ART: Fix ArenaStack::AllocWithMemoryTool()."

ART: Fix ArenaStack::AllocWithMemoryTool().

MEMORY_TOOL_MAKE_NOACCESS() takes the size of the address
range as the second argument, not the end the range.

Bug: 27156726
Change-Id: I05c8224a1d3c619919b203f407fb770c7c49cc9f

Merge "Fix lint error"

Let the CFI test communicate via stdout rather than arbitrary sleep.

I suspect the gcstress test might have been flaky due to the sleep not
being long enough and I would like to avoid just arbitrarily increasing it.

Change-Id: I5d4e0be35aa9f48b66167df04b8a539b21787053

Fix lint error

Change-Id: I6f7785abece7ead31b4737fc6b54712c26582e2b

Merge "Expect less in the presence of a string init call."

Merge "Add an option to disable native stack dumping on SIGQUIT."

Add an option to disable native stack dumping on SIGQUIT.

Some of our (stress) run-tests do ANR dumping, which end up
stressing libunwind, that has known problems. To avoid getting
flakes due to libunwind, disable native stack dumping on SIGQUIT
for our run-tests.

bug:27185632
bug:24664297
Change-Id: I69085e48db903d6240448d71666ae2dcd091922e

Expect less in the presence of a string init call.

The compiler currently relies on the dex cache being populated
for doing proper type propagation. If it hasn't, we may end up in the
situation where the DexMethodInliner has recognized a String.<init> call
(because DexMethodInliner only looks at signatures, and does not resolve
types), but the graph builder doesn't see a type and assume it needs
to do access checks and clinit checks on it.

Change-Id: Id79313b0610b127909e3e057305b6632b0b172f7

Merge "ART: Run SsaBuilder from HGraphBuilder"

Merge "ART: Implement HSelect with CSEL/FCSEL on arm64"

Merge "Optimizing: Simplify consecutive type conversions."

Optimizing: Remove unnecessary And before TypeConversion.

For example `(byte) (x & 0xff)` doesn't need the `& 0xff`.

Bug: 23965701
Change-Id: I5fc8419491aff2cdc7074451e74e873b5f582d41

Optimizing: Simplify consecutive type conversions.

Merge two consecutive type conversions to one if the result
of such merged conversion is guaranteed to be the same and
remove all implicit conversions, not just conversions to the
same type. Improve codegens to handle conversions from long
to integral types smaller than int.

This will make it easier to simplify `(byte) (x & 0xffL)` to
`(byte) x` where the conversion from long to byte is done by
two dex instructions, long-to-int and in int-to-byte.

Bug: 23965701
Change-Id: I833f193556671136ad2cd3f5b31cdfbc2d99c19d

Merge "Switch the tests to using Jack for compilation"

ART: Implement HSelect with CSEL/FCSEL on arm64

Change-Id: I549af0cba3c5048066a2d1206b78a70b496d349e

Merge "X86_64: Allow HSelect to generate CMOV from memory"

X86_64: Allow HSelect to generate CMOV from memory

Use the cmov with Address operand to allow CMOV from stack location.

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

dex2oat: Show memory usage values in bytes

Memory usage values are rounded down to an integer and shown in the
unit that would be most appropriate (refer to art::PrettySize());
e.g. if the value is at least 1 MB, but less than 1 GB, then it will
be shown in MB. However, that can introduce a very large error; for
example, a value that is close to, but less than 2 MB, will be
displayed as 1 MB.

This change forces dex2oat to print the raw value in bytes as well,
which may make memory usage analysis more accurate.

Change-Id: Id86a9fe21e8af0f02e77fac21cad51d35d941294

Merge "Add X86/X86_64 support for CMOV from memory."

Merge "Add MODULE_LICENSE_GPL.. for art/runtime/openjdkjvm."

Add MODULE_LICENSE_GPL.. for art/runtime/openjdkjvm.

This needs a companion makefile that declares a no-op module
so that the build system can leach of its LOCAL_PATH to collect
all subdirectory sources.

bug: 27160386

(cherry picked from commit 9a3a10ac819f456f7bcc626046342a845902fe96)

Change-Id: Ib11845c2172aad272c45ec2a3b4b2f7d508c1596

ART: Run SsaBuilder from HGraphBuilder

First step towards merging the two passes, which will later result in
HGraphBuilder directly producing SSA form. This CL mostly just updates
tests broken by not being able to inspect the pre-SSA form.

Using HLocals outside the HGraphBuilder is now deprecated.

Bug: 27150508
Change-Id: I00fb6050580f409dcc5aa5b5aa3a536d6e8d759e

Merge "Don't call IsNative if the declaring class can be null."

Merge "Disable on stack replacement for now."

Disable on stack replacement for now.

Change-Id: I7b57252d4872e8c280e95de4ad4ff55a3ca22685

Don't call IsNative if the declaring class can be null.

There's a check down below the IsNative call that makes
sure the declaring class is not null.

Change-Id: I744bf01105eadeea9893ae626970c4b6b2f64a8b

Merge "ART: Simplify patchoat"

ART: Simplify patchoat

Removed unused configurations. Cut out cruft that isn't used anymore
since multi-image. Make it smaller to not require new selinux
permissions for OTAs.

Split up the main function, to make the flow clearer for image vs
app/oat.

Bug: 25612095
Bug: 26317072

(cherry picked from commit 02bc0b70beedfe258c2147c2b4a205fbf3d6eebc)

Change-Id: Ieb55bd1a13cbb44fd0e2979a8a61a509df9c1e7e

Merge "Fix bug with verification of constructors"

Fix bug with verification of constructors

We would incorrectly allow the storing of values into superclass
fields before the superclass constructor was called.

Bug: 26965384

Change-Id: I45b824fbdbfc133663ed6d3306853595b5dc9262

Merge "Generalized "dom-based" dynamic BCE to symbolic base + offset."

Generalized "dom-based" dynamic BCE to symbolic base + offset.

Rationale:
So far, if all others failed, BCE would use a dominator-based
dynamic deoptimization to eliminate bounds checks in e.g. a[0],
a[1], a[2], etc. This CL generalizes this to any symbolic base
with offset in e.g. a[base], a[base+1], etc. The runtime tests
(two for symbolic, one for constant) carefully account for
arithmetic wrap-around.

bug=26680114

Change-Id: I7432a200fd69791914ed776c77fa62567b5863c0

Add X86/X86_64 support for CMOV from memory.

Add support for the memory form of CMOV.  Add tests.

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

Merge "Fix app image memory leak"

Fix app image memory leak

There was a memory leak if we failed to add an image space, this
was caused by releasing ownership without adding it to the heap.

Bug: 22858531

Change-Id: Ia3c97ec822ca5362f52008e813fa434107198bc9