Merge "Improve gethostbyname_r use in art::JDWP::JdwpSocketState::Establish."

Merge "Reduce app image log spam"

Reduce app image log spam

Change to VLOG or file exists.

Bug: 26782386
Bug: 22858531
Change-Id: Ic73d110d44216c2846e4ea789070bea2af8043a3

Merge "Revert "Revert "Load app images"""

Revert "Revert "Load app images""

This reverts commit 1bc977cf2f8199311a97f2ba9431a184540e3e9c.

Bug: 22858531

Change-Id: Ide00bf3a73a02cba3bb364177204ad1b13f70295

Merge "Only create non-moving space at fixed address if zygote"

Only create non-moving space at fixed address if zygote

Pulled from app image CL, hopefully fixes dex2oat asan failure.

Change-Id: I022f7509b534679e5a4ef99ec7a0c622472cf740

Improve gethostbyname_r use in art::JDWP::JdwpSocketState::Establish.

Try a range of values for the work buffer size, instead of
using a fixed size.

Also, turn a couple of constants defined as macros into
constexprs.

Change-Id: I2354bfb51935bb0cad775c13d6cae06902b4a934

Merge "Use dlmalloc from external/dlmalloc instead of bionic/."

Merge "Revert "Add option to generate compressed backtrace info.""

Revert "Add option to generate compressed backtrace info."

This reverts commit 5fdcc3c931b70204fd8c491afa66f57f8428490f.

Change-Id: I9c1f5aad6933a46af6717e3a90a51f76111f9c8a

Merge "Fix MIPS64 booting problem"

Merge "Add option to generate compressed backtrace info."

Merge "Revert "Optimizing: double-negated bitwise operations simplifications""

Revert "Optimizing: double-negated bitwise operations simplifications"

Fails compiling the Wallet.apk with:

dex2oatd F 40736 41007 art/compiler/optimizing/optimizing_compiler.cc:194] Error after instruction_simplifier: art::SSAChecker: Instruction Add:59 in block 4 does not dominate use Or:153 in block 4.

This reverts commit 96798493170521691d709be50dd2102ead47b083.

Change-Id: Ia4b02e62e6133aa104f5db12ba82d5561b6fc090

Merge "Revert "X86: Use the constant area for more operations.""

Revert "X86: Use the constant area for more operations."

Hits a DCHECK:

dex2oatd F 19461 20411 art/compiler/optimizing/pc_relative_fixups_x86.cc:196] Check failed: !invoke_static_or_direct->HasCurrentMethodInput()

This reverts commit dc00454f0b9a134f01f79b419200f4044c2af5c6.

Change-Id: Idfcacf12eb9e1dd7e68d95e880fda0f76f90e9ed

Merge "Optimizing: double-negated bitwise operations simplifications"

Merge "X86: Use the constant area for more operations."

Merge "Revert "Load app images""

Revert "Load app images"

Fails when a method is duplicated (see test 097-duplicate-method)

Bug: 22858531

This reverts commit f7fd970244f143b1abb956e29794c446e4d57f46.

Change-Id: Ib30ae5be00cc568e799290be6b3c8f29cbbe4c20

Merge "Disable the CC collector verbose log."

Merge "Temporarily disable some libcore tests for the CC collector."

Disable the CC collector verbose log.

To reduce the amount of buildbot libcore test logs.

Bug: 12687968
Change-Id: Idadca6622c745ef12629ccb26dcbdc9ebc379a00

Temporarily disable some libcore tests for the CC collector.

Bug: 26711853
Bug: 12687968
Change-Id: Id791ac14afd7aff74f92aff88adefe4dcad7a20f

Merge "Bump image version"

Bump image version

Version bump in previous CL was rebased out.

Change-Id: Ifcd76b852716cc2a6c08287e83b9861621df7605

Merge "Load app images"

Load app images

Support in-place patching of the app image based on boot image
location and app oat location. Only loads for art run test so far
since we do not automatically generate app images for app installs.

N5 maps launch time (~200 runs):
Before: 930ms
After: 878.18ms
After + image class table: 864.57ms

TODO:
Oatdump support.
Store class loaders as class roots in image.

Bug: 22858531

Change-Id: I9cbc645645e62ea2ed1ad8e139e91af7d88514c1

X86: Use the constant area for more operations.

Allow FP HNeg to use the constant area to hold the constant to flip the
sign bit.

Enhance some math intrinsics to allow the use of the constant
area: Abs{Float,Double}, {Min,Max}{FloatFloat,DoubleDouble}.

Allow compares of floats/doubles to constants using the constant area.

These eliminate almost all uses of loading constants from the stack.

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

Merge "Raise ulimit for run-test 961."

Use dlmalloc from external/dlmalloc instead of bionic/.

Bug: http://b/17207577
Change-Id: I86d731e5236947c2cc21cbd17de29bd310a60a5b

Raise ulimit for run-test 961.

Run-test 961 was rubbing against the ulimit, causing spurious
failures. This raises the ulimit the test runs under.

Bug: 26733951

Change-Id: I43f7c5f8e5e9e23ed139bd1fcc6bf24c526c546b

Add option to generate compressed backtrace info.

Add flag --generate-mini-debug-info which generates
LZMA compressed .symtab and .debug_frame, which are
sufficient to print java backtraces in libunwind.

If enabled, it increases the size of boot.oat by about 3.5%.

Change-Id: Ic3c2ef7704c05fa328720c6781ca2a9b8e3935a3

Merge "Improve documentation and assertions of read barrier instrumentation."

Merge "Allow x86 memory operands for 64-bit popcnt on x86."

Merge "Generate unique ID for each type during native debugging."

Generate unique ID for each type during native debugging.

This allows the native debugger to find dynamic types of variables.

Change-Id: I901022b7db7d3c1db9f4b678ebafcb4eefed4ba7

Merge "Create helper class for DWARF expressions."

Optimizing: double-negated bitwise operations simplifications

Generic instruction simplifications applying to bitwise operations when
both inputs are Not's. And and Or are handled by De Morgan's laws,
removing one instruction:
~a & ~b -> ~(a | b)
~a | ~b -> ~(a & b)
Xor is handled by this trivial relation, removing two instructions:
~a ^ ~b = a ^ b

The simplifications only happen when neither Not is used by other
instructions.

Change-Id: I5d5187af2f625c475c3e49466af6bc3e87595f8f

Create helper class for DWARF expressions.

This ensures we generate valid DWARF opcodes and
it slightly simplifies the debug info writer.

Change-Id: I2ef8a9222c9a129c58f500741acd209b67e2ecff

Merge "Revert "Revert "ARM64 Baker's read barrier fast path implementation."""

Fix MIPS64 booting problem

Add missing MarkGCCard in compareAndSwapObject intinsic.

Additionaly, don't do a null test in MarkGCCard if the value
cannot be null for MIPS64.

Change-Id: Iad50f9e6be8cd27fedb31abb00d5829498941696

Merge "Revert "Revert "Fast ART x86 interpreter"""

Merge "Revert "Revert "Inline methods with loops."""

Revert "Revert "Inline methods with loops.""

Bug: 26689526

This reverts commit 451ad8d1be9a1949ea3c3e3a713a9e76198a8b2d.

Change-Id: If484fe4c0744254dd7568fd5006e574d621a1855

Merge "Revert "Revert "Revert "Change condition to opposite if lhs is constant""""

Revert "Revert "Revert "Change condition to opposite if lhs is constant"""

Fails two checker tests:

458-checker-instruction-simplification
537-checker-jump-over-jump

This reverts commit 884e54c8a45e49b58cb1127c8ed890f79f382601.

Change-Id: I22553e4e77662736b8b453d911a2f4e601f3a27e

Improve documentation and assertions of read barrier instrumentation.

For ARM, x86, x86-64 back ends.  The case of the ARM64 back
end is already handled in
https://android-review.googlesource.com/#/c/197870/.

Bug: 12687968
Change-Id: I6df1128cc100cbdb89020876e1a54de719508be3

Revert "Revert "ARM64 Baker's read barrier fast path implementation.""

This reverts commit 28a2ff0bd6c30549f3f6465d8316f5707b1d072f.

Bug: 12687968
Change-Id: I6e25c70f303368629cdb1084f1d7039261cbb79a

Merge "With irreducible loops, we can have a null destination."

With irreducible loops, we can have a null destination.

bug:26684047
Change-Id: I23d1c74caec4aaebc05362a588f6cfec2132bdc8

Merge "Disable flaky test."

Disable flaky test.

Change-Id: I003c2ab9ed0b0629a4b44c42a2139a99b05f40ae

Merge "Revert "Revert "Change condition to opposite if lhs is constant"""

Merge "Revert "Some minor simplifications in code and tests.""

Revert "Some minor simplifications in code and tests."

Fails 530-checker-loops on arm

This reverts commit bf03fcd10a3ffa15468d335f26697b0473e45b36.

Change-Id: I739aaaccd0509d02a62ef01e797a6d45bfe941df

Merge "Revert "ARM/ARM64: Extend support of instruction combining.""

Revert "ARM/ARM64: Extend support of instruction combining."

The test fails its checker parts.

This reverts commit debeb98aaa8950caf1a19df490f2ac9bf563075b.

Change-Id: I49929e15950c7814da6c411ecd2b640d12de80df

Revert "Revert "Change condition to opposite if lhs is constant""

This reverts commit a05cacc11fa075246c38497c01b949745fadc54b.

Change-Id: Ifdc261fd4dfb2c538017fe1d69af723aafd4afef

Merge "Update to INFO to not confuse adb logcat readers."

Merge "Fix cpu-info/cpp-defines mismatch."

Fix cpu-info/cpp-defines mismatch.

Rationale:
__POPCNT__ is the right way to cpp-test for popcnt support

Change-Id: I3dac215f862701e04519e89ca97c159b1e4385c8

Merge "Disable Homogeneous space for dex2oat"

Merge "Revert "Revert "run-test: Add tests for 1.8 reflection implementations"""

Merge "Use current pid not tid to create a backtrace map."

Allow x86 memory operands for 64-bit popcnt on x86.

By Mark's request.
This allows:

 popcnt edx, [esp + 16]
 popcnt ebx, [esp + 20]
 add ebx, edx
 .....

instead of the more elaborate

 mov edx, [ecx + 16]
 mov ebx, [ecx + 20]
 popcnt eax, edx
 popcnt ebp, ebx
 add ebp, eax

Change-Id: Iea30ad7b9a1aba8591e3dbb72e24ef12e81cc2ce

Merge "Support for x86 popcnt."

Revert "Revert "Fast ART x86 interpreter""

This reverts commit 2d093a1213cc2f85b5e5e02782332657c479eb94.

Disable x86 mterp compilation on Mac host builds (but keep enabled
for all target builds).

Change-Id: Ie355279f166d2964a786646ee53f065b7e0f5ede

Support for x86 popcnt.

Change-Id: I0fc4e745764f1749a6437a199a594f3d8ea53eef

Merge "Add a METHOD_SUPER to the MethodType enum."

Merge "Revert "ARM64 Baker's read barrier fast path implementation.""

Revert "ARM64 Baker's read barrier fast path implementation."

This reverts commit c8f1df9965ca7f97ba9e6289f8c7a717765a59a9.

This breaks master.

Change-Id: Ic07f602af8732e2835bd11f65e3b9e766d3349c7

Revert "Revert "run-test: Add tests for 1.8 reflection implementations""

This reverts commit 6a52d29eadb9d0a882439202303e8eae9f13bd26.

Change-Id: I05310a6e07ddcc9b2777b7df8f23e02498775256

Merge "Implemented BitCount as an intrinsic. With unit test."

Merge "Some minor simplifications in code and tests."

Merge "Add attribute to fix build."

Add attribute to fix build.

Change-Id: I833b2e55ae7cc312404bd07b98ddf89a357bc786

Merge "Move backward branch instrumentation to all branch."

Merge "ARM64 Baker's read barrier fast path implementation."

Merge "Improves recent changes to art::JDWP::JdwpSocketState::Establish."

Merge "ARM/ARM64: Extend support of instruction combining."

Move backward branch instrumentation to all branch.

This is in preparation for osr, and later dead branch profiling.

Change-Id: I4597c0784c894c310f295dd534967d67dcefa419

Improves recent changes to art::JDWP::JdwpSocketState::Establish.

(This is a follow-up CL to
https://android-review.googlesource.com/#/c/197630.)

Capture the size of the work buffer passed to
gethostbyname_r in a constant and document it.

Also wrap long lines in runtime/jdwp/jdwp_socket.cc.

Change-Id: Ieb0694c56611a491c5f949ce8ed22ab3795f1883

Merge "Revert "Revert "Write dex files to oat file early."""

Merge "Revert "run-test: Add tests for 1.8 reflection implementations""

Revert "run-test: Add tests for 1.8 reflection implementations"

Bug: 24618811

Too many failures on goto/ab

This reverts commit 54c4e0d7adbf9179df04f5ed5d8d5a3ff69add84.

Change-Id: Ic6305c14d78b38e5c13ea42e1777ad197ff2f45c

Revert "Revert "Write dex files to oat file early.""

This reverts commit 919f5536182890d2e03f59b961acf8f7c836ff61.

Fix linker error (Mac build):
Replace inline definition of art::ZipArchive::~ZipArchive()
with an out-of-line definition in zip_archive.cc to avoid
direct reference to CloseArchive() from libart-compiler due
to inlining. Note that libart is linked against -lziparchive
but libart-compiler is not.

Change-Id: I92620ea0200282ca7ba9b7f61a592cb6468d90d8

Merge "Fix JDWP host tests on Trusty."

Fix JDWP host tests on Trusty.

The buffer passed by art::JDWP::JdwpSocketState::Establish
to gethostbyname_r on Trusty was too small, leading to the
following error message during JDWP host testing:

art/runtime/jdwp/jdwp_socket.cc:283] gethostbyname_r('localhost') failed: Resolver internal error

This CL doubles the size of this buffer to avoid this
failure.

Change-Id: Ie7bb1ea8c755d869f2cc50cc2f6cc82828499706

Merge "Let Art tests use the default jack version"

ARM/ARM64: Extend support of instruction combining.

Combine multiply instructions in the following way:
ARM64:
MUL/NEG -> MNEG
ARM32 (32-bit integers only):
MUL/ADD -> MLA
MUL/SUB -> MLS

Change-Id: If20f2d8fb060145ab6fbceeb5a8f1a3d02e0ecdb

Implemented BitCount as an intrinsic. With unit test.

Rationale:
Recognizing this important operation as an intrinsic has
various advantages:
(1) having the no-side-effects/no-throw allows for
    much more GVN/LICM/BCE.
(2) Some architectures, like x86_64, provide direct
    support for this operation.

Performance improvements on X86_64:
CheckersEvalBench (32-bit bitboard): 27,210KNS -> 36,798KNS  =  + 35%
ReversiEvalBench  (64-bit bitboard): 52,562KNS -> 89,086KNS  =  + 69%

Change-Id: I65d549b0469b7909b12c6611cdc34a8640a5751f

Merge "Fix missing case in ARM64 codegen."

Fix missing case in ARM64 codegen.

Rationale:
Rather than excluding conditions that are not handled,
changed the right-hand-side is zero optimized code
to list handled conditions explicitly instead.

bug=26689526

Change-Id: I636e01548659c579d9e318f07bda2c24a12371e5

Merge "Add stack trace info to hprof completion message"

Some minor simplifications in code and tests.

Rationale: fell through the cracks of previous "intrinsics" CL.

Change-Id: If8ada79dfd70bea991c11d2b18661b951b6c4cd4

Add a METHOD_SUPER to the MethodType enum.

This makes some of the code simpler by disambiguating between regular
invoke-virtual and invoke-super, which have slightly different
requirements.

Change-Id: I50e86e681bb63c617c043367feb5b89e2ddbfc00

Use current pid not tid to create a backtrace map.

The BacktraceMap::Create() function assumes that the passed in
parameter is the current pid not tid. If a tid is passed in, then
the logic that creates the local map in libunwind is not used
and can bypass all of the special handling of local maps. The result
can be crashes if this map data gets out of sync with the real maps
found in the process.

Bug: 26676205
Bug: https://code.google.com/p/android/issues/detail?id=199219
Change-Id: Ibab9f7a7b99648a2cdaa185958bf6ffc368079ce

Merge "JIT: Fix incorrect CHECK"