am bb3dabaf: Merge "ART: Improve Indenter performance."

* commit 'bb3dabaf804e0d0bd2bf34e69091b2f86fd4e78b':
  ART: Improve Indenter performance.

Merge "ART: Improve Indenter performance."

am 568e8f39: Merge "Fix 496 test."

* commit '568e8f394b264a9b7932f081f971b141e563641e':
  Fix 496 test.

Merge "Fix 496 test."

Fix 496 test.

Make sure the class requested by LoadedByMyClassLoader.bar
is not loaded yet ('Main' is already loaded), in order to
do a proper class loader lookup and not use a dex cache
version.

Change-Id: I77215ae489e6d2c50b90684951bccd0bbf8c1a53

ART: Improve Indenter performance.

This reduces the time taken by oatdump_test by ~15s (16%)
on host and ~30s (5%) on N5. The Indenter.overflow() was
previously the hottest function according to perf.

Change-Id: I96eb69ce3f5420cd09dba23b29f8025cedc3d80a

am d4a02cfc: Merge "Replace some run-time assertions with compile-time ones in ART."

* commit 'd4a02cfc59a449d216faa687cfdaa4252b4a433f':
  Replace some run-time assertions with compile-time ones in ART.

Merge "Replace some run-time assertions with compile-time ones in ART."

am 0022bdf7: Merge "ART: Fix data loss when symbolizing oat files."

* commit '0022bdf71533a300deebe6b9cf287eaceeb0620c':
  ART: Fix data loss when symbolizing oat files.

Merge "ART: Fix data loss when symbolizing oat files."

am 508755a3: Merge "Use GetOffsetDuringLinking instead of GetOffset."

* commit '508755a3d4d3d67aa834c033c79073c0a08a8fbd':
  Use GetOffsetDuringLinking instead of GetOffset.

Merge "Use GetOffsetDuringLinking instead of GetOffset."

am 87dc1b09: Merge "ART: Compiler generated GC map should take care of temp registers."

* commit '87dc1b09cee6e972fbe3ad48ba8b5ae68d37085f':
  ART: Compiler generated GC map should take care of temp registers.

Use GetOffsetDuringLinking instead of GetOffset.

GetOffset assumes the class has been resolved, but LinkFields
is called before the class is marked as such.

Change-Id: I6f9fc21cd220881806f00461a82a14e459e26a8b

Merge "ART: Compiler generated GC map should take care of temp registers."

am b5061a82: Merge "Do not overwrite an input register in shift operations."

* commit 'b5061a821d96cb1af7ba24d21a4d2d59f7f16c7c':
  Do not overwrite an input register in shift operations.

Merge "Do not overwrite an input register in shift operations."

Do not overwrite an input register in shift operations.

'second_reg' is an input register that can survive the instruction.
Instead use the output register as a temporary result.

bug:21667432
Change-Id: I1a4577b0333c3fb184645023d5eae30555bbf65c

am 1f22dbc1: Merge "ART: Fix GraphVisualizer dlopen crash"

* commit '1f22dbc10d9fdc96a6814e737718098e36a0ea3c':
  ART: Fix GraphVisualizer dlopen crash

Merge "ART: Fix GraphVisualizer dlopen crash"

ART: Fix GraphVisualizer dlopen crash

The HGraphVisualizer used to crash if it failed to dlopen the
libart(d)-disassembler.so. This patch fixes that.

Bug: 22001285
Change-Id: Ibe9b9af17a77e22e3571c886f77918e9491b7827

am da0e224d: Merge "Restore DDMS recent allocation tracking\'s behavior"

* commit 'da0e224d79262b11fbac9e7b1fdc11c4df31b77f':
  Restore DDMS recent allocation tracking's behavior

am 3355963c: Merge "Fix another miranda method moving GC bug"

* commit '3355963cee00fe97cc7aaeaab2c6e31c5657ff6a':
  Fix another miranda method moving GC bug

Merge "Restore DDMS recent allocation tracking's behavior"

Merge "Fix another miranda method moving GC bug"

Fix another miranda method moving GC bug

Need to copy miranda methods over before we allocate the new vtable
or else we may have stale miranda gc roots.

(cherry picked from commit 6e80460bdf0aa9bd273d4a4d665d679c651b5f4f)

Bug: 21664466
Change-Id: Ib3e415bb9e7df7abfa18c98fe01f790fa39622dc

Restore DDMS recent allocation tracking's behavior

Make the allocation tracker mark recently allocated objects as roots,
so the allocation records are not swept. Because DDMS needs recent
allocation tracking records even for dead objects. This should fix
the flaky failures for 098-ddmc test, but it cannot pass GC stress
test (OOM). Re-enabled 098-ddmc for other tests.

There should be an option to not mark them as roots, when user only
needs HPROF dump with traces but not DDMS's recent allocation tracking.
Probably need to add a new JNI API function for this option.

There could be another way to keep a second list of recent allocation
records and maintain a type cache for them, so not to make the objects
roots. But it's more complex, and not sure which is better.

Also reduce memory usage for AllocRecordStackTrace objects, and change
default stack depth to 16. Rename the property that controls the stack
depth to "debug.allocTracker.maxStackDepth" so developer can change it.

Bug:20037135
Change-Id: Ic6b9ae87bdcda558be6f14ded8057e763439881c

am 336b0b1b: Merge "base: Fix an infinite loop in HashSet::Insert"

* commit '336b0b1baaf2b3eccb26ccf1b7d502c90c3f33fd':
  base: Fix an infinite loop in HashSet::Insert

Merge "base: Fix an infinite loop in HashSet::Insert"

base: Fix an infinite loop in HashSet::Insert

Also adds a test for HashSet::ShrinkToMaximumLoad

(This bug was only reachable when using ShrinkToMaximumLoad, which is not
called from anywhere other than the new test)

Change-Id: I5276b4b3f4ecf6090bb545ddd1752758b11609dd

am ac854d27: Merge "New experimental GC stress mode"

* commit 'ac854d27aeafb3dca04dfc5c716d0436f1d5ffc1':
  New experimental GC stress mode

Merge "New experimental GC stress mode"

am 5aa39bb7: Merge "Change GC to allow strings to use large object space."

* commit '5aa39bb7f93cf2e8f73d06980b588519825c0ebe':
  Change GC to allow strings to use large object space.

New experimental GC stress mode

Tries to do a GC for every unique call stack (up to 16 frames).
The goal is to catch moving GC bugs and lock violations without being
rediculously slow. Some tests fail on 64 bits, 32 bit host doesn't
work. N5 is booting.

Added runtime -Xgc options: gcstress and nogcstress.

Bug: 21664466

Change-Id: Icb8e420f2048e8ee83bcca7937563166a2638f5c

am d423552a: Merge "Make large string use large object space."

* commit 'd423552afd00bac3cdb523b9f8681f3f58163564':
  Make large string use large object space.

Merge "Change GC to allow strings to use large object space."

Change GC to allow strings to use large object space.

Bug: 21209641

(cherry-picked from commit 37fd5275c665b483f2d990634a500c4e79aae87f)

Change-Id: I043f59eeb601a95c7d01b10a844c2237167ee862

Merge "Make large string use large object space."

Make large string use large object space.

Bug: 21209641

(cherry-picked from commit 38fafa18f0fcf610f6314a0fc4165e04e6e5d017)

Change-Id: If6d708307ba1ecdde6bf856d31dceac9ee782fb7

am ec3a4e7c: Merge "runtime: Partially implement box-lambda and unbox-lambda experimental opcodes"

* commit 'ec3a4e7cdc4f268b40d923227c125429f4ee4884':
  runtime: Partially implement box-lambda and unbox-lambda experimental opcodes

am 11dc78c8: Merge "ART: Allow checking RETURN_VOID_NO_BARRIER on unresolved classes"

* commit '11dc78c8d36b4a65f4d955bc4303cc8279f1151e':
  ART: Allow checking RETURN_VOID_NO_BARRIER on unresolved classes

Merge "runtime: Partially implement box-lambda and unbox-lambda experimental opcodes"

Merge "ART: Allow checking RETURN_VOID_NO_BARRIER on unresolved classes"

runtime: Partially implement box-lambda and unbox-lambda experimental opcodes

These opcodes are not yet fully specified, and *will* change before they become shippable.
Do not write production code against experimental opcodes.

--

Implement partial interpreter support for new dex instructions box/unbox-lambda.
* box-lambda will take a closure and convert it into an Object
* unbox-lambda will take an Object and convert it to a closure

(Currently does not implement object identity or variable capture).

All new opcodes are disabled by default, use runtime option -Xexperimental-lambdas to enable them.

Change-Id: I3c15ccf8a26ccecd1d35808a8c1b4149220f6019

am f7012291: Merge "Run an empty checkpoint before marking ends in the CC collector."

* commit 'f7012291ed9c984e2b50d83d7a2d5758cd64c99a':
  Run an empty checkpoint before marking ends in the CC collector.

Merge "Run an empty checkpoint before marking ends in the CC collector."

ART: Allow checking RETURN_VOID_NO_BARRIER on unresolved classes

We must check that all fields are non-final, even for unresolved
classes. Fall back to the dex file for this information.

Bug: 22005084
Bug: https://code.google.com/p/android/issues/detail?id=177694
Change-Id: If2126d1be5dccadbbedcdccabde7398688d6e5ca

am ae393c3b: Merge "ART: Avoid recursive abort in barrier"

* commit 'ae393c3b24cfdad97aaded8c0ff1ee7262b46ffe':
  ART: Avoid recursive abort in barrier

Merge "ART: Avoid recursive abort in barrier"

am 59cb5218: Merge "Fix in ReferenceTypePropagation"

* commit '59cb52189d350fa9fb38b767fa9b25d971375833':
  Fix in ReferenceTypePropagation

ART: Avoid recursive abort in barrier

Try to avoid recursive abort in barrier. May lead to a segfault
instead of a nested abort in bad cases.

Change-Id: I0e7976b77c243956dbcf81142bd5df81d5927ce0

Merge "Fix in ReferenceTypePropagation"

am c30e721b: Merge "ART: Don\'t do pre-linked calls to the interpreter bridge."

* commit 'c30e721b5ddcb4ba6835c5b721e1c5095f4a3d7f':
  ART: Don't do pre-linked calls to the interpreter bridge.

Merge "ART: Don't do pre-linked calls to the interpreter bridge."

am f91fb4a6: Merge "ART: Disallow classes that are abstract and final"

* commit 'f91fb4a6a565809dd077ee128d11f30b67e6f2ff':
  ART: Disallow classes that are abstract and final

Merge "ART: Disallow classes that are abstract and final"

ART: Don't do pre-linked calls to the interpreter bridge.

Bug: 21897404
Change-Id: I6cca9d3b436df2eed2b0192607de0faf5cc2ecf7

am 2d655f1c: Merge "Revert "Revert "Use IsAssignableFrom instead of IsSubclass for robustness."""

* commit '2d655f1ce89a714094a4f55ac75edcf9a34c7e24':
  Revert "Revert "Use IsAssignableFrom instead of IsSubclass for robustness.""

Merge "Revert "Revert "Use IsAssignableFrom instead of IsSubclass for robustness."""

Revert "Revert "Use IsAssignableFrom instead of IsSubclass for robustness.""

Don't use IsAssignableFrom, but check beforehand if the referrer is an
interface. Otherwise, we are being too aggressive on removing clinit
checks on interfaces (being a subclass doesn't imply the interface
has been initialized).

bug:21870666

This reverts commit 463580ca5a1e75e27ad0207537ffc6252091326a.

Change-Id: I2a13974d28477ee3e5224142cc260cbf46d19aa9

am a63d8f44: Merge "fault_handler : debug output sp in hex and not decimal."

* commit 'a63d8f44740a8406adda6aaff7ed75ef923c085f':
  fault_handler : debug output sp in hex and not decimal.

Merge "fault_handler : debug output sp in hex and not decimal."

fault_handler : debug output sp in hex and not decimal.

Easier to compare against hex values in tombstones and debuggerd
dumps.

Change-Id: I4a4512ea1ce5872fdb6b41e824fa3475dfcfb71d

am ae370391: Merge "Use compiling class instead of outer class for access checks."

* commit 'ae370391057b88607b0e451fdea020b257684b91':
  Use compiling class instead of outer class for access checks.

am a948b1a0: Merge "Opt compiler: Add disassembly to the \'.cfg\' output."

* commit 'a948b1a0a917b9fd4bbd10a5ba4823594b78373d':
  Opt compiler: Add disassembly to the '.cfg' output.

Merge "Use compiling class instead of outer class for access checks."

Merge "Opt compiler: Add disassembly to the '.cfg' output."

Opt compiler: Add disassembly to the '.cfg' output.

This is automatically added to the '.cfg' output when using the usual
`--dump-cfg` option.

Change-Id: I864bfc3a8299c042e72e451cc7730ad8271e4deb

am 8ec0e203: Merge "ART: Simplify (Not)Equal bool vs. int to true/false"

* commit '8ec0e20347e13592539a8c0786b1db1735149800':
  ART: Simplify (Not)Equal bool vs. int to true/false

am 0f0be726: Merge "Revert "Use IsAssignableFrom instead of IsSubclass for robustness.""

* commit '0f0be7261c68ebb701677b70d40e373accef4d09':
  Revert "Use IsAssignableFrom instead of IsSubclass for robustness."

Merge "ART: Simplify (Not)Equal bool vs. int to true/false"

ART: Simplify (Not)Equal bool vs. int to true/false

Optimizations on the HGraph may produce comparisons of bool and ints.
Instruction simplifier will simplify these only for 0/1 int constants.
Since the range of bool is known, comparison against all other int
constants can always be determined statically.

Change-Id: I502651b7a08edf71ee0b2589069f00def6aacf66

Merge "Revert "Use IsAssignableFrom instead of IsSubclass for robustness.""

Revert "Use IsAssignableFrom instead of IsSubclass for robustness."

Not correct for Quick.

Bug:21870666

This reverts commit af268bbc4fbd0b9847acf1f7b79c038806118467.

Change-Id: Id44849cc877a42495619fd71bfd2db75159b6868

Use compiling class instead of outer class for access checks.

bug:21869678
Change-Id: Ica6ecc759adc76950dcfe9d4d859dd593a742c62

am e85647ed: Merge "Use IsAssignableFrom instead of IsSubclass for robustness."

* commit 'e85647ed00d07169193cc07ffa09e3b70ce38f59':
  Use IsAssignableFrom instead of IsSubclass for robustness.

am 8cf7cbf7: Merge "Fix wrong DCHECK in bounds check elimination."

* commit '8cf7cbf7854199cfd5001fa9503c78c3b15f0b58':
  Fix wrong DCHECK in bounds check elimination.

Merge "Use IsAssignableFrom instead of IsSubclass for robustness."

Merge "Fix wrong DCHECK in bounds check elimination."

am 7aa55725: Merge "Fix String Change baseline compiler errors."

* commit '7aa557256a87a8a7ad4be7d9ff90aee54a99124c':
  Fix String Change baseline compiler errors.

Merge "Fix String Change baseline compiler errors."

Fix String Change baseline compiler errors.

The String Change adds multiple move results if the uninitialized string
is in multiple registers. This adds StoreLocals on the same instruction,
which isn't allowed. Now, a LoadLocal is added for each extra move
needed.

Change-Id: I49f676cb85de051e737e0653138cbb947a73e48b

am 74f774f0: Merge "Make dump-oat-target work with second architecture"

* commit '74f774f0d6389c432122ca262c6705118f5ebd6e':
  Make dump-oat-target work with second architecture

Merge "Make dump-oat-target work with second architecture"

Fix wrong DCHECK in bounds check elimination.

The lower range of an array length instruction can
be changed by other instructions than HBoundsCheck,
like HNewArray.

bug:21862741
Change-Id: Idbe50ac114287ea6d852fb6fe9f9e2d440d18af5

am 883ac209: Merge "Move LD_USE_LOAD_BIAS to run-test-jar to reduce log verbosity."

* commit '883ac209066d61404627a8820770755d60068a2b':
  Move LD_USE_LOAD_BIAS to run-test-jar to reduce log verbosity.

Fix in ReferenceTypePropagation

Take into account that we might not be able to resolve the method when
we visit the invoke, or the field when we visit a field-access.

Bug: 21914925
Change-Id: Ie4bcfeddaa78780d8cfe28e77d1135ee219a15ad

ART: Disallow classes that are abstract and final

Make the verifier fail such classes.

Bug: 21873151
Change-Id: I217f3d71f44bccdcee7ca830e092c807928bed39

Merge "Move LD_USE_LOAD_BIAS to run-test-jar to reduce log verbosity."

Move LD_USE_LOAD_BIAS to run-test-jar to reduce log verbosity.

LD_USE_LOAD_BIAS is needed so that we can use dlopen.
Without it, we will fallback to our own loader and
dump all mmap for diagnostics which makes the tests verbose.

Change-Id: Ib12e9834b3454ffd726642859550545826ce38bf

Make dump-oat-target work with second architecture

Bug: 19909409
Change-Id: Iff21c5d3ce0e6b057a677da6726ee42af64463e7

am 1e73a95d: Merge "Use signed encoding when using relative CFI addresses."

* commit '1e73a95d74848020fab512a95a7f9a7ada72497b':
  Use signed encoding when using relative CFI addresses.

Merge "Use signed encoding when using relative CFI addresses."

am a60bd747: Merge "ART: Check for expected args for instance methods"

* commit 'a60bd74737fcd1b3737bc523741efa1097bbd782':
  ART: Check for expected args for instance methods

Use signed encoding when using relative CFI addresses.

This is required for gdb to work.
libunwind works with either encoding.

Bug: 21924613
Change-Id: I4e4f1cf9c65d48fa885a5993eeeed0253a3f2579

Merge "ART: Check for expected args for instance methods"

ART: Check for expected args for instance methods

Don't assume that the receiver is expected. This was writing out
of bounds (or triggering a DCHECK).

Bug: 21817284
Bug: 21872240
Bug: https://code.google.com/p/android/issues/detail?id=176571
Bug: https://code.google.com/p/android/issues/detail?id=176572
Change-Id: I7cad2c1dfc39feb0b1b4660deeb43afdcb844ba3

am ba7837b2: Merge "ART: Check var-arg count earlier in method verifier"

* commit 'ba7837b2308aeaaa613383297737488af558aed7':
  ART: Check var-arg count earlier in method verifier

Merge "ART: Check var-arg count earlier in method verifier"

Run an empty checkpoint before marking ends in the CC collector.

To avoid a race condition between the end of marking and a mutator
still in the middle of a read barrier.

Bug: 12687968
Bug: 21564728
Change-Id: I4962d895b4df89d2bcde97cbabdb98a14a19dd6b