am 788d6d8b: am 46ff9b61: Merge "ART: Fix reporting initialized classes by CompilerDriver."

* commit '788d6d8b9b83c3bba05f71cc4ce1682647f5822d':
  ART: Fix reporting initialized classes by CompilerDriver.

am 46ff9b61: Merge "ART: Fix reporting initialized classes by CompilerDriver."

* commit '46ff9b6100e66306233534e26c16d3c6738070a9':
  ART: Fix reporting initialized classes by CompilerDriver.

Merge "ART: Fix reporting initialized classes by CompilerDriver."

am 8525b9a2: am d4de42f6: Merge "ART: Remove old DCHECK that trips Baseline"

* commit '8525b9a2fc89447185f9245dacc8f638f92427bf':
  ART: Remove old DCHECK that trips Baseline

am 4bbe339e: am 3b574aad: Merge "Bailout from compilation if an invoke is malformed."

* commit '4bbe339e801046cbd22f71f9e1c7c446e3c25f18':
  Bailout from compilation if an invoke is malformed.

am afcf1055: am 7b295db7: Merge "Fix a bug in optimizing when the null constant has been DCE."

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

am d4de42f6: Merge "ART: Remove old DCHECK that trips Baseline"

* commit 'd4de42f6fd0f00c5b3ca01fed6a26d11e617c3b9':
  ART: Remove old DCHECK that trips Baseline

am 3b574aad: Merge "Bailout from compilation if an invoke is malformed."

* commit '3b574aadd67c5ce261a7383c38044af4cf993393':
  Bailout from compilation if an invoke is malformed.

am 7b295db7: Merge "Fix a bug in optimizing when the null constant has been DCE."

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

Merge "ART: Remove old DCHECK that trips Baseline"

Merge "Bailout from compilation if an invoke is malformed."

ART: Remove old DCHECK that trips Baseline

Codegen verified that the entry block always falls through to the next
block. While this is the case with Optimizing, it doesn't hold for
Baseline but it doesn't need to since codegen handles it fine.

Bug:21913514
Change-Id: I751ef227e6cf103af3e7fc35fca4b01c663385a1

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

Change-Id: Ie9fa9dbaedaad0350a52391252e18c045056454d

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

ART: Fix reporting initialized classes by CompilerDriver.

Fix a bug where the CompilerDriver was erroneously reporting
classes as initialized during AOT compilation when they were
not guaranteed to be initialized at runtime.

This fix prevents the Quick compiler from inlining calls to
static methods in classes that are not guaranteed to be
initialized, so that the runtime performs the initialization
required for correctness.

Bug: 21847756
Change-Id: I6fee5ef9c05c2e5190ab8a9fe61365d5119011c5

am ad0895d3: (-s ours) am f61ab978: (-s ours) Merge "Add support for inlining already sharpened interface calls. DO NOT MERGE ANYWHERE" into mnc-dev

* commit 'ad0895d3681e5696bb592e466fa8ed51224cff44':
  Add support for inlining already sharpened interface calls. DO NOT MERGE ANYWHERE

am 23fad477: Fix reference comparison after redundant phi elimination.

* commit '23fad477d90b826956158aac41d70461c2c7a3ed':
  Fix reference comparison after redundant phi elimination.

am f61ab978: (-s ours) Merge "Add support for inlining already sharpened interface calls. DO NOT MERGE ANYWHERE" into mnc-dev

* commit 'f61ab97807441935cab89ebe6e24279e03d62bfe':
  Add support for inlining already sharpened interface calls. DO NOT MERGE ANYWHERE

Merge "Add support for inlining already sharpened interface calls. DO NOT MERGE ANYWHERE" into mnc-dev

Add support for inlining already sharpened interface calls. DO NOT MERGE ANYWHERE

bug:21867144

Test had to change a bit because it was using some features from
aosp not present in mnc-dev.

(cherry picked from commit 3507105caea0d209c66a95d5ec5d739949105c5d)

Change-Id: If12ea9a5c64e86b5bd3a5fd4a943bff60fe49762

Fix reference comparison after redundant phi elimination.

Otherwise, the graph could remain mistyped.

bug:21776173

(cherry picked from commit 51d400d4ebd41b9fb4d67ac3179f8fb66a090fdd)

Change-Id: Iff36dfa4e79b14a9dd85c37e0fbb9e1080dd0364

am bee9b6ef: am 3c4fb2d1: Merge "Forgot this change in last commit."

* commit 'bee9b6ef11cdcc67c82ef33c165994f3beaee3ff':
  Forgot this change in last commit.

am 3c4fb2d1: Merge "Forgot this change in last commit."

* commit '3c4fb2d1836a81823677191ad91203779427b610':
  Forgot this change in last commit.

Merge "Forgot this change in last commit."

am c6e3ac6d: am f16474d0: Merge "Fix some java_lang_Class related moving GC bugs"

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

am f16474d0: Merge "Fix some java_lang_Class related moving GC bugs"

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

Merge "Fix some java_lang_Class related moving GC bugs"

Fix some java_lang_Class related moving GC bugs

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

Bug: 21898408
Change-Id: Icb754074dfb469473101d20d6873a5bc3274abc5

am f8509d31: am 5592c688: Merge "Add missing EndAssertNoThreadSuspension"

* commit 'f8509d3192ffd0959b822f31f18089315b873f76':
  Add missing EndAssertNoThreadSuspension

am 5592c688: Merge "Add missing EndAssertNoThreadSuspension"

* commit '5592c688070fa327f56036c4c67c790289db63d9':
  Add missing EndAssertNoThreadSuspension

Merge "Add missing EndAssertNoThreadSuspension"

am b0c9a125: am 0420f81a: Merge "Release dummy MemMaps corresponding to dlopen."

* commit 'b0c9a1251fa8fc16928dd102195b74cfc93cd32d':
  Release dummy MemMaps corresponding to dlopen.

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

Change-Id: I28fc52ae481ef92cba45fc1b5abcf07c995f524c

am 0420f81a: Merge "Release dummy MemMaps corresponding to dlopen."

* commit '0420f81a5c6c08bc62060b8ff5dda8949f9a9ce9':
  Release dummy MemMaps corresponding to dlopen.

Merge "Release dummy MemMaps corresponding to dlopen."

am a3b9d4ed: Add missing EndAssertNoThreadSuspension

* commit 'a3b9d4eda1ef0845f307891f349eaf752750ac16':
  Add missing EndAssertNoThreadSuspension

Release dummy MemMaps corresponding to dlopen.

This fixes memory leak.

Change-Id: I857f078e8559a56b2d10dd081256c41ab28c8ec6

Add missing EndAssertNoThreadSuspension

(cherry picked from commit a3b9d4eda1ef0845f307891f349eaf752750ac16)

Bug: 21872507
Change-Id: I20f9920e4a12511ce498c252bca955d82a4f4102

Add missing EndAssertNoThreadSuspension

Bug: 21872507
Change-Id: I20f9920e4a12511ce498c252bca955d82a4f4102

Forgot this change in last commit.

bug:21746693
Change-Id: I1ae9690d393cf1bae10cb06ff1c6754ab2a50595

am cd0d0b93: am b66fda7c: Merge "ART: Allow PackedSwitch instructions with zero targets"

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

am b66fda7c: Merge "ART: Allow PackedSwitch instructions with zero targets"

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

Merge "ART: Allow PackedSwitch instructions with zero targets"

am 9ede035a: am 43d154be: Merge "Revert "ART: Allow PackedSwitch instructions with zero targets""

* commit '9ede035ac5b64dd1a44847ea797eea13946b1fac':
  Revert "ART: Allow PackedSwitch instructions with zero targets"

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

am 43d154be: Merge "Revert "ART: Allow PackedSwitch instructions with zero targets""

* commit '43d154bed3ad0bba766211af1f2637500d30a93f':
  Revert "ART: Allow PackedSwitch instructions with zero targets"

Merge "Revert "ART: Allow PackedSwitch instructions with zero targets""

Revert "ART: Allow PackedSwitch instructions with zero targets"

This fixed Optimizing but revealed the same issue in the interpreter.

This reverts commit 241f9c41924e33e0c3bab9a7c4306397458749ca.

Change-Id: Iad5a28b24f2c21d3575cf8ecc8b7c8fbf98d1132

am c957bb09: am cb5dc7f4: Merge "ART: Allow PackedSwitch instructions with zero targets"

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

am cb5dc7f4: Merge "ART: Allow PackedSwitch instructions with zero targets"

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

am afe9e134: Set default visibility on art_sigsegv_fault.

* commit 'afe9e134a773242b142b0740383f085f08ec1cba':
  Set default visibility on art_sigsegv_fault.

Merge "ART: Allow PackedSwitch instructions with zero targets"

ART: Allow PackedSwitch instructions with zero targets

Optimizing's Builder wrongly assumed that a PackedSwitch always has
at least one target. This patch removes the corresponding DCHECK and
adds a regression test case.

Bug:21863783
Change-Id: I32b7033ed38de6f1d1a6ee5d5bf12f3a47c9b37e

Set default visibility on art_sigsegv_fault.

libart.so is built with flag -fvisibility=protected
which complicates setting breakpoints on art_sigsegv_fault.
According to the GCC manual "Protected visibility indicates
that the symbol will be placed in the dynamic symbol table,
but that references within the defining module will bind
to the local symbol.". This implies that a breakpoint on
a protected symbol will not be hit.

Without this change gdbclient with ART debugging enabled
doesn't break at art_sigsegv_fault as expected.

Bonus build fix included (commit c37769b6) :

Fix lint error.

Blank line at the start of a code block.

(cherry picked from commit f775bab533cd385c00ee30b56bdcb6f962fc5239)
(cherry picked from commit c37769b63faf7b120a6ed80f5cfb92667f5090c4)

bug: 21893559

Change-Id: Ie388f027f717a7bd2351a0163a38e50d3acbba8f

am 98c8ade0: am edb83c60: Merge "Fix bug in optimizing around instanceof."

* commit '98c8ade02a02743b01a93942504051ce3246db94':
  Fix bug in optimizing around instanceof.

am edb83c60: Merge "Fix bug in optimizing around instanceof."

* commit 'edb83c606e034d76bed1331f34cdc435df47bb95':
  Fix bug in optimizing around instanceof.

Merge "Fix bug in optimizing around instanceof."

Fix bug in optimizing around instanceof.

We were too aggressive when removing instanceof. We should
not remove it when there is one of the two static types that
is an interface.

Change-Id: I1fd80915b99b094f7b4393e7adb2b160201b30d5

am 29518ae1: am 4b9a93e7: Merge "Remove bogus DCHECK in BCE."

* commit '29518ae102dd8438c5920f6757014d032f6cba97':
  Remove bogus DCHECK in BCE.

am f3c1e0d9: am 445e3502: Merge "More stack map dumping."

* commit 'f3c1e0d98b90a90281d5196b1efc5cf494e834b2':
  More stack map dumping.

am 4f579be1: am a2890553: Merge "Fix lint error."

* commit '4f579be197e9841e5c71cc28823c8056789fa8fc':
  Fix lint error.

am 419357d8: am 233cf0e8: Merge "Exercise sun.misc.Unsafe.getObject."

* commit '419357d89f948aeae875d569c9a312e7d3571415':
  Exercise sun.misc.Unsafe.getObject.

am 17ab6767: am 088bd0d2: Merge "Revert "Revert "ART: Implement literal pool for arm, fix branch fixup."""

* commit '17ab67676c951cfd5eca61d7f6d2eb20d40f7d54':
  Revert "Revert "ART: Implement literal pool for arm, fix branch fixup.""

am fc4e395a: am b3daa231: Merge "Set default visibility on art_sigsegv_fault."

* commit 'fc4e395af2c41862d1519b74ec818cdb52b04691':
  Set default visibility on art_sigsegv_fault.

am d56dd524: am ed44919d: Merge "CFI test: kill the other process in native code."

* commit 'd56dd5248cbd0d520dfd2ae3dc1c712d3e91b28d':
  CFI test: kill the other process in native code.

am 4b9a93e7: Merge "Remove bogus DCHECK in BCE."

* commit '4b9a93e7774d6650f6f3b637d018109d6fe97546':
  Remove bogus DCHECK in BCE.

am 445e3502: Merge "More stack map dumping."

* commit '445e35028bc5cb6d921dc60e579b7b40ceb16c71':
  More stack map dumping.

am a2890553: Merge "Fix lint error."

* commit 'a28905535905ed2b86de6f621991b8775beb493e':
  Fix lint error.

am 233cf0e8: Merge "Exercise sun.misc.Unsafe.getObject."

* commit '233cf0e8584225f1f7d21d4999cda18c23469d59':
  Exercise sun.misc.Unsafe.getObject.

am 088bd0d2: Merge "Revert "Revert "ART: Implement literal pool for arm, fix branch fixup."""

* commit '088bd0d28acb70302d20f6e19a9db7f0536c09ee':
  Revert "Revert "ART: Implement literal pool for arm, fix branch fixup.""

am b3daa231: Merge "Set default visibility on art_sigsegv_fault."

* commit 'b3daa2312747eaf852c948c189ca0745db116447':
  Set default visibility on art_sigsegv_fault.

am ed44919d: Merge "CFI test: kill the other process in native code."

* commit 'ed44919da8f7ab96117fd9d30b3fbfcd88f1433c':
  CFI test: kill the other process in native code.

Merge "Remove bogus DCHECK in BCE."

Remove bogus DCHECK in BCE.

When creating a phi for the array length when we add HDeoptimization
nodes, we might update accesses in inner loops to use that phi instead
of the array length. The BCE phase was not expecting this case.

Change-Id: I639f4ea6f5889726142041a42736183f162c7437

Merge "More stack map dumping."

More stack map dumping.

Change-Id: I1f734c0c9eabda1cc6adaf32579e4c3be4a45c9a

Merge "Fix lint error."

Merge "Exercise sun.misc.Unsafe.getObject."

Fix lint error.

Blank line at the start of a code block.

Change-Id: Ifc0d78acc8b2eecbdc1a340009757f9218ce2c5c

Merge "Revert "Revert "ART: Implement literal pool for arm, fix branch fixup."""

Revert "Revert "ART: Implement literal pool for arm, fix branch fixup.""

This reverts commit fbeb4aede0ddc5b1e6a5a3a40cc6266fe8518c98.

Adjust block label positions. Bad catch block labels were the
reason for the revert.

Change-Id: Ia6950d639d46b9da6b07f3ade63ab46d03d63310

Merge "Set default visibility on art_sigsegv_fault."

Set default visibility on art_sigsegv_fault.

libart.so is built with flag -fvisibility=protected
which complicates setting breakpoints on art_sigsegv_fault.
According to the GCC manual "Protected visibility indicates
that the symbol will be placed in the dynamic symbol table,
but that references within the defining module will bind
to the local symbol.". This implies that a breakpoint on
a protected symbol will not be hit.

Without this change gdbclient with ART debugging enabled
doesn't break at art_sigsegv_fault as expected.

Change-Id: Ib73344e710966eddf55a6facb29558504a511f00

Merge "CFI test: kill the other process in native code."

CFI test: kill the other process in native code.

Killing it from Java does not seem reliable and the CFI
test tends to leave alive processes on the system.
This might be particularly troubling on the build bot.

Change-Id: I888b61037ab77fb4ff929762ea40af53f7a6ed6e

am 1070c1ce: am b3e3000a: Merge "Find the dlopened file using address rather than file path."

* commit '1070c1ce23448fab59e326368bacbb13d54288c6':
  Find the dlopened file using address rather than file path.

am b3e3000a: Merge "Find the dlopened file using address rather than file path."

* commit 'b3e3000a59a3b0d765840bd1a939213a2b400763':
  Find the dlopened file using address rather than file path.

Merge "Find the dlopened file using address rather than file path."

Find the dlopened file using address rather than file path.

This should fix the build bot.  It was failing because the
path which we pass to dlopen is not the path that gets mmaped.

Bug: 21854739
Change-Id: I0c28c74b62743f74f9ff3ae89e6f1beb0c1113ae

am fd1c3574: am 89c63739: Merge "Add more logging output for dlopen."

* commit 'fd1c357462d3decb4bb50ebc6ccb26b6ce7a17f2':
  Add more logging output for dlopen.

am 3c9229d7: am 86b35f97: Merge "Revert "Revert "Use dlopen to load oat files."""

* commit '3c9229d767191c19d4e16c2354996591f6b74fcd':
  Revert "Revert "Use dlopen to load oat files.""

am 89c63739: Merge "Add more logging output for dlopen."

* commit '89c63739e280ab94399d263624558e1d99fb5af7':
  Add more logging output for dlopen.

am 86b35f97: Merge "Revert "Revert "Use dlopen to load oat files."""

* commit '86b35f97ce1848608e70eb1f0ea9f73d25b80d09':
  Revert "Revert "Use dlopen to load oat files.""

Merge "Add more logging output for dlopen."

Merge "Revert "Revert "Use dlopen to load oat files."""

Add more logging output for dlopen.

Investigation of build bot failure which I can not reproduce locally.

Bug: 21854739
Change-Id: I4584bb10ad44e4bcf8232c060e40b36ff924ffa2

am 4eacb2bd: am 7e7e69a0: Merge "Print more diagnosis info on to-space invariant violation."

* commit '4eacb2bda0d0d6708cb7f8e64b91ae9511cd41c5':
  Print more diagnosis info on to-space invariant violation.

am 7e7e69a0: Merge "Print more diagnosis info on to-space invariant violation."

* commit '7e7e69a0421e30f5e6ffa498db25dfd0c13d500e':
  Print more diagnosis info on to-space invariant violation.

Merge "Print more diagnosis info on to-space invariant violation."

Revert "Revert "Use dlopen to load oat files.""

This reverts commit 7a46b7e235a7c35191cd9ddbab6ca4db0e535da5.

Change-Id: I33439d0e3d7a7e63591a8c7b356e716dcb62bca1