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"

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.

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"

ART: Check var-arg count earlier in method verifier

Check the count in vararg instructions before filling the temp
array. Avoids a DCHECK.

Bug: 21869663
Bug: 20170976

Change-Id: I1ff93502ab84c5967ad54d1b50dba517da791637

am d1a06338: Merge "Fix GC map generation edge case"

* commit 'd1a06338b9aa818b730ef1d46549a09af8f35c5b':
  Fix GC map generation edge case

Merge "Fix GC map generation edge case"

Fix GC map generation edge case

Previously we checked the number of bits, but then rounded up to a
byte boundary. If ref_bitmap_bits was 65535, it would pass the check
but not actually fit in the 13 bits for ref_bitmap_bytes since we
rounded up to 8192 bytes.

(cherry picked from commit 1626b919005df42ccc90ebe68fc57b7c56d44029)

Regression test infeasible due to smali limitations.

Bug: 21888453
Change-Id: If4b769431f27ecca14fe7852d017bcde01b3e146

am d5ff8c58: Merge "Fix case where block has no predecessor for StringChange."

* commit 'd5ff8c584b230da350f4ebfffec1d245c09bf26b':
  Fix case where block has no predecessor for StringChange.

Merge "Fix case where block has no predecessor for StringChange."

Fix case where block has no predecessor for StringChange.

Removes part that checks for throwing half of instruction. It's no
longer necessary. Also adds regression test.

Bug: 21902684

(cherry-picked from commit 3e91a44bc9063f7f69b5415e3cf162991f73283f)

Change-Id: I8d7308ff227816fb475341c2b74ed84d5695a2f5

am e102f193: Merge "Fix StringChange for optimizing compiler."

* commit 'e102f1937ca14df87594469430d2e6e0ebe0f357':
  Fix StringChange for optimizing compiler.

Use IsAssignableFrom instead of IsSubclass for robustness.

We can get interface methods as referrer. Instead of crashing,
just use IsAssignableFrom.

bug:21870666

Change-Id: I7978e36fb981d44611635fbaf3899138bae60672

Merge "Fix StringChange for optimizing compiler."

am e4bb626f: Merge "Quick: Handle total high/low register overlap on arm/mips."

* commit 'e4bb626ff09e409c9482e515716de7cd595ea466':
  Quick: Handle total high/low register overlap on arm/mips.

Merge "Quick: Handle total high/low register overlap on arm/mips."

am 87a135e8: Merge "Be careful with predecessor/successor index."

* commit '87a135e8699119c9ef99b0221acf481d7d6501d4':
  Be careful with predecessor/successor index.

am ff3fd957: Merge "Quick: Fix optimizations for empty if blocks."

* commit 'ff3fd957020f0e5cf5cc279f4bbbca3545ce8745':
  Quick: Fix optimizations for empty if blocks.

Merge "Be careful with predecessor/successor index."

Be careful with predecessor/successor index.

When we simplify the CFG, we must preserve things that were already
simplified. For example, the index in the predecessor list or
successor list of a block must be preserved for ensuring the
first block is a loop pre header.

bug:21867463

Change-Id: Ic3fcb3eb2c3fb109d8a57ee2a6b6d4d65fdb9410

Merge "Quick: Fix optimizations for empty if blocks."

am dbc0345e: Merge "Opt compiler: Add a description to MIP64 slow paths."

* commit 'dbc0345e7a066726fda6dbfe47592963f4602c56':
  Opt compiler: Add a description to MIP64 slow paths.

Merge "Opt compiler: Add a description to MIP64 slow paths."

Opt compiler: Add a description to MIP64 slow paths.

Change-Id: I773a543cd4f146d64819cc347e57e6472dea0b38

am 7ed9cec8: Merge "MIPS: Initial version of optimizing compiler for MIPS64R6."

* commit '7ed9cec8bc302a0b4fe5a127b58bfddd702a70e8':
  MIPS: Initial version of optimizing compiler for MIPS64R6.

Merge "MIPS: Initial version of optimizing compiler for MIPS64R6."

MIPS: Initial version of optimizing compiler for MIPS64R6.

Bug: 21555893
Change-Id: I874dc356eee6ab061a32f8f3df5f8ac3a4ab7dcf
Signed-off-by: Alexey Frunze <Alexey.Frunze@imgtec.com>
Signed-off-by: Douglas Leung <douglas.leung@imgtec.com>

am 71af6f7a: Merge "Opt compiler: Add a description to slow paths."

* commit '71af6f7aab7491665fe2d6beb9af4c251a99ad56':
  Opt compiler: Add a description to slow paths.

Merge "Opt compiler: Add a description to slow paths."

Quick: Handle total high/low register overlap on arm/mips.

OpRegCopyWide() in arm and mips backends didn't handle the
total register overlap when the registers holding the source
and destination pairs are the same but in reverse order.

Bug: 21897012
Change-Id: Ia1f0005cbf5cc1c2d30899575dea21b3efa20ae6

Quick: Fix optimizations for empty if blocks.

If a block ending with if-eqz or if-nez has the same "taken"
and "fallthrough", we cannot assume that the value has been
checked against zero in one of the succesors. This affects
the null check elimination pass as well as GVN. Refactor all
those checks to a single function in BasicBlock and check
that the "taken" and "falthrough" are different when needed.

Bug: 21614284
Change-Id: I8c6ac23e96cdaf5984786a555ebbd28110f095cb

Opt compiler: Add a description to slow paths.

Change-Id: I22160d90de3fe0ab3e6a2acc440bda8daa00e0f0

am 97d207c3: Merge "Use compiling class instead of referrer for access checks."

* commit '97d207c3beba0a4ed6402e882c5909c5ac5bde6d':
  Use compiling class instead of referrer for access checks.

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

Use compiling class instead of referrer for access checks.

bug:21869670
Change-Id: Ief329247e90202835e27fa38803e02dd2e2ae8ca

am d8d7b90c: Merge "Do not expect 0 or 1 only when comparing a boolean."

* commit 'd8d7b90ca57e8e3762951bb419b6f2645bb414c5':
  Do not expect 0 or 1 only when comparing a boolean.

am e4387574: Merge "Verifier: check an aput, even if we know it will fail."

* commit 'e43875749fbc674fbad23ba11af64f25ab2add0d':
  Verifier: check an aput, even if we know it will fail.

Merge "Do not expect 0 or 1 only when comparing a boolean."

Merge "Verifier: check an aput, even if we know it will fail."

Do not expect 0 or 1 only when comparing a boolean.

bug:21866529
Change-Id: I81ffba609a357010bd86073eb979024fc668ed20

am be59b852: Merge "Make Main public to pass tests on all configurations."

* commit 'be59b85251f085359d40bb89b1aa7e037c8ac0ef':
  Make Main public to pass tests on all configurations.

Merge "Make Main public to pass tests on all configurations."

am 511b7cb2: Merge "Use the right bug number. Spotted by vmarko."

* commit '511b7cb2c2c1f276875ff2762bbc97337d824a63':
  Use the right bug number. Spotted by vmarko.

Merge "Use the right bug number. Spotted by vmarko."

Make Main public to pass tests on all configurations.

There's a mismatch between compile-time and runtime access checks.
This doens't fix it but keep us green.

bug:21923303

Change-Id: I1d4a80557f9d8ca9d8bef18a8c23230ed46d4a4c

Verifier: check an aput, even if we know it will fail.

bug:21867457
Change-Id: I01e333d858995d0e1e083a50cf8d460a86381f2c

Use the right bug number. Spotted by vmarko.

bug:21863767

Change-Id: I203072188c0c97b2d6709e1a59555b45fc5a0ab2

Fix StringChange for optimizing compiler.

Uses optimizing compiler more and fixes x86_64 invoke codegen.

Bug: 21902634

(cherry-picked from commit e0a9a53ec4b4ccbf9b1d67957fb99a45b469ccc2)

Change-Id: I56881889bee7092b8401b090af1c0f1004c11667

am f12ffdb5: Merge "Fix moving GC bug in DoFilledNewArray"

* commit 'f12ffdb549f1947125e3f56694cb479a619f02bf':
  Fix moving GC bug in DoFilledNewArray

Merge "Fix moving GC bug in DoFilledNewArray"

Fix moving GC bug in DoFilledNewArray

Previously we read from componentClass after allocating the array.

Bug: 21783443

Change-Id: I5283982edab479434e27416509e1436b4176fe01

am 542ca97d: Merge "test: Fix lambda test incorrectly globbing together compiler arguments"

* commit '542ca97d3e4266896407b9abd29b36b1c59d798c':
  test: Fix lambda test incorrectly globbing together compiler arguments

Merge "test: Fix lambda test incorrectly globbing together compiler arguments"

test: Fix lambda test incorrectly globbing together compiler arguments

This also fixes test failures when running imageless tests.

Change-Id: I9b0deb836907060160e31c434cf0710507f9665a

am 46d87021: Merge "Fix moving GC bugs in proxy stub for X86/X86_64"

* commit '46d8702167d679dbf87ba9fa8fd0f1b6bbb3ed62':
  Fix moving GC bugs in proxy stub for X86/X86_64

am 247e9774: Merge "interpreter: Add experimental lambda opcodes for invoke/create-lambda"

* commit '247e97746016b49183b52f4c386bedf7cd53e590':
  interpreter: Add experimental lambda opcodes for invoke/create-lambda

Merge "Fix moving GC bugs in proxy stub for X86/X86_64"

Merge "interpreter: Add experimental lambda opcodes for invoke/create-lambda"

Fix moving GC bugs in proxy stub for X86/X86_64

Needed to restore the refs.

Bug: 21907554
Change-Id: I562906dff07dcaa78dfb39646ba9ab35a5f56c6c

am 2b696ab5: Merge "Quick: Don\'t expect move-exception in every catch block."

* commit '2b696ab55e817b583d7b3b3ffd39e48c38db6989':
  Quick: Don't expect move-exception in every catch block.

Merge "Quick: Don't expect move-exception in every catch block."