am 5d4430b9: Merge "[optimizing] trivial x86 explicit null check fix"

* commit '5d4430b94446ea303327a311ad153767ed689863':
  [optimizing] trivial x86 explicit null check fix

Merge "[optimizing] trivial x86 explicit null check fix"

am 2910e15a: Merge "Fix oat_file_assistant_test\'s ReserveImageSpace()."

* commit '2910e15a5c67821c935e310ca51772d82095def2':
  Fix oat_file_assistant_test's ReserveImageSpace().

[optimizing] trivial x86 explicit null check fix

Change a cmp reg,0 to test reg,reg.  I don't know that this code
is even invoked.

Change-Id: Ifddffcb22d8a4060b7abbea17d8e7168535e409b
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>

Merge "Fix oat_file_assistant_test's ReserveImageSpace()."

am ebbb1e32: Merge "Use variable encoding for StackMap."

* commit 'ebbb1e322d8c89e69424a543faa03402e5b63673':
  Use variable encoding for StackMap.

Merge "Use variable encoding for StackMap."

Use variable encoding for StackMap.

dex_pc, native_pc_offset, dex_register_map_offset, inline_info_offset,
and register_mask can now be encoded in 1, 2, 3, or 4 bytes.

Change-Id: I15f93e8226ce374204d44c5a80a9fd89bda2687c

am 5942d57e: Merge "Revert "Add small library for writing of DWARF data.""

* commit '5942d57ed5f727d357787aedc089e51aed8eaefa':
  Revert "Add small library for writing of DWARF data."

Merge "Revert "Add small library for writing of DWARF data.""

Revert "Add small library for writing of DWARF data."

objdump differences on the bots

This reverts commit 75c3d61011a06c1253e773c42e53ee6f137dbe53.

Change-Id: Ie5947a27b9fe5de9d5f0838a024b5cf646ec701e

am f1c2b1f3: Merge "ART: Inline MemoryRegion operations"

* commit 'f1c2b1f395b8d47aa54a3b7a4ef1329eeb8af5d4':
  ART: Inline MemoryRegion operations

Merge "ART: Inline MemoryRegion operations"

am 7411a699: Merge "Use unique location catalogs to encode Dex register maps."

* commit '7411a69982c055692966c685636b6a226f10215e':
  Use unique location catalogs to encode Dex register maps.

ART: Inline MemoryRegion operations

Functions of the MemoryRegion class method are sufficiently simple
yet do not get inlined. Since these methods are called frequently
during stack map generation, inlining them can save a non-negligible
amount of compile time. This patch inserts attributes to force this.

Change-Id: I8ab4f480103fd536f61f52b805864d91bf58428e

Merge "Use unique location catalogs to encode Dex register maps."

am ba623352: Merge "Ensure MemoryRegion::Load & MemoryRegion::Store are word-aligned."

* commit 'ba6233527e455368bba02efe963dd1e24d971727':
  Ensure MemoryRegion::Load & MemoryRegion::Store are word-aligned.

Use unique location catalogs to encode Dex register maps.

- For each CodeInfo object (i.e. for each method), compute a
  variable index size location catalog  of unique Dex
  register locations.  In Dex register maps, instead of
  storing the actual location of a (live) Dex register,
  store the index of the location catalog entry containing
  that Dex register location.
- Adjust art::StackMapStream,
  art::CheckReferenceMapVisitor::CheckOptimizedMethod,
  art::StackVisitor::GetVRegFromOptimizedCode, and
  art::StackVisitor::SetVRegFromOptimizedCode.
- Add LoadBits and StoreBits methods to art::MemoryRegion
  to load and store a block of adjacent bits in a memory
  region.
- Update compiler/optimizing/stack_map_test.cc.
- Remove the default value for parameter EmptyFn of
  art::HashMap.  This default value did not seem to make
  sense, as it would create an "empty function" for type Key
  by default, whereas art::HashMap expects an "empty
  function" for type std::pair<Key, Value>.

Change-Id: Id9e49d7756c253ce41c36630cd832208d06c2e28

Merge "Ensure MemoryRegion::Load & MemoryRegion::Store are word-aligned."

Ensure MemoryRegion::Load & MemoryRegion::Store are word-aligned.

Change-Id: Ib19fb93abec4789a111dfd48fcac6065e2d3ec82

am 46653009: Merge "Add small library for writing of DWARF data."

* commit '466530095eae3f3e8a13bd41b5489450b2e66c0a':
  Add small library for writing of DWARF data.

Merge "Add small library for writing of DWARF data."

am 2d9cd63c: Merge "Tests still failing on buildbot\'s devices."

* commit '2d9cd63ca365331d02e3030c3ff799737a6f178a':
  Tests still failing on buildbot's devices.

Merge "Tests still failing on buildbot's devices."

Tests still failing on buildbot's devices.

Change-Id: I7085706884c64978eaef6ba9eb0900b9fe9e4112

Add small library for writing of DWARF data.

Change-Id: Ifa77aec7584bfcdcc05ddb049c160be9b6dda7c5

am 8eeecbc5: Merge "ART: General-case negation in boolean simplifier"

* commit '8eeecbc50b5f6f569b4d6d1a141624f0018ffb2d':
  ART: General-case negation in boolean simplifier

Merge "ART: General-case negation in boolean simplifier"

am e6842242: Merge "Update clean-oat-host after latest oat location changes."

* commit 'e684224201233ed35e15c3df8944bd3444984edf':
  Update clean-oat-host after latest oat location changes.

ART: General-case negation in boolean simplifier

Code transformations on the HGraph may optimize out the condition
instruction of an If and replace it with a boolean value. In such
case, the boolean simplifier would not know how to negate the
condition and would fail. This patch implements negation in this
general case with 'equals 0' as a substitute for the non-existing
boolean Not instruction.

Bug: 19992954
Change-Id: I152036fcc6bbecccc767d3024a5c060177597d88

Merge "Update clean-oat-host after latest oat location changes."

am c4bd0e6a: Merge "Workaround for network issues: use 127.0.0.1."

* commit 'c4bd0e6a7f4839ea99222f06979cc2369cb9bf10':
  Workaround for network issues: use 127.0.0.1.

Merge "Workaround for network issues: use 127.0.0.1."

Workaround for network issues: use 127.0.0.1.

Change-Id: I3c0ed810165bc8945c1dfaa028eb9a7bed674281

am d9269b97: Merge "Fix 099-vmdebug and 802-deoptimization art tests for Mips."

* commit 'd9269b9768101d4b1c82bd99561731d5d0083dc7':
  Fix 099-vmdebug and 802-deoptimization art tests for Mips.

Merge "Fix 099-vmdebug and 802-deoptimization art tests for Mips."

Fix 099-vmdebug and 802-deoptimization art tests for Mips.

There are 2 bugs that are causing these 2 tests to fail the same way.
The first one is we should be using $t9 for function calls so $gp can
be calculated correctly. The second bug is there can't be a gap between
the quick frame and the callee save frame, otherwise the WalkStack()
function will get confused and crash.

Bug: 19003184
Change-Id: I3c545ce18268deb73150fca2a7d7a798540f1cf2

am e33d0cb5: Merge "ART: Don\'t fail on unsupported conditions in boolean simplifier"

* commit 'e33d0cb5b88703302e3912ca96dabcb06fea4340':
  ART: Don't fail on unsupported conditions in boolean simplifier

Merge "ART: Don't fail on unsupported conditions in boolean simplifier"

ART: Don't fail on unsupported conditions in boolean simplifier

Skip simplification instead of FATALing when an unsupported condition
is found.

Bug: 19992954
Change-Id: Ie2845bead72da63018734e6dd91ce65824658b39

Fix oat_file_assistant_test's ReserveImageSpace().

Bug: 19963714
Change-Id: I9a2db293943cc81592defa8ddc7221bcd36cfdfb

am c4a12419: Merge "Disable host testing for now."

* commit 'c4a12419a3a607716de7f3c9f4c3e004d9362484':
  Disable host testing for now.

Merge "Disable host testing for now."

Disable host testing for now.

Buildbot machines hare not happy with gethostbyname.

Change-Id: I9dd878b442686a3d1dbcad54dbca7aabc4adbdf3

am c06ff145: Merge "On host, say the debuggee needs to be run through bash."

* commit 'c06ff145cfdd8019fbcb68c0554e21df5f95fe27':
  On host, say the debuggee needs to be run through bash.

Merge "On host, say the debuggee needs to be run through bash."

On host, say the debuggee needs to be run through bash.

Change-Id: I2cf7d3123e9d184f40cbaf49f8ac69e1481cc7a4

am 392351a8: Merge "[optimizing] Do not inline intrinsics"

* commit '392351a8aa9c6aec288050ef3f1f5471305b27c1':
  [optimizing] Do not inline intrinsics

Merge "[optimizing] Do not inline intrinsics"

am cf8b2c21: Merge "ART: Arm32 optimizing compiler backend should honor sdiv"

* commit 'cf8b2c21a81959db7fae0868ad02e9234db751c4':
  ART: Arm32 optimizing compiler backend should honor sdiv

Merge "ART: Arm32 optimizing compiler backend should honor sdiv"

ART: Arm32 optimizing compiler backend should honor sdiv

We still support architectures that do not have sdiv.

Issue: https://code.google.com/p/android/issues/detail?id=162257
Change-Id: I6d43620b7599f70a630668791a796a1703b62912

am acb4ea0a: Merge "Implement a proper solution for temps."

* commit 'acb4ea0a7f7a0b75cf6eddd619a3e6002b46347e':
  Implement a proper solution for temps.

Merge "Implement a proper solution for temps."

am 5bb214b6: Merge "Please vogar by not giving a vm-command on host."

* commit '5bb214b6cd25809de2a4d62607ce69cd16587678':
  Please vogar by not giving a vm-command on host.

am 618cc856: Merge "ART: Refactor callbacks_ in common_runtime_test"

* commit '618cc856c0af05ea3b162bd8490999e54a2abc1b':
  ART: Refactor callbacks_ in common_runtime_test

Merge "Please vogar by not giving a vm-command on host."

Merge "ART: Refactor callbacks_ in common_runtime_test"

Please vogar by not giving a vm-command on host.

If the host is configured with shell (and not bash or mksh), the 'art'
script cannot be run by vogar.

Change-Id: Ib88cea2e45de1ff54477ae8ec3a8e831155d86df

am 547bfb87: Merge "[optimizing] Only print full stats in VLOG(compiler)"

* commit '547bfb87696fb53a87d4f4418da86938179af9cd':
  [optimizing] Only print full stats in VLOG(compiler)

Merge "[optimizing] Only print full stats in VLOG(compiler)"

[optimizing] Only print full stats in VLOG(compiler)

Also add a kCompiledQuick stat.

bug:19956318

Change-Id: I0c6c0ac256362824eb9f1418216ab7e2c289a17b

am 69aea147: Merge "Fix user-build on fugu."

* commit '69aea147263055e5fd4cc63f643999883aa5c88a':
  Fix user-build on fugu.

Merge "Fix user-build on fugu."

Fix user-build on fugu.

Calling Delete on an array shifts the elements, so when iterating
over inactives and removing entries we need to decrement for
the found interval, but also its potential other half. The code
used to not decrement for the other half

Change-Id: Idcb1533643c11a37ed4f459fe88aaef208a4bfd6

am fcee9d4a: Merge "Tweak test script while investigating failures."

* commit 'fcee9d4ad44b70cb85aafa9056c08a194c5582e6':
  Tweak test script while investigating failures.

Merge "Tweak test script while investigating failures."

Tweak test script while investigating failures.

- Disable runs on devices failing,
- Add verbose for host failures.

Change-Id: I4efecfbc40f6666e62fd0835257d5b6c3cdcc798

am 4bf34ade: Merge "JDWP: more checking for StackFrame commands"

* commit '4bf34ade61a5dd1537781764aa38acbde5be6fc9':
  JDWP: more checking for StackFrame commands

Merge "JDWP: more checking for StackFrame commands"

ART: Refactor callbacks_ in common_runtime_test

The callback is now important for some runtime checks, and just
resetting callbacks_ is not correct anymore.

Change-Id: I2cb806f1916fd5ea190911a833ce460be557a428

am a3d40d5f: Merge "Fix some reflection errors"

* commit 'a3d40d5f764adfde8fa40d826cd93ba36cd15437':
  Fix some reflection errors

Merge "Fix some reflection errors"

Fix some reflection errors

Fixed incorrectly using 2nd frame instead of 1st in VerifyAccess.
Added regression test to ART. Fixed broken setShort,
getDeclaredFieldInternal.

Change-Id: I4b21d52d998cb768fe9503b8bccec506b7b972e5

am af38d896: Merge "ART: Use enums in Trace"

* commit 'af38d89634edbef394b3ef2cb3390d08dfb8d939':
  ART: Use enums in Trace

Merge "ART: Use enums in Trace"

am f381645a: Merge "Add AccessibleObject and Field to mirror"

* commit 'f381645a336f7092ab6f5900c0a2cf183a9dbdf7':
  Add AccessibleObject and Field to mirror

Merge "Add AccessibleObject and Field to mirror"

ART: Use enums in Trace

Move away from booleans. Will make introduction of streaming mode
a bit easier / obvious.

Change-Id: Id7ae92f6b97f627e848510d473931537d7db0db8

Add AccessibleObject and Field to mirror

Main motivation is to remove all the functionality / field access on
java side to ArtField. Also comes with some reflection speedups /
slowdowns.

Summary results:
getDeclaredField/getField are slower mostly due to JNI overhead.
However, there is a large speedup in getInt, setInt,
GetInstanceField, and GetStaticField.

Before timings (N5 --compiler-filter=everything):

                       benchmark      ns linear runtime
          Class_getDeclaredField  782.86 ===
                  Class_getField  832.77 ===
                    Field_getInt  160.17 =
                    Field_setInt  195.88 =
                GetInstanceField 3214.38 ==============
                  GetStaticField 6809.49 ==============================

After:
          Class_getDeclaredField 1068.15 ============
                  Class_getField 1180.00 ==============
                    Field_getInt  121.85 =
                    Field_setInt  139.98 =
                GetInstanceField 1986.15 =======================
                  GetStaticField 2523.63 ==============================

Bug: 19264997

Change-Id: Ic0d0fc1b56b95cd6d60f8e76f19caeaa23045c77

am 68e22f3b: Merge "ART: Some runtime cleanup"

* commit '68e22f3b982ff9ccbdfb3b65b7cfc16fcae907ba':
  ART: Some runtime cleanup

Merge "ART: Some runtime cleanup"

am d6c93101: Merge "Fix typos spotted by Andreas."

* commit 'd6c93101f81fbdcd68b0fa2965d8af3877d15ea4':
  Fix typos spotted by Andreas.

Merge "Fix typos spotted by Andreas."

ART: Some runtime cleanup

Use an enum for the compiler-callback mode.

Refactor and remove some unnecessary includes in runtime.h.

Change-Id: If2245fa470171311b8e05b677cf6bb28f209585a

am e2c29f4a: Merge "ART: Use instrumented Alloc in Class-Linker"

* commit 'e2c29f4a2717923f190c944a06e02f7d8c3ddcb6':
  ART: Use instrumented Alloc in Class-Linker

Merge "ART: Use instrumented Alloc in Class-Linker"

ART: Use instrumented Alloc in Class-Linker

To support Valgrind over dex2oat and gtests, CreatePathClassLoader
must use instrumented allocation.

Change-Id: I9dc2d009d37bec598c5de67688ebf9810da6e899

am 86787d0b: Merge "ART: PathClassLoader for compiler"

* commit '86787d0bc2c5ad3b9c3b880b0fe2325d2b8bd672':
  ART: PathClassLoader for compiler

Merge "ART: PathClassLoader for compiler"

ART: PathClassLoader for compiler

Use an actual PathClassLoader when compiling apps, instead of a
side structure and cutout.

This CL sets up a minimal object 'cluster' that recreates the Java
side of a regular ClassLoader such that the Class-Linker will
recognize it and use the internal native fast-path.

This CL removes the now unnecessary compile-time-classpath and
replaces it with a single 'compiling-the-boot-image' flag in the
compiler callbacks.

Note: This functionality is *only* intended for the compiler, as
the objects have not been completely initialized.

Bug: 19781184

Change-Id: I7f36af12dd7852d21281110a25c119e8c0669c1d

am 5c7acfc0: Merge "[MIPS] Refactoring code for quick compiler"

* commit '5c7acfc0b5acc50c60c57934341750f7c776d57e':
  [MIPS] Refactoring code for quick compiler

Merge "[MIPS] Refactoring code for quick compiler"

[optimizing] Do not inline intrinsics

The intrinsics generally have specialized code and the code for them
may be faster than what can be achieved with inlining. Thus inliner
should skip intrinsics.

At the same time, easy methods are not worth intrinsifying: ie String
length and isEmpty. Those can be handled by inliner with no problem
and can actually lead to better code since call is not kept around
through all of the optimizations.

Change-Id: Iab38e6c33f79efa54d845d4871cf26fa9b235ab0
Signed-off-by: Razvan A Lupusoru <razvan.a.lupusoru@intel.com>

Implement a proper solution for temps.

We used to play some trickery when updating locations of temps. This
change creates a proper use of the temp, and use it for updating
its location.

Change-Id: I53e9447b87a55137a3a79841db21ad3864854825

Fix typos spotted by Andreas.

Change-Id: I564b4bc5995d91f4c6c4e4f2427ed7c279cb8740

am d68cef75: Merge "Re-enable test."

* commit 'd68cef75a2b57b5c5ab33065d567446b4510b143':
  Re-enable test.

Merge "Re-enable test."

Re-enable test.

The boot image can now be compiled with optimizing.

Change-Id: Idbdec44ccb01e3b0251914da150d9dd0854c61e6

[MIPS] Refactoring code for quick compiler

Code from compiler/dex/quick/mips64 is merged with code
in mips folder.

Change-Id: I785983c21549141306484647da86a0bb4815daaa