Add VLOG(collector) to reduce log spam

VLOG(gc) is enabled from AndroidRuntime, using it too much produces
log spam.

Bug: 22858531
Change-Id: I5306c512eb7f55a178fa607d4b211ef25246e300

Use image oat file instead of image header for immune spaces

The old immune spaces logic used the oat file information in the
image header instead of the actual oat file pointer. This was
incorrect for the app image case since the app image oat file is
not necessarily at the address specified in the header. This bug
could cause an incorrect immune region that caused large objects
to get freed if they were within this immune region.

Added test.

Bug: 22858531

(cherry picked from commit 5351da0225d027a19420153615634a1c78966bca)

Change-Id: Ibf41b0c0a9a7b0d093146311e2603a186033e339

Don't pass in oat file name to NativeLocationInImage

We already have the oat file name stored in the relocation, using
the one passed in for the image info can cause a mismatch.

I suspect this fixes a bug repated to copied methods where there
was a oat file name mismatch. When we assign the native object
relocation for the methods of a class, we use the dex cache from
that class.

However, we use method's declaring class in NativeLocationVisitor
by using ArtMethod::GetDexCache. This could result in using the
wrong ImageInfo if the declaring class is in another image.

Bug: 27216437

(cherry picked from commit e8bf134734ee7de8b89e6d3e3648de6548d8e627)

Change-Id: I9c764800370d50b2f1f121415a68fa8f25d22e63

Fix issue with copied methods not being checked.

In several places we were using IsMiranda to check if a method is
copied. This misses cases involving default methods.

(cherrypicked commit fcea56f9cc51957161fe7a6e35e895fd8c4c4a7f)

Bug: 27216437

Change-Id: I8c800e3e622a9c0ca0f8752c3d5202f433af9a1c

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

(cherry picked from commit 91edc62a9d8d4d8153b6b04140b50a3724cd3597)

Change-Id: I27aea3322ce925909e7ace59d91fc9299f2549ba

Merge "ahat: annotate char[] objects with their string values." into nyc-dev

Merge remote-tracking branch 'goog/master' into nyc-dev

bug:27094810
bug:27149923
bug:27019403
bug:23128949
Change-Id: Iff7685a5000653a3c87d8ba8d316965b9592a594

Merge "Be a bit smarter with JIT code triggering deoptimization."
am: 39e4fab868

* commit '39e4fab868a6052e0fab75e2668600b0e26daa42':
  Be a bit smarter with JIT code triggering deoptimization.

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"
am: 442643920a

* commit '442643920a6c539e98aad76594e99b932b5631ba':
  Support CMOV for x86 Select

Merge "Support CMOV for x86 Select"

Merge "We also need to delete osr entries when deleting ArtMethod."
am: 0723a61dd0

* commit '0723a61dd00fcc7072d82407e0d4a40a8b141d43':
  We also need to delete osr entries when deleting ArtMethod.

Merge "dex2oat: Show memory usage values in bytes"
am: 908cb978e8

* commit '908cb978e81cdde9c0c02d131acbb3ff35aaf3db':
  dex2oat: Show memory usage values in bytes

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"
am: b93c21e83c

* commit 'b93c21e83c8fbf0191093c01a8951adb5be9010b':
  Add custom arena deleter for RegisterLine

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

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

Bug: 26987597
Change-Id: I1b0bf711c642f4b8a7fe56f0cd27496761db4147
(cherry picked from commit c7f7712a3808b8e3046eae8a4dbb7f6a7fc6faf2)

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

* commit '88f38e9b4e9ba0aca3d6fd4823be3a7226c87f33':
  ahat: annotate char[] objects with their string values.

Merge "Refactor the inliner."
am: 4a69a30120

* commit '4a69a30120ecba9431246938e124d56213447137':
  Refactor the inliner.

Merge "X86: Allow long compares to stack operands"
am: ce053b20e8

* commit 'ce053b20e87f33b4417b8f21bb6f2ec4feb3104e':
  X86: Allow long compares to stack operands

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

Remove some outdated comments in art tests
am: 5527ea8235

* commit '5527ea823585aa61d12c8226ec131d7962d3d1c1':
  Remove some outdated comments in art tests

Remove usages of jill.jar, use jack instead
am: 51e4d44178  -s ours

* commit '51e4d44178919bf717e58365ce52a3c852d28465':
  Remove usages of jill.jar, use jack instead

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."
am: a1f1296d83

* commit 'a1f1296d8353c22b43cb2c5437a3ef5b747f7a87':
  HInvokeStaticOrDirect may not have a special input.

Merge "Remove some outdated comments in art tests"
am: 8ea957f846

* commit '8ea957f846c2f0a0eaf3ac5c138942d2ed079b37':
  Remove some outdated comments in art tests

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>

Remove some outdated comments in art tests

(cherry picked from commit 6b9508d537560ed41d91879839d2ec90945eca99)

Change-Id: I354c45ad56d3f4b66ed55e6effccb0be6f1fc994

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

Merge "Remove usages of jill.jar, use jack instead"
am: 01ba22a3d5

* commit '01ba22a3d52e5359256adcbbb2cf9aa85ed5df51':
  Remove usages of jill.jar, use jack instead

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

(cherry picked from commit aaa779a0a5f7afc23814728de68c6fe3ae1450bf)

Change-Id: I87fc0cdd34cf8107601af38f2559597200908173

Remove usages of jill.jar, use jack instead

Change-Id: Icb403ee6fb5bf287d2b72bb253b683b1d95f4e07

Merge "Follow-up to OSR change."
am: 7eca244e79

* commit '7eca244e79480f2ecea341598524a53273959c2b':
  Follow-up to OSR change.

Merge "Follow-up to OSR change."

Merge "Remove code cache check from native bridge run-test"
am: 5322e55ab9

* commit '5322e55ab9a15996a197456ca39d9c77488cd5c1':
  Remove code cache check from native bridge run-test

Follow-up to OSR change.

Change-Id: I99677697178273490201a22aba21dfe5b8e2c8ba

Merge "Change how DWARF detects end of prologue."
am: db65d7453c

* commit 'db65d7453ce559dd01cf73df8832833c33949a80':
  Change how DWARF detects end of prologue.

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

Merge "Fix test by checking if we\'re using JIT."
am: 01298eec8f

* commit '01298eec8f988ce0f98651f89723ed4f0061b43e':
  Fix test by checking if we're using JIT.

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

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

* commit 'c017318a7c5a9142b1fff9f57eb4105b1d397bbd':
  Revert "Revert "Make it possible to enable native debugging through debug flags""

Merge "Fill some gaps in .debug_loc."
am: 2a776e1228

* commit '2a776e1228bd6d1e9f86a898f408d593c9017b80':
  Fill some gaps in .debug_loc.

Merge "Temporarily disable compressed CFI tests."
am: 3694e91faa

* commit '3694e91faa11567b568ddafab162d172503415b3':
  Temporarily disable compressed CFI tests.

Merge "Re-enable OSR."
am: 707954befd

* commit '707954befd63af9d970cc3f697dd9bfbdcbcf37b':
  Re-enable OSR.

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."
am: 7dc277b171

* commit '7dc277b171b4d8294c4d152d8a6020995dc95f98':
  Optimizing: Use range-based loops in BCE.

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

* commit 'dd867c87f32022e2bd8198bb1c294a611d81de50':
  Let the CFI test communicate via stdout rather than arbitrary sleep.

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."
am: afd33d77f2

* commit 'afd33d77f2df61467dae38cfd59b8e8e6a490058':
  Optimizing: Remove unnecessary And before TypeConversion.

Merge "Optimizing: Remove unnecessary And before TypeConversion."

Merge "Fix broken tests"
am: e8e3d9253f

* commit 'e8e3d9253fefffd843b42b1da76b37ef65bc100c':
  Fix broken tests

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()."
am: b3871cd241

* commit 'b3871cd241eed77bfdd83c36182c19273c351da6':
  ART: Fix ArenaStack::AllocWithMemoryTool().

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

Merge "Fix lint error"
am: af5b2a28ab

* commit 'af5b2a28ab0b35276fe6e527820ecccc350843bb':
  Fix lint error

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."
am: 26962cc2a1

* commit '26962cc2a1e773e99c1711bef9b36e199fdc8989':
  Expect less in the presence of a string init call.

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

* commit 'c86382a7d1685728fe000df43dd8ba7bc22278a5':
  Add an option to disable native stack dumping on SIGQUIT.

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

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

Merge "ART: Run SsaBuilder from HGraphBuilder"
am: b01e0e2e93

* commit 'b01e0e2e931b9670690a9eeffdbee73db588715d':
  ART: Run SsaBuilder from HGraphBuilder

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"
am: 9b8beaa5d0

* commit '9b8beaa5d04afcab0e3ddb6306ed610c0d11da8d':
  ART: Implement HSelect with CSEL/FCSEL on arm64

Merge "Optimizing: Simplify consecutive type conversions."
am: ccf88e56ff

* commit 'ccf88e56ff4e30b14aba8d4de0ae67aa9c178e5a':
  Optimizing: Simplify consecutive type conversions.

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"
am: b6de3d92e9

* commit 'b6de3d92e932c0cded1c2e208f6b8fa4eebe6f02':
  Switch the tests to using Jack for compilation

Merge "Switch the tests to using Jack for compilation"