am 0a487e74: (-s ours) am 122349a7: Merge "Fix StringChange for optimizing compiler." into mnc-dev

* commit '0a487e74549876f597d2dad41bbf30ef576df440':
  Fix StringChange for optimizing compiler.

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

* commit '759662ef35b45d9b260dbc896c91048574e6bffe':
  Be careful with predecessor/successor index.

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

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

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 e15aa290: am dbc0345e: Merge "Opt compiler: Add a description to MIP64 slow paths."

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

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

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

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

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

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 6cec8674: am 71af6f7a: Merge "Opt compiler: Add a description to slow paths."

* commit '6cec86742a283d4c455583ce5578ebb0d85aa82d':
  Opt compiler: Add a description to slow paths.

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: 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

am 2812995b: am 6e82168b: Fix moving GC bug in DoFilledNewArray

* commit '2812995bd1ded58979e5cf982b4c5db66ffd4910':
  Fix moving GC bug in DoFilledNewArray

am e2e271fd: am dfc5db6a: Fix moving GC bugs in proxy stub for X86/X86_64

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

am c82e3578: am bf1fa2cc: Follow up on CL 151605

* commit 'c82e3578544faffcd0de344b4f279b199386ea2d':
  Follow up on CL 151605

Opt compiler: Add a description to slow paths.

Change-Id: I22160d90de3fe0ab3e6a2acc440bda8daa00e0f0

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

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

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 be91f759: am d8d7b90c: Merge "Do not expect 0 or 1 only when comparing a boolean."

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

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

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

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 62f5b393: am be59b852: Merge "Make Main public to pass tests on all configurations."

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

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

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

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

* commit '20ede5b878df4466a4b3ad5ee00911a41b3ab3e6':
  Use the right bug number. Spotted by vmarko.

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

am 122349a7: Merge "Fix StringChange for optimizing compiler." into mnc-dev

* commit '122349a7672da6636ca94aded7f2d3642f9962e3':
  Fix StringChange for optimizing compiler.

Merge "Fix StringChange for optimizing compiler." into mnc-dev

Fix StringChange for optimizing compiler.

Uses optimizing compiler more and fixes x86_64 invoke codegen.

Bug: 21902634
Change-Id: Ia2a87d013c4746b107014a04a22a0a37269cfdb2

am 6e82168b: Fix moving GC bug in DoFilledNewArray

* commit '6e82168bcb339b162f8fac1034a1153eec421eae':
  Fix moving GC bug in DoFilledNewArray

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

* commit '4acbbe7c347af68c2216304cb02a8947a7f49c2b':
  Fix moving GC bug in DoFilledNewArray

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

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

Fix moving GC bug in DoFilledNewArray

Previously we read from componentClass after allocating the array.

Bug: 21783443

(cherry picked from commit 52ea33b10370d60d4ce877aec529626537b7813b)

Change-Id: I5283982edab479434e27416509e1436b4176fe01

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

resolved conflicts for merge of 23b4feeb to mnc-dev-plus-aosp

Change-Id: I2fc26211d276d7af21f254802c215aaf3deecd53

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

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

am dfc5db6a: Fix moving GC bugs in proxy stub for X86/X86_64

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

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

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

Fix moving GC bugs in proxy stub for X86/X86_64

Needed to restore the refs.

(cherry picked from commit 9346ff0cfad6344d0bf4eaa69362dbe1987ac054)

Bug: 21907554
Change-Id: I562906dff07dcaa78dfb39646ba9ab35a5f56c6c

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 bf1fa2cc: Follow up on CL 151605

* commit 'bf1fa2ccb5e7409910b99dc46b616e44c66ade68':
  Follow up on CL 151605

am ef484d44: Fix nested deoptimization.

* commit 'ef484d442a3dcae2cd1842c5be0623f5cf71e4ab':
  Fix nested deoptimization.

Follow up on CL 151605

- Fixes return type of StackedShadowFrameRecord::GetType
- Makes StackedShadowFrameType an enum class (scoped enum)
- Moves DeoptimizationReturnValueRecord and StackedShadowFrameRecord
  to thread.cc file and use forward declaration in thread.h header
- Fixes tools/generate-operator-out.py for scoped enum classes.

Bug: 20845490

(cherry picked from commit f795869da0a1fa006fdcdacd8afb6149a63fc1a7)

Change-Id: I6b67e288b1db563699161e58ec2e2330d42dd8f5

Fix nested deoptimization.

Handle nested deoptimization cases. Create a stacked shadow frame
records to keep track of deoptimization shadow frames. Shadow frames
under construction can be tracked in the same stack.

Bug: 20845490

(cherry picked from commit 1f2d3ba6af52cf6f566deb38b7e07735c9a08fb6)

Change-Id: I768285792c29e7c3cfcd21e7a2600802506024d8

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

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

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

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

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"

resolved conflicts for merge of 26d53e38 to mnc-dev-plus-aosp

am 07c6f5a3: Quick: Don\'t expect move-exception in every catch block.

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

Fix moving GC bugs in proxy stub for X86/X86_64

Needed to restore the refs.

Bug: 21907554
Change-Id: I562906dff07dcaa78dfb39646ba9ab35a5f56c6c

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

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

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.

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

The dalvik bytecode doesn't require a move-exception in
a catch handler that ignores the exception.

Bug: 21873167

(cherry picked from commit 2d1a0a408fd148f7b2a2d670e6942ec3d920f875)

Change-Id: Ia00e5791fabdd1b88e9a63b741600ed63d918703

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

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

The dalvik bytecode doesn't require a move-exception in
a catch handler that ignores the exception.

Bug: 21873167
Change-Id: I3b49218a8c7ff021141387bd929bb2ae798f8509

am 00efb232: am e71eb1d7: Fix some java_lang_Class related moving GC bugs

* commit '00efb23292a689a4d869cc5cb7f001293f801392':
  Fix some java_lang_Class related moving GC bugs

am ea946dab: am 3d266a96: Merge "Don\'t check the return type in the builder."

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

am e71eb1d7: Fix some java_lang_Class related moving GC bugs

* commit 'e71eb1d7cdc59127b6fcf101230376ce7c892a89':
  Fix some java_lang_Class related moving GC bugs

am 3d266a96: Merge "Don\'t check the return type in the builder."

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

Fix some java_lang_Class related moving GC bugs

There was some missing handles around mirror::Class*.

(cherry picked from commit 05b7226787f1470ad93f6f632fed60f70bc8631e

Bug: 21898408
Change-Id: Icb754074dfb469473101d20d6873a5bc3274abc5

Merge "Don't check the return type in the builder."

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

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 invoke/create-lambda, and a
new opcode format 25x.

* Does not verify, in fact verification will soft fail when we see those opcodes.
* Compilers will punt to interpreter since they don't support new opcodes.
* As there is no way to capture/liberate variables yet, the "closure" is just
  an ArtMethod for the time being.

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

For example:
  dalvikvm ... -Xexperimental-lambdas ...
  dex2oat --runtime-arg -Xexperimental-lambdas ...

Change-Id: I6c996ca32a9b54ec45ec21d7a959b84dfb8a24eb

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

Change-Id: I6ef7be233d1cd69dddcc4499e4e4fad54753099f

am 4288cc07: am ee9bc65d: Merge "ART: Allow PackedSwitch instructions with zero targets" into mnc-dev

* commit '4288cc074fe693798cab5810953b027583c11fdc':
  ART: Allow PackedSwitch instructions with zero targets

am d5fdbc21: am 6ecc2967: Quick: Fix marking unused registers in DCE.

* commit 'd5fdbc21d66a35eb2d6d102ffde03cd5a5ff1900':
  Quick: Fix marking unused registers in DCE.

am c7a3379c: am 822c00d0: Merge "Run a simplification pass before code generation."

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

am 79f9a072: am 01b6ec8c: Merge "Don\'t special case HCurrentMethod in DCE."

* commit '79f9a0722b80493cc4c7b659be14bc2442f9c078':
  Don't special case HCurrentMethod in DCE.

am 742923b6: am 92fe57fe: Merge "Quick: Fix marking unused registers in DCE."

* commit '742923b60fbc7897e97b97dea36f179a9a1b2fa8':
  Quick: Fix marking unused registers in DCE.

am ee9bc65d: Merge "ART: Allow PackedSwitch instructions with zero targets" into mnc-dev

* commit 'ee9bc65d3069c7c5c3f66dd9459915df54c3482f':
  ART: Allow PackedSwitch instructions with zero targets

am 6ecc2967: Quick: Fix marking unused registers in DCE.

* commit '6ecc2967918e7d7bed299d4dbf48ac63effc1088':
  Quick: Fix marking unused registers in DCE.

Merge "ART: Allow PackedSwitch instructions with zero targets" into mnc-dev

am 822c00d0: Merge "Run a simplification pass before code generation."

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

am 01b6ec8c: Merge "Don\'t special case HCurrentMethod in DCE."

* commit '01b6ec8c982400fe415533296ab0b563971c239d':
  Don't special case HCurrentMethod in DCE.

am 92fe57fe: Merge "Quick: Fix marking unused registers in DCE."

* commit '92fe57fe6e3512616f71b2b2183344c56d6753e6':
  Quick: Fix marking unused registers in DCE.

ART: Allow PackedSwitch instructions with zero targets

Optimizing and the interpreter wrongly assumed that a PackedSwitch
always has at least one target. This patch removes the corresponding
DCHECKs and adds a regression test case.

This is a resubmission of CL I32b7033ed38de6f1d1a6ee5d5bf12f3a47c9b37e

Bug: 21863783
Change-Id: I04e6e124bdd16591ba27c79490e6ce183c36b691
(cherry picked from commit 2ef645ba50544b879a82ea30e606f18c9af98917)

Quick: Fix marking unused registers in DCE.

Bug: 21702651

(cherry picked from commit 511d408909b7fe9be3c95e032cc7f426f4a01e6e)

Change-Id: I059e0b841543334cd08a27431e0ea853b42e4f35

Merge "Run a simplification pass before code generation."