resolved conflicts for merge of 6425b37b to mnc-dev-plus-aosp

Change-Id: I5bc20d3bf1db1e190ff9d21abc81bdf5dbeffa9f

am 15e0965d: am bbcec62c: Merge "Don\'t check the return type in the builder." into mnc-dev

* commit '15e0965d3bd6f0ceca6334c7b3e6ae5dd14f3a2a':
  Don't check the return type in the builder.

am c7cf0ee0: am 39eb41f6: Merge "Do not expect 0 or 1 only when comparing a boolean." into mnc-dev

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

am 1e3055a2: am 39d40afb: Verifier: check an aput, even if we know it will fail.

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

Merge "resolved conflicts for merge of 18218b8b to mnc-dev-plus-aosp" into mnc-dev-plus-aosp

resolved conflicts for merge of 18218b8b to mnc-dev-plus-aosp

Change-Id: I30ae4f273eda2d4ecf2fbc0147012a9fa750c9d5

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

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

am 1e256bf2: Be careful with predecessor/successor index.

* commit '1e256bf257e8d97df9b2178ae8658b731ca2d662':
  Be careful with predecessor/successor index.

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.

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

(cherry picked from commit 8b20f88b0a8d1b374dd5eaae289d19734c77b8f8)

Change-Id: I2581b5a50942290da96cd9ec876f6f2573e0a6c4

Merge "resolved conflicts for merge of 15621f64 to mnc-dev-plus-aosp" into mnc-dev-plus-aosp

am bbcec62c: Merge "Don\'t check the return type in the builder." into mnc-dev

* commit 'bbcec62c0484fbfb82ee2c317e8afa478a63027b':
  Don't check the return type in the builder.

resolved conflicts for merge of 15621f64 to mnc-dev-plus-aosp

Change-Id: I1fc73beebbf842f494b2b414dee2e5d8a2fa5794

am a4d847f4: am c9345cc2: Merge "Fix wrong DCHECK in bounds check elimination." into mnc-dev

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

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

Merge "resolved conflicts for merge of 49b8352b to mnc-dev-plus-aosp" into mnc-dev-plus-aosp

resolved conflicts for merge of 49b8352b to mnc-dev-plus-aosp

Change-Id: I90e06186b7e463290a766a96730c0a7702306f67

Merge "Don't check the return type in the builder." into mnc-dev

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

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

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

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

Don't check the return type in the builder.

The verifier will not complain about dead code. For the compiler,
it is benign.

bug:21863767

(cherry picked from commit 69505f86a1a653bf86c1f2b423b3e0605137d5d1)

Change-Id: I2bc7888b39c91862c6dade130eccdf92f19b3f6f

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.

am 39eb41f6: Merge "Do not expect 0 or 1 only when comparing a boolean." into mnc-dev

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

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

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

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

Merge "Do not expect 0 or 1 only when comparing a boolean." into mnc-dev

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

bug:21866529

(cherry picked from commit 3c4ab80c102ff1bfc0e74d4abddbf5454bf4008d)

Change-Id: Ibdc0d4a9730bfc6e7307282276f084dae5ac55c1

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 a04e7b7c: Run a simplification pass before code generation.

* commit 'a04e7b7ceae01a7ceda41419e4c639a43e18361e':
  Run a simplification pass before code generation.

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

bug:21867457

(cherry picked from commit 66389fbeb41c981648bb2e1e7e2f43089f095591)

Change-Id: Id01edeb871f6bc59518eb341e5b76f021cfdfe06

Run a simplification pass before code generation.

The code generators assume things that only the instruction
simplier ensures. So it has to be run last in case previous
optimiziations broke those assumptions.

bug:21865464

(cherry picked from commit b2bdfce7f805b00668a2521b1c939a0aafb2be49)

Change-Id: Ibf4384a911e400eb7586bbf6b4edd6351034cbd8

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

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

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

* commit '4b7308163995d8592612a54b859f3006b2bd4648':
  Revert "Use IsAssignableFrom instead of IsSubclass for robustness."

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

am ed286710: am ff82263e: Fix String Change baseline compiler errors.

* commit 'ed286710a5d6b0316ce632ed861daff06f958f1c':
  Fix String Change baseline compiler errors.

am 096f357c: Merge "Bailout from compilation if an invoke is malformed." into mnc-dev

* commit '096f357c5dda663c6fbb58bd0154c091aec51f63':
  Bailout from compilation if an invoke is malformed.

am c9345cc2: Merge "Fix wrong DCHECK in bounds check elimination." into mnc-dev

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

am 25fde612: Fix a bug in optimizing when the null constant has been DCE.

* commit '25fde612b0df01a086cd4c801b7bd3a10e93a0e9':
  Fix a bug in optimizing when the null constant has been DCE.

am ff82263e: Fix String Change baseline compiler errors.

* commit 'ff82263e2b96ad099c56c19b91c2286baaf82fa7':
  Fix String Change baseline compiler errors.

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

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

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

* commit '5a7844734935a9c58859230afbe402f8db1e3d71':
  Fix wrong DCHECK in bounds check elimination.

Merge "Bailout from compilation if an invoke is malformed." into mnc-dev

Bailout from compilation if an invoke is malformed.

Because the verifier does not check trivially dead instructions,
the compilers must prepare for bogus instructions. This change
fixes the case the arguments for an invoke do not match the
formal parameters.

bug:21865459

(cherry picked from commit 2e33525bd4eb892246b4c244c6d4ebf6c6d07501)

Change-Id: I392f86eafefde28263fe35a31f17b398ff8dfc24

Merge "Fix wrong DCHECK in bounds check elimination." into mnc-dev

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

(cherry picked from commit 8d82a0c2b2b12f259ccb357d3b1e699c68ad0400)

Change-Id: I1bb1a4f4c6673509dd3fb5184c32992bed876250

Use compiling class instead of outer class for access checks.

bug:21869678
Change-Id: Ica6ecc759adc76950dcfe9d4d859dd593a742c62

Fix a bug in optimizing when the null constant has been DCE.

If it has been DCE, we should create a new one, instead of
using the old one.

Also move the first DCE to a place where it could actually
be useful.

bug:21870788

(cherry picked from commit 18e6873c469b48aaed22148451523479eece98e3)

Change-Id: I3b3ab2dafe8ce5fb60868fd1a6ef0eeefe666e0c

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.

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.

bug:21902634

(cherry picked from commit aa919207d2fb63af11d72d3b7cdbc435769565af)

Change-Id: I057d14cdac437d06eec20caaddd430c304e58196

Merge "Use IsAssignableFrom instead of IsSubclass for robustness."

Merge "Fix wrong DCHECK in bounds check elimination."

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

* commit 'b4688335b7275dcde8582ba44f0bf1ab4437cff7':
  Fix String Change baseline compiler errors.

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 6be9c366: am 74f774f0: Merge "Make dump-oat-target work with second architecture"

* commit '6be9c366b59f7371334c7e88c611f99907aa3689':
  Make dump-oat-target work with second architecture

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 44beb33f: am 883ac209: Merge "Move LD_USE_LOAD_BIAS to run-test-jar to reduce log verbosity."

* commit '44beb33fac4e75574b8d66a4702d20c30983593c':
  Move LD_USE_LOAD_BIAS to run-test-jar to reduce log verbosity.

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.

am 7060f281: am 7257ece4: Use signed encoding when using relative CFI addresses.

* commit '7060f2812b6eca80968a7480698fe943c276e90c':
  Use signed encoding when using relative CFI addresses.

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

am 7257ece4: Use signed encoding when using relative CFI addresses.

* commit '7257ece418469b284ae3cf1d9ba6617a62bdfc4f':
  Use signed encoding when using relative CFI addresses.

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 40ae4519: am 1e73a95d: Merge "Use signed encoding when using relative CFI addresses."

* commit '40ae45194aa92f2fea952e1aaa407419451f31a1':
  Use signed encoding when using relative CFI addresses.

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

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

am 73ad9401: am 673b4302: ART: Check for expected args for instance methods

* commit '73ad9401d93bdbfe51b00e51361e4ecd2047aac5':
  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.

(cherry picked from commit 17065880693d1b15ffeb60b9955a2d092839977f)

Bug: 21924613
Change-Id: I4e4f1cf9c65d48fa885a5993eeeed0253a3f2579

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

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

* commit '46cbfe60a615bc1ea8e8580b2111477211edb1cb':
  ART: Check for expected args for instance methods

am 673b4302: ART: Check for expected args for instance methods

* commit '673b4302edf6d1604e69a1427eea5324016bbab2':
  ART: Check for expected args for instance methods

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

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

(cherry picked from commit ef0b1a1dbc99fe7c92f9598cbfc164763c1b66d7)

Change-Id: I7cad2c1dfc39feb0b1b4660deeb43afdcb844ba3

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 3c0ea8b9: am cbf5d896: ART: Check var-arg count earlier in method verifier

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

am cbf5d896: ART: Check var-arg count earlier in method verifier

* commit 'cbf5d89665cfb48778b2946794fbea43067ff04a':
  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

(cherry picked from commit 29498a23bcfe47a7134552aacad5524ecb484a49)

Change-Id: I1ff93502ab84c5967ad54d1b50dba517da791637

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

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

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 d7f7bf7f: am 1bd841a2: Merge "ART: Fix BCE lint issue" into mnc-dev

* commit 'd7f7bf7fea3b09bd0549bdf99e98b84a69157585':
  ART: Fix BCE lint issue

am 1bd841a2: Merge "ART: Fix BCE lint issue" into mnc-dev

* commit '1bd841a26a0810decbd3cd9dcc3c0dca5773dc2b':
  ART: Fix BCE lint issue

Merge "ART: Fix BCE lint issue" into mnc-dev

ART: Fix BCE lint issue

Bug: 21034044

(cherry picked from commit 45d68f138a31a3ff9b45cda313f0ba27f1431f26)

Change-Id: I7f382a3124955eff5c0b96ca39ec67fb658fa3d0

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

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

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

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