Merge "Fix build aflter conflicting changes got in."
am: 07f7affb33

* commit '07f7affb338b529d34e4e18b7e8a631c16bd9765':
  Fix build aflter conflicting changes got in.

Merge "Small inlining improvements."
am: 1d01ef7f0f

* commit '1d01ef7f0f79b032449c35203ca9543fb8a0b22f':
  Small inlining improvements.

Merge "Add statistics support for some optimizations"
am: afe8400458

* commit 'afe84004583488ded7f99bb703ed4c1a601ff034':
  Add statistics support for some optimizations

Merge "Implement polymorphic inlining."
am: f8a08735f6

* commit 'f8a08735f6c60ddee2bb5646dcad561361cce079':
  Implement polymorphic inlining.

Merge "Implement Optimizing\'s constant folding as a visitor."
am: 99da8cd881

* commit '99da8cd8818e015b21d1c2af73ac40749698c930':
  Implement Optimizing's constant folding as a visitor.

Merge "Mark __jit_debug_register_code as a unique symbol on aarch64"
am: d9c05309e8

* commit 'd9c05309e8fab26a6087abd5773859f112cdefe9':
  Mark __jit_debug_register_code as a unique symbol on aarch64

Merge "Remove string init map."
am: 7d9ddcc909

* commit '7d9ddcc909e3a76ce516649c93a8d854b1cf2fb4':
  Remove string init map.

Merge "Fix build aflter conflicting changes got in."

Fix build aflter conflicting changes got in.

Change-Id: Ia3846e9a0dad137fe797bf3f2ece0cb78c106ec0

Merge "Small inlining improvements."

Merge "Add statistics support for some optimizations"

Add statistics support for some optimizations

This patch adds support for the --dump-stats facility with some
optimizations
and fixes all build issues introduced by the patch:
I68751b119a030952a11057cb651a3c63e87e73ea (which got reverted)

Change-Id: I5af1f2a8cced0a1a55c2bb4d8c88e6f0a24ec879
Signed-off-by: Jean-Philippe Halimi <jean-philippe.halimi@intel.com>

Merge "Implement polymorphic inlining."

Small inlining improvements.

- Use the type_index in the current dex file for classes not
  defined in the current dex file.
- Make the loading of the vtable field of a class have no side effects
  to enable gvn'ing it.

Note that those improvements only affect the JIT, where we don't have
checker support.

Change-Id: I519f52bd8270f2b828f0920a1214da33cf788f41

Implement polymorphic inlining.

For example, before:
HInvokeVirtual

After:
If (receiver == Foo) {
  // inlined code.
} else if (receiver == Bar) {
  // inlined code
} else {
  // HInvokeVirtual or HDeoptimize(receiver != Baz)
}

Change-Id: I5ce305aef8f39f8294bf2b2bcfe60e0dddcfdbec

Merge "Implement Optimizing's constant folding as a visitor."

Merge "Mark __jit_debug_register_code as a unique symbol on aarch64"

Merge "Remove string init map."

Remove string init map.

Partial revert of the String init change.

- Make Quick bailout in the presence of String allocation.
- Rely on the compiler for knowing when dex registers alias.

bug:27173201

Change-Id: I0bf58ba3825c71cef110b53f3a0a6f567cb2ef9a

Merge "Set bias on != comparison for isNaN."
am: a1f65135cd

* commit 'a1f65135cd2315159ac302f904ba0c5ba0d7fd0e':
  Set bias on != comparison for isNaN.

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

Set bias on != comparison for isNaN.

Change-Id: I83969ecf7252b5e001bdd501c4ca31e7d0608854

Merge "Clean up register line"
am: f598e2f183

* commit 'f598e2f183f496065597267a6fc142aa9a56e7c4':
  Clean up register line

Merge "Clean up register line"

Merge "Implement isNaN intrinsic through HIR equivalent."
am: 293a79e89c

* commit '293a79e89c9dbdb63932bf2444d991fd6901a7a1':
  Implement isNaN intrinsic through HIR equivalent.

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 commit 'dfd4a9e8d36ad791505c6d80c0f627e6a0a764bf' into HEAD

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

Implement Optimizing's constant folding as a visitor.

Refactor the logic of art::HConstantFolding::Run into a new
visitor, art::HConstantFoldingVisitor.

Change-Id: Id8d3c3826f6dff6cc2d18a01f6c48d79bde483b3

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