am a092ee40: Merge "Use non volatile lock words in semispace collector."

* commit 'a092ee404ababbf8fc945111dc4a571284c9f02f':
  Use non volatile lock words in semispace collector.

am 94e8a973: Merge "Add untested x86-64 downcall and exception assembly."

* commit '94e8a97310af35cb0bae309e796b554cdb586ca5':
  Add untested x86-64 downcall and exception assembly.

am b99985a0: Merge "Fix a class-loading bug in the verifier when throwing NPE"

* commit 'b99985a089e23929c920201c14875b51c5b6217e':
  Fix a class-loading bug in the verifier when throwing NPE

am 425640d6: Merge "Use LIRSlowPath for throwing NPE."

* commit '425640d6fac3432a44e236a002e8401a7cd59c5c':
  Use LIRSlowPath for throwing NPE.

am 79f2c53f: Merge "String.IndexOf method handles negative start index value in incorrect way"

* commit '79f2c53f3cb8649c19aeded82f7b8dcc96015808':
  String.IndexOf method handles negative start index value in incorrect way

Merge "Use non volatile lock words in semispace collector."

Use non volatile lock words in semispace collector.

GSS FormulaEvaluationActions.EvaluateAndApplyChanges:
Before GC time: 9.1s
After GC time: 7.98s

Fixed timing logger errors.

Change-Id: I4193c6ccbbbe7a7220dfaabbf3472a5dcebae616

Merge "Add untested x86-64 downcall and exception assembly."

Merge "Fix a class-loading bug in the verifier when throwing NPE"

Fix a class-loading bug in the verifier when throwing NPE

When throwing an NPE for invocation, we try to resolve the class of
the method being called. When in the interpreter and having quickened
code, that failed.

Bug: 14133618
Change-Id: I4964b908bb26a82a12263fb86f5dc39c9042479b

Merge "Use LIRSlowPath for throwing NPE."

Add untested x86-64 downcall and exception assembly.

Change-Id: Ic555f9f5af8c3a2110a92e55772ff6c0128e5c19

Merge "String.IndexOf method handles negative start index value in incorrect way"

am 984cd08f: Merge "Code cleanup in preparation for x64 backend."

* commit '984cd08f363153aabfa436b61277c30ad6fa4e7d':
  Code cleanup in preparation for x64 backend.

am c2b2bbf1: Merge "Simplify HInvokeStatic code generation."

* commit 'c2b2bbf1bbdf6273298b79d6006611593ed9f3a0':
  Simplify HInvokeStatic code generation.

am e8b2e6e7: Merge "Preparation for transition to libc++."

* commit 'e8b2e6e70decca7ce324350c9c03c9a432ed64d6':
  Preparation for transition to libc++.

Merge "Code cleanup in preparation for x64 backend."

Code cleanup in preparation for x64 backend.

- Use InvokeDexCallingConventionVisitor for setting
  up HParameterValues
- Use kVregSize instead of kX86WordSize when dealing with
  virtual registers.

Change-Id: Ia520223010194c70a3ff0ed659077f55cec4e7d8

Merge "Simplify HInvokeStatic code generation."

Simplify HInvokeStatic code generation.

HPushArgument is not needed for now (but might be when we start
optimizing). Also, calling convention for 64bits backend will
require to know more about the argument than the argument's
index. Therefore currently let HInvokeStatic setup the arguments,
which is possible because arguments of a calls are virtual registers
and not instructions.

Change-Id: I8753ed6083aa083c5180ab53b436dc8de4f1fe31

String.IndexOf method handles negative start index value in incorrect way

The standard implementation of String.IndexOf converts the negative value of
the start index to 0 and searching will start from the beginning of the string.
But current implementation may start searching from the incorrect memory
offset, that can lead to sigsegv or return incorrect result.
This patch adds the handler for cases when fromIndex is negative.

Change-Id: I3ac86290712789559eaf5e46bef0006872395bfa
Signed-off-by: Alexei Zavjalov <alexei.zavjalov@intel.com>

Merge "Preparation for transition to libc++."

Preparation for transition to libc++.

Move the dependency on libc++ to its own makefile so that we can switch in a
single place between libc++ and stlport.

Change-Id: Ie61e7d054dcd049e36d5e7298c27d8a4abe6edf7

am f7933e7f: Merge "Disable another timing sensitive test on dist builds"

* commit 'f7933e7f295bfe61cb0baf89469581c563032e96':
  Disable another timing sensitive test on dist builds

am 520ab876: Merge "Build dex2oat as 64b for the target when using zygote64"

* commit '520ab8768791cf4d51a543023960991c30a75e69':
  Build dex2oat as 64b for the target when using zygote64

Merge "Disable another timing sensitive test on dist builds"

Merge "Build dex2oat as 64b for the target when using zygote64"

am 5e17616b: Merge "Fix volatile wide put/get to be atomic on x86 arch"

* commit '5e17616b7386159cef8f2fb6ffe3cbc4fb1373e4':
  Fix volatile wide put/get to be atomic on x86 arch

Disable another timing sensitive test on dist builds

Change-Id: I1c0dc16ff22b23dbd238cbeba3f4c078c3ffeaed

Merge "Fix volatile wide put/get to be atomic on x86 arch"

am a214996d: Merge "Decrease target utilization for foreground apps."

* commit 'a214996df216757456b548eb9484fe04f8f9b08b':
  Decrease target utilization for foreground apps.

Fix volatile wide put/get to be atomic on x86 arch

Current implementation puts memory barriers for volatile fields.
Volatile semantics needs atomicity, which are not guaranteed by memory barriers.

The patch forces all wide volatile fields to be loaded/stored using
xmm registers.

Change-Id: Ie78e186d13ffa237e6e93747b71d26651fa02866
Signed-off-by: Yevgeny Rouban <yevgeny.y.rouban@intel.com>

Build dex2oat as 64b for the target when using zygote64

In single-zygote 64b builds we should build dex2oat as 64b to
avoid installd issues

Change-Id: I5f281d90fbd44abbca4182e39162b184be9f1b35

Merge "Decrease target utilization for foreground apps."

Use LIRSlowPath for throwing NPE.

Get rid of launchpads for throwing NPE and use LIRSlowPath instead.
Also clean up some code of using LIRSlowPath for checking div
by zero.

Bug: 13170824

Change-Id: I0c20a49c39feff3eb1f147755e557d9bc0ff15bb

am df4f96ab: Merge "Fix thread join hangs"

* commit 'df4f96abfea464b834da858d839efc98cc032728':
  Fix thread join hangs

Decrease target utilization for foreground apps.

GC time in FormulaEvaluationActions.EvaluateAndApplyChanges goes from
26.1s to 23.2s. Benchmark score goes down ~50 in
FormulaEvaluationActions.EvaluateAndApplyChanges, and up ~50 in
GenericCalcActions.MemAllocTest.

Bug: 8788501
Change-Id: I412af1205f8b67e70a12237c990231ea62167bc0

Merge "Fix thread join hangs"

am d0235a3a: Merge "Fix typo in ISA centralization"

* commit 'd0235a3ab3da419cb8cc1f678569473bff97a752':
  Fix typo in ISA centralization

Merge "Fix typo in ISA centralization"

Fix typo in ISA centralization

X86 needs sign-extension.

Change-Id: I1253fbc9aa0e732e37b976a0723ec89155e7e2e7

am b94fae66: Merge "Centralize instruction-set pointer-size, alignment, 64b-width code in instruction_set.h/cc"

* commit 'b94fae66dac14be955c68028447cdad84ee22bf9':
  Centralize instruction-set pointer-size, alignment, 64b-width code in instruction_set.h/cc

Merge "Centralize instruction-set pointer-size, alignment, 64b-width code in instruction_set.h/cc"

am 2faf284e: Merge "Long support in optimizing compiler."

* commit '2faf284e072230d4fd447955fa72ac4d9a3b51a0':
  Long support in optimizing compiler.

Merge "Long support in optimizing compiler."

Long support in optimizing compiler.

- Add stack locations to the Location class.
- Change logic of parameter passing/setup by setting the
  location of such instructions the ones for the calling
  convention.

Change-Id: I4730ad58732813dcb9c238f44f55dfc0baa18799

Fix thread join hangs

Current implementation notifies only one of joining threads on thread exit.
This causes the other joining threads to hang.

This patch notifies all joining threads: just call NotifyAll() instead of Notify().

Change-Id: If75f101d3d3d876f1aa7dcf537e56a37cc347803
Signed-off-by: Yevgeny Rouban <yevgeny.y.rouban@intel.com>

am a3eaf66b: Merge "Disable JDWP field watchpoint capabilities"

* commit 'a3eaf66b968270b938a2795fe55e778c55f1c24b':
  Disable JDWP field watchpoint capabilities

Merge "Disable JDWP field watchpoint capabilities"

Disable JDWP field watchpoint capabilities

We do not fully support field watchpoint yet. We can only enable these
capabilities when we do report these events.

Bug: 8267708
Change-Id: I07ccf7ddffdd26e143b7fb0f13ebe030d75e8491

am 51f32a65: Merge "Allow use of instance filter for JDWP events"

* commit '51f32a651666b06f3cdac4a80a65db1d7bc90aac':
  Allow use of instance filter for JDWP events

am 172109b7: Merge "Fix JDWP count event modifier"

* commit '172109b72383f109f0ee240955e15643ba755f63':
  Fix JDWP count event modifier

Merge "Allow use of instance filter for JDWP events"

Merge "Fix JDWP count event modifier"

am 9623c666: Merge "Less confusing heap transition message."

* commit '9623c6668962559e818d1e7f05a58dcb96c71fa9':
  Less confusing heap transition message.

Merge "Less confusing heap transition message."

Less confusing heap transition message.

The heap transition message now just says how long the transition
took and up to how much memory was saved (it can still be negative).

Change-Id: If421dcc2aa9cc40740b858dcc2ff1b8e595f9a40

am ebc726a9: Merge "Don\'t call SetFootprintLimit in CreateZygoteSpace."

* commit 'ebc726a931fd76a83a7dcdd3cf3505cee387cead':
  Don't call SetFootprintLimit in CreateZygoteSpace.

Merge "Don't call SetFootprintLimit in CreateZygoteSpace."

Don't call SetFootprintLimit in CreateZygoteSpace.

Since SetFootprintLimit uses the allocators accounting it can set a
word inside of the zygote space mem map. This was causing issues
since it was occasionally clobbering one of the object classes. This
occured since we compact objects into the space which was used by the
allocator's internal accounting during zygote bin packing.

Also addressed some TODO.

Bug: 14056343
Change-Id: I9fc5a61174fa141c0f44d0ed0ae0cc6d77cb4e0a

Allow use of instance filter for JDWP events

Bug: 13922844
Change-Id: Ic4e6d7af7c80928c25417db5f43049614d225c97

Fix JDWP count event modifier

Bug: 14053614
Change-Id: I736f6d6802e58e7a4fb29d0558bdd2abd3b2a48e

am c311d0c8: Merge "Prepare field watchpoint support"

* commit 'c311d0c85e17773042daaa7a4abc58b3e3c0a0c1':
  Prepare field watchpoint support

am 0a01b382: Merge "Clear unused roots from debugger"

* commit '0a01b38244053e72f33339e733b3ae5e15a5966b':
  Clear unused roots from debugger

am 1837bcb6: Merge "Deduplicate code starting JDWP thread"

* commit '1837bcb6e9b3b41a0cec8af19cd3bbf8ebebbf22':
  Deduplicate code starting JDWP thread

Merge "Prepare field watchpoint support"

Prepare field watchpoint support

Adds field read/write events in the instrumentation. The debugger now registers
as a listener for these events so JDWP field access and field modification
events can be reported.

This CL will be followed by another one to report these events from the
interpreter. Therefore no JDWP field access and field modification events
can be sent for now.

Bug: 8267708
Change-Id: If2a93eb590805567d69015c83cce9cd2ab712cbd

Merge "Clear unused roots from debugger"

Merge "Deduplicate code starting JDWP thread"

am 30143fda: Merge "Fix clang build."

* commit '30143fdafa238f9d6922b056931a2fbe005129f6':
  Fix clang build.

Merge "Fix clang build."

Fix clang build.

Change-Id: Iaf0b76cfc02bac5f49ab98d494db9a774211f453

am 949e5fdd: Merge "art_quick_unlock_object uses comparison incorrectly"

* commit '949e5fdd1e46935ca770ac64e0a030a0a0f096a7':
  art_quick_unlock_object uses comparison incorrectly

Merge "art_quick_unlock_object uses comparison incorrectly"

am d6b9b627: Merge "Refactor space bitmap to support different alignments."

* commit 'd6b9b62748e120f7a02eb315e5cc8179312a5d1a':
  Refactor space bitmap to support different alignments.

am 107cbcc0: Merge "Reduce bump pointer space capacity."

* commit '107cbcc013df137cdad2a2eb238607b7673c5819':
  Reduce bump pointer space capacity.

Merge "Refactor space bitmap to support different alignments."

Merge "Reduce bump pointer space capacity."

Reduce bump pointer space capacity.

Bug: 14058912
Change-Id: I7ae53e667539530d56ca2f5d10214e69c9c994e2

am 0cedc9b2: Merge "Recognize and ignore options \'-ea\' and \'-da\'"

* commit '0cedc9b2fda88305b5291dee78673197239f2b93':
  Recognize and ignore options '-ea' and '-da'

Merge "Recognize and ignore options '-ea' and '-da'"

Centralize instruction-set pointer-size, alignment, 64b-width code
in instruction_set.h/cc

This allows to clean up some places that currently make explicit
comparisons.

Change-Id: I0dcc924c52fa53306f706aceea93a2d4a655c5df

Recognize and ignore options '-ea' and '-da'

-ea:<package or class> and -da:<package or class> options are ignored.

This patch just adds -ea and -da options without any package or
class specified.

Change-Id: Ide04b8eacb1b2edb415328130bc12e05dc3ddc7b
Signed-off-by: Yevgeny Rouban <yevgeny.y.rouban@intel.com>

Refactor space bitmap to support different alignments.

Required for:
Using space bitmaps instead of std::set in mod union table +
remembered set.
Using a bitmap instead of set for large object marking.

Bug: 13571028

Change-Id: Id024e9563d4ca4278f79607cdb2f81895121b113

Deduplicate code starting JDWP thread

Also waits for thread_start_cond_ in a loop and removes needless mutex lock in
JdwpState::Run when checking thread state.

Change-Id: I6fca2151b5343b2906e9fa3b439029e6ad5b086d

Clear unused roots from debugger

SingleStepControl and DebugInvokeReq hold roots which are only used during a
limited amount of time. Clear these roots once we finish using them.

Also cleans some code around single-step.

Change-Id: Ica2f65e04a4d60272199490cd40ad1803d3b9f02

am 043a7a61: Merge "Fix hammerhead phone boot on "

* commit '043a7a6182870964021476484b1534106ff20600':
  Fix hammerhead phone boot on master.

Merge "Fix hammerhead phone boot on master."

Fix hammerhead phone boot on master.

Wasn't failing on AOSP.

Change-Id: I8e531bf7493d36dc2667afeaf3133ce8560a2ce1

am 95cd0662: Merge "Fix heap verification with SS/GSS."

* commit '95cd06622b956d7cd5b65248b1f381e602753f2c':
  Fix heap verification with SS/GSS.

Merge "Fix heap verification with SS/GSS."

Fix heap verification with SS/GSS.

Since I had removed the SwapSemiSpaces it was trying to verify
references in the from space of the semispace collector, which isn't
necessarily valid.

Bug: 13989162

Change-Id: I8e0b7119a1a53d63f2c71ecdb276290eda5bae07

am 24801569: Merge "Fix an occasional ThreadStress crash."

* commit '248015696ee087878656fb53e37cbf7be2b97e4d':
  Fix an occasional ThreadStress crash.

am 37df2868: Merge "Clean-up heap spaces."

* commit '37df286803b39fe559dd0fe342a661115d96ae57':
  Clean-up heap spaces.

Merge "Fix an occasional ThreadStress crash."

Merge "Clean-up heap spaces."

Fix an occasional ThreadStress crash.

Also, fix the NativeAllocations test for the case with the GSS
collector as the default GC. Since kGcCauseForAlloc (incorrect) was
being passed into the collector instead of kGcCauseForNativeAlloc
(correct) from Heap::RegisterNativeAllocation(), the GCC collector
never invoked a whole heap collection which was necessary to do
sufficent finalizations to pass the test.

Bug: 13988451
Bug: 11650816
Change-Id: Ib02f061751cd777e0d3bfa81b29e04a874dc58a0

Clean-up heap spaces.

We now use the CMS collector instead of the semispace collector when
the phone is booting. We still perform compaction during the zygote
space creation. This reduces time spent in GC by ~2s during boot
and doesn't affect zygote space size.

Changed the space creation logic to create the temp space when a
background transition occurs.

Added a flag to each space which is true if you are allowed to
move objects that are within this space.

Removed SwapSemiSpaces call from the semi space collector, it is now
the job of the caller to do this with threads suspended. This
simplifies the logic in the zygote compaction / heap transition code
since these do not copy from one semispace to another.

Added Space::Clear to RosAllocSpace and DlMallocSpace. This greatly
simplifies the code used for collector transitions.

Time spent in GC creating zygote space:
Before: 3.4s, After: 1.28s
No change in zygote space size.

Bug: 13878055

Change-Id: I700348ab7d5bf3aa537c0cd70c0fed09aa4b0623

am 9b417e4f: Merge "AArch64: List arm64 as instruction set option"

* commit '9b417e4f0f87da6bfe8dc5f02c987acfcb6dca31':
  AArch64: List arm64 as instruction set option

Merge "AArch64: List arm64 as instruction set option"