am a943a75a: Merge "ART: Invoke Checker from run-test scripts"

* commit 'a943a75a890f43ff9517e13bb6c16de8bfa35d90':
  ART: Invoke Checker from run-test scripts

Merge "ART: Invoke Checker from run-test scripts"

am 452773fe: Merge "Address review comments."

* commit '452773fec59e033da4bb258a64bd65bd2ae658e8':
  Address review comments.

Merge "Address review comments."

Address review comments.

Comments were from:
https://android-review.googlesource.com/#/c/121992.

Change-Id: I8c59b30a356d606f12c50d0c8db916295a5c9e13

ART: Invoke Checker from run-test scripts

This patch moves Checker-based tests of the optimizing compiler into
the art/test directory and modifies the run-test scripts to dump
the CFG during compilation and to verify the graph using Checker as
part of the "running" stage. Outputs generated by running the test
and running Checker are concatenated and compared with expected.txt.

Checker is invoked only if the test's name name matches the format
"<number>-checker-*" and it's currently enabled only for optimizing +
host configs. The tests are still invoked on other configs but
without Checker.

Change-Id: Ib24da808cd4bca66f07e0dbeb913a418065f2859

am e210661a: Merge "ART: Fix compiler warning at arm/int_arm.cc:644"

* commit 'e210661a68e933cc6982368c24cca165eb61962f':
  ART: Fix compiler warning at arm/int_arm.cc:644

Merge "ART: Fix compiler warning at arm/int_arm.cc:644"

ART: Fix compiler warning at arm/int_arm.cc:644

Clang can report warning:
int_arm.cc:644:89: error: 'ops.art::ArmMir2Lir::EasyMultiplyOp::shift'
may be used uninitialized in this function
OpRegRegRegShift(kOpRsub, r_tmp1, r_src, r_src, EncodeShift(kArmLsl, ops[0].shift));

That warning becomes blocker for libart-compiler.so compilation.

This patch fixes the only case where 'shift' member was uninitialized.

Change-Id: I6428170994c9f97e7a3d85d752b97dfcdff0c8a4
Signed-off-by: Dmitry Petrochenko <dmitry.petrochenko@intel.com>

am d599b393: Merge "Checker cleanup"

* commit 'd599b39357cb0d3ab8b719357c86ffe609bb3fcb':
  Checker cleanup

Merge "Checker cleanup"

am 951d70b4: Merge "Quick: Clean up optimization pass order."

* commit '951d70b42400453f9d1746d155b8337c07c86acc':
  Quick: Clean up optimization pass order.

am e45ff20f: Merge "Constant fold after inlining."

* commit 'e45ff20fa341f88cc1c4a87089e0731914ccf7d0':
  Constant fold after inlining.

am 1235c46d: Merge "Do not use STMP, it conflicts with the calling convention."

* commit '1235c46dc5d5a329a9074e2168219a3e85ad6d48':
  Do not use STMP, it conflicts with the calling convention.

Merge "Quick: Clean up optimization pass order."

Merge "Constant fold after inlining."

Constant fold after inlining.

- Inlining opens up new opportunities for constant folding.
- Fix a bug in constant folder where the result type was not
  correctly set for the folding of a HCompare.
- Improve graph checker's coverage.

Change-Id: I0943bf8ff65505c4addc4a555a526b55e00b5268

Merge "Do not use STMP, it conflicts with the calling convention."

Do not use STMP, it conflicts with the calling convention.

Hard-float calling convention uses S14 and D7 for argument passing,
so we cannot use them.

Change-Id: I77a2d8c875677640204baebc24355051aa4175fd

am c2c7a33a: Merge "Do not use register pair in a parallel move."

* commit 'c2c7a33a25169cdf19a0dcf45ddb3747055c7296':
  Do not use register pair in a parallel move.

Merge "Do not use register pair in a parallel move."

am 212327a5: Merge "Adds the dexfuzz fuzzing tool to ART."

* commit '212327a5186ebce53840de55d8b80b89be1119e2':
  Adds the dexfuzz fuzzing tool to ART.

Merge "Adds the dexfuzz fuzzing tool to ART."

Checker cleanup

Added a comment, removed unused temp directory code.

Change-Id: I3df87e801e72ccd5a87c7dbc67335e9936623560

am 2bd68b77: Merge "Fix infinite loop in GenerateIdentityHashCode"

* commit '2bd68b77b5bf43e77304ef97e9d096b0ad0cab32':
  Fix infinite loop in GenerateIdentityHashCode

Merge "Fix infinite loop in GenerateIdentityHashCode"

Fix infinite loop in GenerateIdentityHashCode

Root Cause:
  If no one changes the seed, it will become infinite loop due to below condition
    (expected_value & LockWord::kHashMask) == 0

Solution:
  Changes the seed before entering the next loop

Added test.

Bug: 19046417

Change-Id: I7d1c377dd1bda780681514b24d61ebc776bc80ab

am fa048428: Merge "Fix refs_and_args callee save order for Mips32."

* commit 'fa04842818fe2d0086872dea311db14a732d52f0':
  Fix refs_and_args callee save order for Mips32.

Merge "Fix refs_and_args callee save order for Mips32."

Quick: Clean up optimization pass order.

Move the TypeInference pass to post-opt passes and make it
a PassMEMirSsaRep as we need to rerun the pass if the SSA
representation has changed. (Though we currently don't have
any pass that would require it.)

The results of MethodUseCount and ConstantPropagation passes
are used only in the BBOptimization and codegen and stay
valid across BBOptimization and SuspendCheckElimination, so
move them out of post-opt passes to just before the BBOpt
(and reverse the dependency between ConstantPropagation and
init reg locations passes).

Change-Id: If02c087107cef48d5f9f7c18b0a0ace370fe2647

am d16d76ac: Merge "Fix comment referring to nonexistent ElementDeleter"

* commit 'd16d76ac64b1b9ed17e9cc17a6afb85f16ce4b51':
  Fix comment referring to nonexistent ElementDeleter

Merge "Fix comment referring to nonexistent ElementDeleter"

am 12c03ac7: Merge "Add --no-clean to checker (keeps the generated files around)"

* commit '12c03ac7575db242a6f35739bb459e8277115da4':
  Add --no-clean to checker (keeps the generated files around)

Merge "Add --no-clean to checker (keeps the generated files around)"

Add --no-clean to checker (keeps the generated files around)

Change-Id: Iefef18f3456f79684077912e1f8b614d56e13ca6

Do not use register pair in a parallel move.

The ParallelMoveResolver does not work with pairs. Instead,
decompose the pair into two individual moves.

Change-Id: Ie9d3f0b078cef8dc20640c98b20bb20cc4971a7f

am 36740379: Merge "Remove test that is now invalid."

* commit '36740379b9b1c81b7eb06ea9c9df411d0a9a765e':
  Remove test that is now invalid.

Merge "Remove test that is now invalid."

Remove test that is now invalid.

It does not make sense to have moves to the same destination
within a single parallel move. There is now an explicit
DCHECK to prevent these situations.

Change-Id: I3a33e748579ecb53b95476e77a37164d2f145f5a

am 6a5f5b25: Merge "Fix a bug in the register allocator."

* commit '6a5f5b25b3866966175859bc20216c5519d8b029':
  Fix a bug in the register allocator.

Merge "Fix a bug in the register allocator."

Fix a bug in the register allocator.

When allocating a register blocked by existing intervals,
we need to split inactive intervals at the end of their
lifetime hole, and not at the next intersection. Otherwise,
the allocation for following intervals will not see
that a register is being used by the split interval.

Change-Id: I40cc79dde541c07392a7cf4c6f0b291dd1ce1819

am ddcaf45d: Merge "ART: Mips64 runtime support"

* commit 'ddcaf45db2874ffc37d2a8820e815db19a54c517':
  ART: Mips64 runtime support

Merge "ART: Mips64 runtime support"

ART: Mips64 runtime support

Interpret-only Mips64 runtime support.

Change-Id: Iee22d0c8c77105d9b2f03a67dc4e09957fe0ab0a

am 3e384652: Merge "Increase default timeout"

* commit '3e384652fe16127ce4c9c25d4fa44ce1c5a9dc9f':
  Increase default timeout

Merge "Increase default timeout"

Increase default timeout

Bug: 19027813
Change-Id: I13af576dc3e980dc82e466435f41b322e2cd873e

am 3ea0b9fb: Merge "Print more info in MarkSweep::VerifyRoot"

* commit '3ea0b9fbb6aa8c451a12ccfb7befd5a871be877e':
  Print more info in MarkSweep::VerifyRoot

Merge "Print more info in MarkSweep::VerifyRoot"

Print more info in MarkSweep::VerifyRoot

Refactored old root callback to use a new class called RootInfo.
RootInfo contains all the relevant info related to the root
associated with the callback. The MarkSweep::VerifyRoot function
now uses this info to print the StackVisitor's described location
if the GC root is of the type kRootJavaFrame.

Some other cleanup.

Example output:
E/art     (12167): Tried to mark 0x123 not contained by any spaces
E/art     (12167): Attempting see if it's a bad root
E/art     (12167): Found invalid root: 0x123 with type RootJavaFrame
E/art     (12167): Location=Visiting method
'void java.lang.Runtime.gc()' at dex PC 0xffffffff (native PC 0x0)
vreg=0

(cherry picked from commit 12f7423a2bb4bfab76700d84eb6d4338d211983a)

Bug: 18588862
Change-Id: Ic5a2781f704e931265ffb3621c2eab4b2e25f60f

am 9f06b194: Merge "ART: Remove compiler invocation from Checker"

* commit '9f06b1946ae3ef1d2fd75bbf7f0a288bc611fe58':
  ART: Remove compiler invocation from Checker

am 0223a19f: Merge "ART: Allow to compile interpret-only mips64 files"

* commit '0223a19fa4fc39ba89de2b87b03b98a5fd97778b':
  ART: Allow to compile interpret-only mips64 files

am 726078d2: Merge "ART: Optimizing compiler intrinsics"

* commit '726078d229fdb2857f3d8794fa50783f571cb7a2':
  ART: Optimizing compiler intrinsics

Merge "ART: Remove compiler invocation from Checker"

Merge "ART: Allow to compile interpret-only mips64 files"

ART: Allow to compile interpret-only mips64 files

Include enough infrastructure to allow cross-compiling for mips64,
interpret-only. This includes the instruction-set-features, frame
size info and utils assembler.

Also add a disassembler for oatdump, and support in patchoat.

Note: the runtime cannot run mips64, yet.

Change-Id: Id106581fa76b478984741c62a8a03be0f370d992

Merge "ART: Optimizing compiler intrinsics"

am 88c7c58e: Merge "ART: Only run addr2line in abort thread dumps"

* commit '88c7c58ecabe2174bc269bb8f41628348727460d':
  ART: Only run addr2line in abort thread dumps

am 5306f5d7: Merge "Fix false alarm on thread suspend timeout"

* commit '5306f5d7e0b4a6ebeacdb8c73bd4335577838cad':
  Fix false alarm on thread suspend timeout

Merge "ART: Only run addr2line in abort thread dumps"

ART: Remove compiler invocation from Checker

In anticipation of running Checker from run-test, this patch removes
code which compiles Java files as this will be handled by run-test.

It simultaneously adds support for verifying an entire directory of
annotated test files.

Change-Id: Id91c19c83fddade076c77789316d59f00893e63b

ART: Only run addr2line in abort thread dumps

Avoid addr2line overhead for SIGQUIT and other thread dumps.

Bug: 18933933
Change-Id: I81bb291ede870d1d14b30bb0afdc47830ed14e8b

Merge "Fix false alarm on thread suspend timeout"

Fix comment referring to nonexistent ElementDeleter

Bug: 18809837
Change-Id: I40ad727907be505b7cb63f3c4bba7f395aaec727

ART: Optimizing compiler intrinsics

Add intrinsics infrastructure to the optimizing compiler.

Add almost all intrinsics supported by Quick to the x86-64 backend.
Further intrinsics require more assembler support.

Change-Id: I48de9b44c82886bb298d16e74e12a9506b8e8807

am 420457e6: Merge "ART: Logging levels for Checker"

* commit '420457e6040184a6e1639a4c84fcc8e237bd8a3d':
  ART: Logging levels for Checker

Merge "ART: Logging levels for Checker"

am c507c7f8: Merge "[optimizing compiler] Compute live spill size"

* commit 'c507c7f8f905f77b039e58441b2e7da20fa65342':
  [optimizing compiler] Compute live spill size

Merge "[optimizing compiler] Compute live spill size"

am f80dfb49: Merge "ART: Increase run-test 114 Java-level timeout"

* commit 'f80dfb498451dc59e8f3375e6cf617055d3e76d1':
  ART: Increase run-test 114 Java-level timeout

Merge "ART: Increase run-test 114 Java-level timeout"

[optimizing compiler] Compute live spill size

The current stack frame calculation assumes that each live register to
be saved/restored has the word size of the machine.  This fails for X86,
where a double in an XMM register takes up 8 bytes.  Change the
calculation to keep track of the number of core registers and number of
fp registers to handle this distinction.

This is slightly pessimal, as the registers may not be active at the
same time, but the only way to handle this would be to allocate both
classes of registers simultaneously, or remember all the active
intervals, matching them up and compute the size of each safepoint
interval.

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

ART: Increase run-test 114 Java-level timeout

Increase the timeout to be larger than the run-test timeout. That
way ART will dump all threads when running the test on the host.

Change-Id: I842b056bba650e5c7c3293376a48d48b7468e73d

am e81e1cd7: Merge "ART: Allow scoped adjustments to log verbosity"

* commit 'e81e1cd75b7dc6e44c5654e3ae329e86662ca02b':
  ART: Allow scoped adjustments to log verbosity

Merge "ART: Allow scoped adjustments to log verbosity"

ART: Allow scoped adjustments to log verbosity

Add ScopedLogSeverity to adjust the logging level.

Suppress warnings by default in gtests. Suppress errors in instances
where errors are expected.

Change-Id: If3ef865813e9505ab60bc90baed63ff11d90afbb

am 63991dd7: Merge "Disable test for NDEBUG, it uses libarttest.so."

* commit '63991dd7c9bd68f23121e420c005628d7307cba3':
  Disable test for NDEBUG, it uses libarttest.so.

Merge "Disable test for NDEBUG, it uses libarttest.so."

Disable test for NDEBUG, it uses libarttest.so.

Change-Id: Ie275583aec421f3a40663ff44388a73e890ff894

am a61a3295: Merge "ART: Mac build fix for unused constant"

* commit 'a61a329525d4b4d4cb796660f083da4ec1a26ca1':
  ART: Mac build fix for unused constant

Merge "ART: Mac build fix for unused constant"

ART: Mac build fix for unused constant

Change-Id: I77e70c877e8a3608555f128e0edf8784db178e86

am 659b1ebd: Merge "ART: Host timeout Mac build fix."

* commit '659b1ebdd569a35b41201e4f2ebc625d20594661':
  ART: Host timeout Mac build fix.

Merge "ART: Host timeout Mac build fix."

ART: Host timeout Mac build fix.

Mac fix for unsupported SIGRTMIN introduced in commit
038bb2252ed1d7132f45006507e389b7ba1617ce.

Bug: 18933933
Change-Id: I502cda51802d67f089c09240d89f71166ed56c6d

am 44c78b3f: Merge "ART: Simple structural class check"

* commit '44c78b3f33688888078990df4fb5f14c78247a20':
  ART: Simple structural class check

Merge "ART: Simple structural class check"

am 1c1d40ab: Merge "ART: dex2oat flag for HGraphVisualizer dump file"

* commit '1c1d40ab7fd856eac52e1a3be7ce60da3fc21fb5':
  ART: dex2oat flag for HGraphVisualizer dump file

ART: Logging levels for Checker

Small patch which enables Checker to print errors but not info
messages as opposed to the current all or nothing. This is useful for
running Checker with run-test.

Change-Id: I500ded0db4f83ed9bbdef9c1f7b1eb18a90d7f0e

ART: Simple structural class check

Adds a simple check to class-loading when the embedded dex file in
an oat file and the dex file on the class path where we found the
class do not match.

We require that the number of methods and fields do not change, as
that will almost certainly mean that quickened and other compiled
offsets are wrong now. This is a reasonably lightweight change, but
we should investigate a full comparison including name and type of
members.

Bug: 17937814
Bug: 18708951

(cherry picked from commit 15a33b3f88546bce85dcb9d28caf200da51154d7)

Change-Id: Icb9638bebd369ab23822817f4a97c8dd8625fea5

Merge "ART: dex2oat flag for HGraphVisualizer dump file"

am cbabd3cb: Merge "ART: Clean assembler_test disassemblies"

* commit 'cbabd3cba33662b296df7c1d392bdd35b3c978ad':
  ART: Clean assembler_test disassemblies

am fe7e224a: Merge "ART: Dump all threads on test timeout"

* commit 'fe7e224a1f7c2a95ca45246ed54d65f1d3ddf051':
  ART: Dump all threads on test timeout

Merge "ART: Clean assembler_test disassemblies"

Merge "ART: Dump all threads on test timeout"

ART: Dump all threads on test timeout

Use SIGRTMIN+2 as a special signal to dump all threads similar to
SIGQUIT. Use nested timeouts to enforce a test timeout and dump
the threads in a deadlock.

Bug: 18933933
Change-Id: I4209047eeca07ff360d7c19922d5b5da64fd69a5

am 4db3aeb8: Merge "ART: Fix indentation in Mips backend"

* commit '4db3aeb88001367a032df33e5801c9add6a14b06':
  ART: Fix indentation in Mips backend

Merge "ART: Fix indentation in Mips backend"

ART: Fix indentation in Mips backend

Fix the indentation to be standard.

Change-Id: I39a16716be3429dfef6df0a585e24423b46363a2