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."

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

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."

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."""

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"

Add stack trace info to hprof completion message

Bug: 25795907
Change-Id: Iba39ac9467ed57ee37c7dd17f1b21e28c7e556c7

Merge "Relax the time interval the profiling data is saved."

Relax the time interval the profiling data is saved.

Bug: 26080105
Change-Id: I7c19e25760f26a4750ccdf60b33762ab3aa45e7c

Merge "Revert "Inline methods with loops.""

Merge "Revert "Fix gtests after loop inlining support.""

Revert "Inline methods with loops."

This reverts commit 82fc9bb45dbf8ff728122fb7ab72d1eb7b2f4869.

Loop inlining exposes issues with BCE.

Bug: 26689526
Change-Id: Id9983d7f9d3c5579d91e56e4699d4d939517b2dc

Revert "Fix gtests after loop inlining support."

This reverts commit 0e342cd5ccf0f648e165fa4fd270b44dfbea8be6.

Loop inlining exposes BCE issues.

Bug: 26689526
Change-Id: Iab0f8f502d8b389af4efc2e99e28cae59c00ee9b

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

Revert "Write dex files to oat file early."

This reverts commit 625a64aad13905d8a2454bf3cc0e874487b110d5.

Breaks the Mac build:

Undefined symbols for architecture i386:
  "_CloseArchive", referenced from:
      ... in oat_writer.o
ld: symbol(s) not found for architecture i386

Change-Id: I21608bc51437834e1e6abde9bcbe5e7d9998197e

Merge "Fix gtests after loop inlining support."

Fix gtests after loop inlining support.

Change-Id: Id6288fff8911ec9545e83d97f90ae566e13a75af

Merge "ART: DCHECK still too strong for String NewInstance"

ART: DCHECK still too strong for String NewInstance

Even though the presence of a irreducible loop phi is the reason why
String.<init> may see its first input not being the NewInstance, it
may see a different phi from the same cycle. We have to relax the
corresponding DCHECK.

Bug: 26676472
Change-Id: Iba6fffa74833c57aad8aa07dc382ff6b93838de1

Merge "Write dex files to oat file early."

Let Art tests use the default jack version

(cherry picked from commit 261cf1b14a06c86e4a19972eabb5b43bc82bb717)

Change-Id: I7184a752b067e59fc28804cba92df62e713a0785

Merge "ART: Cannot assume String.<init> called on NewInstance"

Merge "Inline methods with loops."

ARM64 Baker's read barrier fast path implementation.

Introduce an ARM64 fast path implementation in Optimizing
for Baker's read barriers (for both heap reference loads and
GC root loads).  The marking phase of the read barrier is
performed by a slow path, invoking the runtime entry point
artReadBarrierMark.

Other read barrier algorithms continue to use the original
slow path based implementation, which has been renamed as
GenerateReadBarrierSlow/GenerateReadBarrierForRootSlow.

Bug: 12687968
Bug: 26601270
Change-Id: I60da15249b58a8ee1a065ed9be2c4e438ee17150

ART: Cannot assume String.<init> called on NewInstance

Irreducible loops create uneliminatable phis for all live vregs. This
breaks the StringFactory optimization which assumes that the first
input is always a NewInstance instruction.

Bug: 26676472

Change-Id: Ib7dfdadbafbbfef89e1f5b1a80eb75ecf792621a

Inline methods with loops.

Compiling Gms/Fb/Framework/Docs:
- Overall compilation-time increase: 2.2%
- Overall code size increase: 1.1%

Performance improvements:
- Richards with jit: +6%
- Takl: +11%

Change-Id: I0a6fcf2a360e5ad193cd95b5c4fe92227ac6bd96

Merge "Disable MemMapTest.MapAnonymousExactAddr32bitHighAddr on MIPS32."

Disable MemMapTest.MapAnonymousExactAddr32bitHighAddr on MIPS32.

This test case is failing on a Creator Ci20 development
board because of memory allocation limitations.

Bug: 26635783
Change-Id: I02f1012f8005fd516aac82412c71af7a4652a80c

Merge "Make sure that OfflineCompilationInfo is set to nullptr when needed."

Make sure that OfflineCompilationInfo is set to nullptr when needed.

Local variables weren't initialized when in tests and the early
return in ProcessProfiles didn't make sure that the output
parameter is set to nullptr.

Bug: 26080105
Change-Id: Id88a6abb515c3ab9a6ebac56bc9dac7920c3d58f

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

Merge "Revert "Fast ART x86 interpreter""

Revert "Fast ART x86 interpreter"

This reverts commit 99229c71efda9363faa571017c52a215c6e28f83.

Assembly syntax issue with the Mac build.

Change-Id: I271d18d31963fae46e4077ddd39313bec0b3a39e

run-test: Add tests for 1.8 reflection implementations

Add test for Method#isDefault for now, requires experimental lang-dev
jack version.

Bug: 24618811
Change-Id: If5a796fe5609ea13ec9b1565619a38b932462989

Merge "Remove dex_file_verifier test of newly allowed behavior."

Remove dex_file_verifier test of newly allowed behavior.

We previously tested that the dex_file_verifier would error out on
interface methods with code but default methods mean this test is no
longer accurate.

Bug: 24618811

Change-Id: I8ec50ebd93b4bb78acacfafcb6c33c92415a68eb

Merge "Change space base test to use parameters"

Merge "Fast ART x86 interpreter"

Change space base test to use parameters

Still need to convert remaining space tests to use gtest value
parameters.

Bug: 10245322
Change-Id: Iad3183d46f2fff1ec2aee6923c989433885adf81