resolved conflicts for merge of 57504548 to lmp-dev-plus-aosp

Change-Id: I17dba9a68f3af09033cd0a5a221bb8474aa18502

am 04a685d8: Merge "Added --gcstress and --gcverify shortcut flags to run-test"

* commit '04a685d86cdf7d386f9dc2efc1626f04becace4e':
  Added --gcstress and --gcverify shortcut flags to run-test

am 3e6ff5bc: Merge "Move explicit GC after we are done initalizing all classes."

* commit '3e6ff5bcf6f365fd869a5ce62235d4957c71bdbe':
  Move explicit GC after we are done initalizing all classes.

Merge "Move explicit GC after we are done initalizing all classes."

am b74ff132: Merge "Reduce stack usage for overflow checks"

* commit 'b74ff132f217776c1ab698712bc2dbc526297288':
  Reduce stack usage for overflow checks

am 826f1038: Merge "Use handle in one case that spans a gc-point."

* commit '826f1038187e00399a8c6e0044ad0ea29c0ba000':
  Use handle in one case that spans a gc-point.

am c739de01: Merge "Avoid use of std::string where we have const char*." into lmp-dev

* commit 'c739de01b905729b8587003a52a55b62358e979d':
  Avoid use of std::string where we have const char*.

am eeedc3fe: am 83715081: Add allocation rate tracking to systrace.

* commit 'eeedc3fe01bfddeda348b18d388ae1f557db9db0':
  Add allocation rate tracking to systrace.

am 9ad4c8f8: (-s ours) am b0f05b96: Add implicit checks for x86_64 architecture.

* commit '9ad4c8f87bbecb19b171132e555bf45da417d226':
  Add implicit checks for x86_64 architecture.

am 83715081: Add allocation rate tracking to systrace.

* commit '837150818af0eee993f7d93c5a50c3c7b46f1dab':
  Add allocation rate tracking to systrace.

Add allocation rate tracking to systrace.

Bug: 16956980

Change-Id: I7074f4ae0f75d1bb58e9860ded4e17ce6978d74e

am b0f05b96: Add implicit checks for x86_64 architecture.

* commit 'b0f05b9654eb005bc8c8e15f615a7f5a312f640c':
  Add implicit checks for x86_64 architecture.

am e39a5e1c: Merge "Execute an application even when dex2oat crashes."

* commit 'e39a5e1ced3d2b87974c646678dd719c02a9a010':
  Execute an application even when dex2oat crashes.

Merge "Added --gcstress and --gcverify shortcut flags to run-test"

Add implicit checks for x86_64 architecture.

This combines the x86 and x86_64 fault handlers into one.  It also
merges in the change to the entrypoints for X86_64.

Replaces generic instruction length calculator with one that only
works with the specific instructions we use.

Bug: 16256184

Change-Id: I1e8ab5ad43f46060de9597615b423c89a836035c
Signed-off-by: Chao-ying Fu <chao-ying.fu@intel.com>

am b6625211: Merge "Fix crash in VirtualMachine.AllThreads JDWP command"

* commit 'b66252112d45fd24eab85122fa8902afd1324542':
  Fix crash in VirtualMachine.AllThreads JDWP command

am 697f4c63: am 4527f61c: ART: Don\'t leave rex_ & length_ uninitialized

* commit '697f4c634f772c0f4d3087ac851fc1911f122d63':
  ART: Don't leave rex_ & length_ uninitialized

am 4527f61c: ART: Don\'t leave rex_ & length_ uninitialized

* commit '4527f61cbc1a3b01ab249181900b8ea547edaf90':
  ART: Don't leave rex_ & length_ uninitialized

ART: Don't leave rex_ & length_ uninitialized

Don't leave fields uninitialized in the x86-64 utils assembler.

Bug: 16993295

(cherry picked from commit 181211de89df5615395a55114fe551b08e86bc4d)

Change-Id: I2d648991fc2a6a4d5f518eb9ef0bb10857f6318a

am c38752d5: Merge "ART: Don\'t leave rex_ & length_ uninitialized"

* commit 'c38752d535eefee3d5cd2399514bf405e9dd3827':
  ART: Don't leave rex_ & length_ uninitialized

am 0d0bbb5b: am 13cccfa3: Merge "Fix crash in VirtualMachine.AllThreads JDWP command" into lmp-dev

* commit '0d0bbb5b65a777594408178a402a833ea99e79cf':
  Fix crash in VirtualMachine.AllThreads JDWP command

am 13cccfa3: Merge "Fix crash in VirtualMachine.AllThreads JDWP command" into lmp-dev

* commit '13cccfa3ce61a41a176358ff2aa53e60d0acacd5':
  Fix crash in VirtualMachine.AllThreads JDWP command

am 6c6c6f66: Merge "Avoid use of std::string where we have const char*."

* commit '6c6c6f6683cbbb180b8b8c8f55886b067320e1e8':
  Avoid use of std::string where we have const char*.

am 1b3e5b6f: Merge "Implement GenInlinedReverseBits"

* commit '1b3e5b6ffbb5f1cb21e6b1128d0031d14ba056d7':
  Implement GenInlinedReverseBits

am 33e70ef9: Merge "Fix art build script"

* commit '33e70ef9bcf916dd637c03eaad41c57c09bc4261':
  Fix art build script

Merge "Fix crash in VirtualMachine.AllThreads JDWP command" into lmp-dev

Merge "Avoid use of std::string where we have const char*." into lmp-dev

Avoid use of std::string where we have const char*.

Removing the ClassHelper caused std::string creation for all calls to
Class::GetDescriptor and a significant performance regression. Make the
std::string an out argument so the caller can maintain it and its life time
while allowing GetDescriptor to return the common const char* case.

Don't generate GC maps when compilation is disabled.

Remove other uses of std::string that are occuring on critical paths.
Use the cheaper SkipClass in CompileMethod in CompilerDriver.
Specialize the utf8 as utf16 comparison code for the common shorter byte
encoding.
Force a bit of inlining, remove some UNLIKELYs (they are prone to pessimizing
code), add some LIKELYs.

x86-64 host 1-thread interpret-only of 57 apks:
Before: 29.539s
After: 23.467s

Regular compile:
Before: 1m35.347s
After: 1m20.056s

Bug: 16853450
Change-Id: Ic705ea24784bee24ab80084d06174cbf87d557ad

Conflicts:
runtime/utils.cc

Merge "Use handle in one case that spans a gc-point."

Use handle in one case that spans a gc-point.

Bug: 16689428
Change-Id: Ib209d2ca1e7024bffb2bc17c5b5899e231c0e2ca

Merge "Avoid use of std::string where we have const char*."

Merge "Fix art build script"

Fix art build script

Use ART_BUILD_HOST_NDEBUG instead of ART_BUILD_NDEBUG.

Change-Id: I0ff590552f47d3354287a155b51936a7aef82f1f
Signed-off-by: Junmo Park <junmoz.park@samsung.com>

Avoid use of std::string where we have const char*.

Removing the ClassHelper caused std::string creation for all calls to
Class::GetDescriptor and a significant performance regression. Make the
std::string an out argument so the caller can maintain it and its life time
while allowing GetDescriptor to return the common const char* case.

Don't generate GC maps when compilation is disabled.

Remove other uses of std::string that are occuring on critical paths.
Use the cheaper SkipClass in CompileMethod in CompilerDriver.
Specialize the utf8 as utf16 comparison code for the common shorter byte
encoding.
Force a bit of inlining, remove some UNLIKELYs (they are prone to pessimizing
code), add some LIKELYs.

x86-64 host 1-thread interpret-only of 57 apks:
Before: 29.539s
After: 23.467s

Regular compile:
Before: 1m35.347s
After: 1m20.056s

Bug: 16853450
Change-Id: Ic705ea24784bee24ab80084d06174cbf87d557ad

am 99c251bb: Merge "Revert "Revert "Remove GCC atomic workarounds."""

* commit '99c251bbd225dd97d0deece29559a430b12a0b66':
  Revert "Revert "Remove GCC atomic workarounds.""

am 9c1dffed: Merge "Initialize art::Atomic\'s value to 0."

* commit '9c1dffed26aa1faa1db1f1875f135cce697d92c4':
  Initialize art::Atomic's value to 0.

Merge "Initialize art::Atomic's value to 0."

Initialize art::Atomic's value to 0.

art::Mutex and art::ConditionVariable were DCHECKing that the value was
zero on entering the constructor without ever initializing the value.
Since 0 is a sensible default, make it the default for art::Atomic
rather than initializing for each use.

Bug: 16301104
Change-Id: I9c98b82c80670b7a6c53d46a08236bbed6a64f8f

Merge "Fix crash in VirtualMachine.AllThreads JDWP command"

Fix crash in VirtualMachine.AllThreads JDWP command

When collecting all the started threads, we may encounter a thread in the
process of being started from Java code (Thread.start) in the threads list. The
crash occurs when we attempt to access the java.lang.Thread peer (calling
Thread::GetPeer) but the JNI global reference to it (Thread::jpeer) has not
been destroyed yet (which is verified with a check). This only happens when the
thread is starting up.

We now check the thread finished starting up before accessing the Java peer.
This also prevents from returning non-started threads in the reply.

Bug: 16660101

(cherry picked from commit a22d115711b5158cfc67b4f260a5b40f3ab303ca)

Change-Id: Ib61dbd0b01f63bfdda708d3c93eac58284bbe236

Merge "Revert "Revert "Remove GCC atomic workarounds."""

Merge "Reduce stack usage for overflow checks"

Reduce stack usage for overflow checks

This reduces the stack space reserved for overflow checks to 12K, split
into an 8K gap and a 4K protected region.  GC needs over 8K when running
in a stack overflow situation.

Also prevents signal runaway by detecting a signal inside code that
resulted from a signal handler invokation.  And adds a max signal count to
the SignalTest to prevent it running forever.

Also reduces the number of iterations for the InterfaceTest as this was
taking (almost) forever with the --trace option on run-test.

Bug: 15435566

Change-Id: Id4fd46f22d52d42a9eb431ca07948673e8fda694

Merge "Execute an application even when dex2oat crashes."

Execute an application even when dex2oat crashes.

Bug: 17000769

Change-Id: Iffeb582862a5e794b6c7364c7ec2368cfd0f2214

Merge "ART: Don't leave rex_ & length_ uninitialized"

ART: Don't leave rex_ & length_ uninitialized

Don't leave fields uninitialized in the x86-64 utils assembler.

Bug: 16993295
Change-Id: I1d1b96f1d666660024116372de0014f1d19fd487

Merge "Implement GenInlinedReverseBits"

Implement GenInlinedReverseBits

Added support for x86 inlined version of reverse method of int and long

Change-Id: I7dbdc13b4afedd56557e9eff038a31517cdb1843
Signed-off-by: Yixin Shou <yixin.shou@intel.com>

am f7216c6f: am 5f1dcee2: Merge "Change FreeMemory and TotalMemory behavior." into lmp-dev

* commit 'f7216c6f5d0c0ad061b9c8f43326fd0c79809f0d':
  Change FreeMemory and TotalMemory behavior.

am 290bf821: Merge "Prevent merge conflicts."

* commit '290bf8214da0621bf9ddefc65a3bd310b935971f':
  Prevent merge conflicts.

am 5f1dcee2: Merge "Change FreeMemory and TotalMemory behavior." into lmp-dev

* commit '5f1dcee2b3cdaf0f745958ff366a84279940c65e':
  Change FreeMemory and TotalMemory behavior.

am d17554a9: am b21f1336: Make method tracing deoptimize on startup.

* commit 'd17554a98b83c0b7bad16d81c3210516b0125d04':
  Make method tracing deoptimize on startup.

am b21f1336: Make method tracing deoptimize on startup.

* commit 'b21f1336be9dd0c2607b2ff53834d62a218d1c83':
  Make method tracing deoptimize on startup.

Make method tracing deoptimize on startup.

Bug: 16848366

(cherry picked from commit bc678bba8ec45dfcb1865676dd6d8188da9e56fc)

Change-Id: Idf75288026a837b95f4619acadec57e38bacbd7b

Merge "Change FreeMemory and TotalMemory behavior." into lmp-dev

Change FreeMemory and TotalMemory behavior.

TotalMemory is now equal to the footprint limit (around when the
next GC will occur).
FreeMemory is now equal to TotalMemory() - bytes_allocated_.
Also added more memory dumping info to DumpGcPerformanceInfo.

Bug: 16520008
Bug: 15819878

(cherry picked from commit dd162fb5990cedf80a5093ecc0e77df82af5f754)

Change-Id: I68239a5295cc1372b3995eb781d67b9e25d6eaed

Merge "Prevent merge conflicts."

Prevent merge conflicts.

Change-Id: I61af7f3c9be47d4776371dfe2b97850ea3f3101f

Added --gcstress and --gcverify shortcut flags to run-test

Change-Id: Ia11fcbc71488710ca32f397c3c1b19613e294d1e

am 66386248: Merge "Make method tracing deoptimize on startup."

* commit '66386248003a46ecbe4fac99aec902cfd0942847':
  Make method tracing deoptimize on startup.

Fix crash in VirtualMachine.AllThreads JDWP command

When collecting all the started threads, we may encounter a thread in the
process of being started from Java code (Thread.start) in the threads list. The
crash occurs when we attempt to access the java.lang.Thread peer (calling
Thread::GetPeer) but the JNI global reference to it (Thread::jpeer) has not
been destroyed yet (which is verified with a check). This only happens when the
thread is starting up.

We now check the thread finished starting up before accessing the Java peer.
This also prevents from returning non-started threads in the reply.

Bug: 16660101

Change-Id: I9e445e5da3a6951143d6c3c9a0d4f136398fde2f

am 775f2b88: am d74824bd: AArch64: Fix art_quick_string_compareto.

* commit '775f2b8807ec1419b3e8d81fc38433ecb9124023':
  AArch64: Fix art_quick_string_compareto.

am d74824bd: AArch64: Fix art_quick_string_compareto.

* commit 'd74824bdd01b2a76b310e1275a1114d39833a708':
  AArch64: Fix art_quick_string_compareto.

am 3792f6fa: Merge "AArch64: Fix art_quick_string_compareto."

* commit '3792f6fa90bd9fe8eb45d5913d992d016e388b37':
  AArch64: Fix art_quick_string_compareto.

AArch64: Fix art_quick_string_compareto.

Though __memcmp16() is implemented in the same module as the caller, it
is still possible that the toolchain would put __memcmp16() into plt. In
that case, IP registers can be trashed when loading the function address.
Use x14/x15 to replace IP0/IP1.

Bug: 16974467

(cherry picked from commit 62ddb328860e907eb76ccd3abed63ba75438fea8)

Change-Id: I40e39d075860bc78624ce6ef8b4f8e33e57fc58c

am 1cabb1cf: Merge "Fix missing operation in CombineBlocks"

* commit '1cabb1cf03f2cf9e008abe6c19a575006d87521f':
  Fix missing operation in CombineBlocks

am 313ec270: am 8c6c5211: Revert "Rename openDexFileNative to openDexFile."

* commit '313ec270ace665657e1f353be6bb478905b3b1c7':
  Revert "Rename openDexFileNative to openDexFile."

am 8c6c5211: Revert "Rename openDexFileNative to openDexFile."

* commit '8c6c52110c5afd438741694520203a6d7e81a2a4':
  Revert "Rename openDexFileNative to openDexFile."

am e95df5fa: Merge "Fix missing operation in SplitBlock"

* commit 'e95df5fa957ca294d6b5048e7d01318a11f2800e':
  Fix missing operation in SplitBlock

am c2798e9a: Merge "Change FreeMemory and TotalMemory behavior."

* commit 'c2798e9ad7d8ccfba9bc5222d347a6d29fc99f79':
  Change FreeMemory and TotalMemory behavior.

Merge "Change FreeMemory and TotalMemory behavior."

Change FreeMemory and TotalMemory behavior.

TotalMemory is now equal to the footprint limit (around when the
next GC will occur).
FreeMemory is now equal to TotalMemory() - bytes_allocated_.
Also added more memory dumping info to DumpGcPerformanceInfo.

Bug: 16520008
Bug: 15819878

Change-Id: Ifa59a2f2136dd0704097411ee183925e6c387fae

Revert "Revert "Remove GCC atomic workarounds.""

This reverts commit 626a2468e4e4f39db7b0f35a4fee87293e360e92.

am 6814d7b6: Merge "Revert "Remove GCC atomic workarounds.""

* commit '6814d7b68ee13ec1a13ace669b025aa080d8b53e':
  Revert "Remove GCC atomic workarounds."

am ce223181: Merge "Add tests for getDeclaredConstructor"

* commit 'ce223181552ed2c71a84eb545b1ee8f01f17aa0c':
  Add tests for getDeclaredConstructor

Move explicit GC after we are done initalizing all classes.

Previously we had the GC happen for each dex file, this added a
few seconds of time on boot.oat creation.

Bug: 16853450

Change-Id: I6b78c8c6f6c3bc20c164d951f2af8efbff0b8600

am fba83719: Merge "Fix more of the Mac build."

* commit 'fba83719dddaa5043c6c6f21e1000ca2faeaea39':
  Fix more of the Mac build.

Merge "Fix missing operation in CombineBlocks"

Fix missing operation in CombineBlocks

explicit throw, conditional branch flags also need to be moved.
There are no quick compiler bugs before this CL.
So regression test is not necessary.

Change-Id: I5f7c0f261fff5f7a46b32763096ab4fe85b2c0c0
Signed-off-by: Junmo Park <junmoz.park@samsung.com>

Merge "Make method tracing deoptimize on startup."

Make method tracing deoptimize on startup.

Bug: 16848366
Change-Id: I929c48622987c090c5233490d4e7836a21e1b57e

Merge "Fix missing operation in SplitBlock"

Merge "Fix more of the Mac build."

Fix more of the Mac build.

Change-Id: I0fa52ef73e86318bb68de2c69bbed81a00bfc3e0

Merge "Add tests for getDeclaredConstructor"

Merge "Revert "Remove GCC atomic workarounds.""

Revert "Remove GCC atomic workarounds."

This reverts commit a29ffd505328b3d580c25fff054e463b7cac08a8.

Change-Id: Ibb4845b8a1378f3d1fb0975f9677758f420f843f

Merge "AArch64: Fix art_quick_string_compareto."

AArch64: Fix art_quick_string_compareto.

Though __memcmp16() is implemented in the same module as the caller, it
is still possible that the toolchain would put __memcmp16() into plt. In
that case, IP registers can be trashed when loading the function address.
Use x14/x15 to replace IP0/IP1.

Bug: 16974467
Change-Id: Ica7294b9bf90342031efa5fd51a86a6fcab5852b

am 6e1fa769: Merge "Add gcstress top-level make rule, eg test-art-host-run-test-gcstress."

* commit '6e1fa769a6baf407116ab18dea8b760d130f7c3a':
  Add gcstress top-level make rule, eg test-art-host-run-test-gcstress.

am 0de6e0df: Merge "Remove GCC atomic workarounds."

* commit '0de6e0df4a9fba1353ea84a65d6a45f3b895621c':
  Remove GCC atomic workarounds.

Remove GCC atomic workarounds.

libcxx now has compatibility with GCC atomic support.
Bug: 16301104

Change-Id: I0ba5abf7147e999a8329c9e8a48b3712370ee4a6

Revert "Rename openDexFileNative to openDexFile."

https://android-review.googlesource.com/#/c/103383/ introduces
absolute paths so we need back the old method name

Bug: 16644204
Bug: 16930747

This reverts commit 350b626eb1cfbaee7619e9453a4c018c07aed003.

(cherry picked from commit a2069c7152eb46e8c64b4c8d7e19e5fab5de6df4)

Change-Id: I55c8cb31bb79b1ed8ea51f63ec504d77f76a582a

Merge "Add gcstress top-level make rule, eg test-art-host-run-test-gcstress."

Add gcstress top-level make rule, eg test-art-host-run-test-gcstress.

Change-Id: I221aa3274a513820b12ac25cdfecc911ce212144

Merge "Remove GCC atomic workarounds."

am fffca74f: am 1c98286d: Merge "ART: Skip compiling redefined classes in apps" into lmp-dev

* commit 'fffca74fde87b7eaa3ec51960cfeca2a63b334b6':
  ART: Skip compiling redefined classes in apps

am 1c98286d: Merge "ART: Skip compiling redefined classes in apps" into lmp-dev

* commit '1c98286d8ceaf3715e06f50acd6f3b0aa5a06f2e':
  ART: Skip compiling redefined classes in apps

Merge "ART: Skip compiling redefined classes in apps" into lmp-dev