Fix non-range String init calls

When the String constructor was called via invoke-direct, it is changed
to the new StringFactory which is static. That means that the args need
to be shifted by one to deal with the change from non-static to static.
However, the non-range version of the invoke-direct was not shifted
correctly causing unstarted runtime initialization to get the wrong
first_dest_reg argument.

(cherry picked from commit 788a0a157cb138c33882511ff09afacde99443b7)

Bug: 21036900
Change-Id: Ibd7a643d877514ea396d7e4ab0dea327207cb78f

Merge "Revert "Revert "Remove junit from our core image."""

Revert "Revert "Remove junit from our core image.""

This reverts commit 7028d59fa70c7c8482975c4f9b42308c6959483b.

Change-Id: Ia005921af58595999b5b76001fbd16dfb484ab35

Merge "Revert "Revert "Do not pass junit.jar to vogar."""

Revert "Revert "Do not pass junit.jar to vogar.""

This reverts commit 5bf24f3d8633efda0b20126be411c2cfa4e75097.

Change-Id: I5512c57e7b725991468e9d08d8110df5809dc854

Merge "ART: Do not relocate app program headers in patchoat."

Merge "[optimizing] Fold HTypeConversion of constants"

ART: Do not relocate app program headers in patchoat.

Change the check whether to relocate program headers in
patchoat to simply look whether there is a PT_LOAD section
with p_vaddr == 0. If there is, don't relocate the headers,
it should be an app. Otherwise, it's a boot image and needs
to be relocated.

Add overflow checking to ElfFileImpl<>::GetLoadedSize().

Bug: 21047854
Change-Id: Ie6737f338687296b6dbf4bb3b36358501dfb1587

Merge "Revert "Do not pass junit.jar to vogar.""

Revert "Do not pass junit.jar to vogar."

Will re-land after fiing vogar.

This reverts commit 3185c8da1dc2c2497168817c083b3c9587ba0883.

Change-Id: I6f5ed14aa3f7deb768195c0797125a34c6255d85

Merge "ART: Fix unused return-value in test"

ART: Fix unused return-value in test

Check the return value of system() call.

Change-Id: I1f5f9621f6a39029b9df5b0d4ab0e230ba6c79a3

Merge "Undo arm64 workaround."

Merge "Revert "Remove junit from our core image.""

Revert "Remove junit from our core image."

To put us back green. Will re-land when vogar is cleaned up.

This reverts commit 0ad65a3c794b18101def5096d8ed151382c9d5aa.

Change-Id: I24a58bd87f133132d7774104eb8f77d6bbd6292b

Merge "Revert "New failure due to a junit/vogar difference.""

Revert "New failure due to a junit/vogar difference."

This reverts commit 203a5555957f4c56c4a5cc7ffcb99600c8ab557e.

Change-Id: I51c3aa7275ab43b4adaf8728245612b8416c8284

Merge "Also print class for NoSuchFieldException"

Merge "New failure due to a junit/vogar difference."

New failure due to a junit/vogar difference.

Change-Id: I2148c56dd67406d719778ac41b6691fbad2614d3

Also print class for NoSuchFieldException

Example message:
Caused by: java.lang.NoSuchFieldException: No field value in class Ljava/lang/String;

Added test.

(cherry picked from commit 3beb245da9392818e3154d47593f82cf0ef69aac)

Bug: 20881251
Bug: 21027454

Change-Id: I4043cbf26c3077952b6c151da0d0edd980da26b1

[optimizing] Fold HTypeConversion of constants

While looking into optimizing long shifts on x86, I found that the
compiler wasn't folding HTypeConversion of constants.  Add simple
conversions of constants, taking care of float/double values
with NaNs and small/large values, ensuring Java conversion semantics.

Add checker cases to see that constant folding of HTypeConversion is
done.

Ensure 422-type-conversion type conversion routiness don't get
inlined to avoid compile time folding.

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

Merge "Remove junit from our core image."

Merge "Silence unused result warnings in art."

Remove junit from our core image.

It's not used, and when we run art with vogar, we get
incompatibilities. For example vogar's TestSuite class
is very different than junit's TestSuite.

Change-Id: I45c78d86c09c28b6136a6e50f6a08a746103794a

Merge "Do not pass junit.jar to vogar."

Do not pass junit.jar to vogar.

Not needed after https://android-review.googlesource.com/#/c/150563.
Also dex2oat now complains about duplicated classes, and fallbacks
to the interpreter if it sees any. Since vogar has its own junit,
dex2oat was complaining.

Change-Id: I579feb0b1a049af3fba664fd2d86c67c66c8eae1

Silence unused result warnings in art.

This lets us turn on _FORTIFY_SOURCE=2 for the host.

The logging change is basically a trick (since we're still ignoring the
result unless it's an EINTR failure).

The profiler change is in a function whose error checking is a mess; the
fd is leaked, and it's not clear what's supposed to happen about the flock
on failure. I didn't exit if the ftruncate fails because the rest of the
function just blunders on, and error handling here needs a deeper rethink.

Bug: http://b/20558757
Change-Id: I7a2873fce08e37f4af826a04b4701d7ac6a8e3fa

Undo arm64 workaround.

Now we have new llvm to compile art for arm64 without -fno-vectorize.
BUG: 21080674

Change-Id: Id1b3a88ab9d4cf78b7f0dfd8cb435ed6e7ab7cb2

Merge "Hold heap bitmap lock in Heap::GetObjectsAllocated"

Hold heap bitmap lock in Heap::GetObjectsAllocated

Fixes a race condition where add and remove space could cause a crash
when we iterated over the spaces.

TODO: Add a spaces lock or something to guard against this.

(cherry picked from commit a395c0a492079d86b312c9edc796d63001576954)

Bug: 21031927

Change-Id: I7f0d558316f8e9d9f22ffd182e8666355bf50d47

Merge "Add test for String init called from unresolvable class."

Merge "Quick: Fix DCE, clear high word flag in RemoveChange()."

Quick: Fix DCE, clear high word flag in RemoveChange().

Bug: 20640451
Change-Id: Ibab707ff8e60816b9871c3f4096c0bbbf535cbe8

Add test for String init called from unresolvable class.

Bug: 20971268

(cherry-picked from f450cf6b06255ead0a43a9e94dc2f2175e6f9849)

Change-Id: I810f599b38a46e738bcacb15ca3c4537e1fb7b77

Merge "Add notes about fast JNI calls."

Add notes about fast JNI calls.

Bug: 19665903
Change-Id: Ie0610aa17ab1365c08cb0b5b0931a8cefb65f252

Merge "Fix bug in removal of *.flock files."

Merge "ART: arm indexOf intrinsics for the optimizing compiler"

Fix bug in removal of *.flock files.

Bug: 20816098
Change-Id: I72ee32884f32af43a54518c9b9772f63957ab5de

Merge "Quick: Fix DCE to mark wide register overlaps correctly."

Merge "Re-enable 098-ddmc for the read barrier config."

Quick: Fix DCE to mark wide register overlaps correctly.

Previously we missed some cases of overlap with registers
coming from previous blocks.

Bug: 20640451
Change-Id: I4b32a7aaea2dea1b0b9560ae3459a4d903683f20

Merge "ART: Rediscover loops after deleting blocks in DCE"

Merge "ART: Split Checker into smaller files"

ART: Split Checker into smaller files

Checker code has become too messy and incomprehensible. This patch
splits it into more manageable and better structured units.
Functionality remains unchanged.

Resubmission of change I870c69827d2be2d09196a51382a3f47f31cd2ba3 due
to omission of file 'tools/checker/file_format/common.py'.

Change-Id: I277a4aa65a2e3b54f0e89901fdb9f289f55a325f

Merge "Revert "ART: Split Checker into smaller files""

Revert "ART: Split Checker into smaller files"

This reverts commit a490be5dd95982f77ff1447bea9ee06604038a96.

Change-Id: Ic3b7cf172200caced9ae2f10d2f200447e6801ee

Merge "ART: Split Checker into smaller files"

ART: Split Checker into smaller files

Checker code has become too messy and incomprehensible. This patch
splits it into more manageable and better structured units.
Functionality remains unchanged.

Change-Id: I870c69827d2be2d09196a51382a3f47f31cd2ba3

ART: arm indexOf intrinsics for the optimizing compiler

Add intrinsics implementations for indexOf in the optimizing
compiler. These are mostly ported from Quick.

Bug: 20889065
Change-Id: I18ee849d41187a381f99529669e6f97040aaacf6

Merge "Small correction in Optimizing's ARM64 code generator."

Merge "Revert "Add new rule to the graph checker.""

Revert "Add new rule to the graph checker."

This reverts commit 9d50775046a51941f86cced03593c61c89ae615e.

Change-Id: I9533116bbaef5fd5d2944c4ebebd2f01b0047e34

Merge "Add new rule to the graph checker."

ART: Rediscover loops after deleting blocks in DCE

The way DCE currently updates loop information does not cover all
cases. This patch removes the logic, resets loop information of live
blocks to pre-SSA state and reanalyzes the affected loops.

Change-Id: I0b996a70235b95a8db0de9a23a03f71db57a21b8

Merge "Simplify floating-point comparisons with NaN in Optimizing."

Merge "ART: Enable Checker annotations in smali files"

ART: Enable Checker annotations in smali files

Change-Id: I8eeb76154b5aa5506fc477d38cb56cae168cdc90

Merge "Update test expectations."

Update test expectations.

Change-Id: Ic0eb5e656fb81e3e8754131296715c75a6faf358

Re-enable 098-ddmc for the read barrier config.

Bug: 20720510
Bug: 12687968
Change-Id: I6e5c61a88bfce15231c1476bc0edc28ae53d081f

Merge "Remove unused StackHandleScope"

Merge "Insert a read barrier in the JNI bridge."

Merge "Fix typo in VLOG message."

Merge "Add more diagnostics info to flaky oat file assistant test."

Merge "ART: x86 indexOf intrinsics for the optimizing compiler"

ART: x86 indexOf intrinsics for the optimizing compiler

Add intrinsics implementations for indexOf in the optimizing
compiler. These are mostly ported from Quick. Add instruction
support to assemblers where necessary.

Change-Id: Ife90ed0245532a5c436a26fe84715dc357f353c8

Add new rule to the graph checker.

Phis typed as `reference` should not have equivalents after building the
SSA.

Change-Id: I9cac189362583ef7c9c7defd3da7338f419f2f94

Remove unused StackHandleScope

It is causing crashes when there is no current thread (because it has
been detached from the runtime). It happens when we sanity check the
stack during runtime shutdown after stopping method tracing.

Bug: 20993343
Change-Id: I549664932f513a01339e5b1c34b3ddc5744888c3

Fix typo in VLOG message.

Change-Id: Iaf4ce1787816afce947c6e963df19d20ef5f5e1b

Add more diagnostics info to flaky oat file assistant test.

Change-Id: Ifd0029d389ba7585a819c01fc9cc846681d0a349

Merge "Add a flag to StackVisitor for inlining."

Add a flag to StackVisitor for inlining.

The flag tells whether the stack walk needs to include inlined
Java frames.

This does not do anything just yet, as we're not inlining anyways.

Change-Id: I716e25094fe56fa335ca1f9a398c1bcdba478e73

Merge "[optimizing] Improve 32 bit long shift by 1."

Merge "Add a parent environment to HEnvironment."

Add a parent environment to HEnvironment.

This code has no functionality change. It adds a placeholder
for chaining inlined frames.

Change-Id: I5ec57335af76ee406052345b947aad98a6a4423a

[optimizing] Improve 32 bit long shift by 1.

Also change FOO << 1 to FOO+FOO in the instruction simplifier.  This is
an architecture independent simplification, which helps 'long << 1' for
32 bit architectures.

Generate an add/adc for long << 1 in x86, in case something is generated
after the simplifier.

Add test cases for the simplification.

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

Merge "[optimizing] Tune some x86_64 moves"

Merge "Don't allow JIT to compile native methods"

Don't allow JIT to compile native methods

Change-Id: I12722309ba52c430e23ab862b29f7a48383f00d2

Merge "Add stub_test implementation for mips & mips64"

Merge "Fix stub_test for Mips64"

Fix stub_test for Mips64

Force ENTRY macro to load GP. Instead .cpload (used for o32 abi,
not working for n64) use .cpsetup to set up GP and save previous
GP value.

Fix expectations in GetSet32Static.

Change-Id: Ie97e32fa2fee7a6e8bae6bc36e78976b8be62b79

Merge "Check IsReferenceVReg during deopt"

Check IsReferenceVReg during deopt

Required since the quick GC maps may not agree with the verifier
ones. Without this check we may copy stale object references into
the shadow frame.

(cherry picked from commit f00baf56ef647684888a407dbb6adadd704a2039)

Bug: 20736048
Change-Id: I7783c8a8ee45cf601b08b4c38f1dec7f7d11380c

Merge "ART: Merge entries with same name and protect in MemMap dump."

ART: Merge entries with same name and protect in MemMap dump.

This should make the MemMap dump less chatty and allow the
logger to keep more relevant output.

Bug: 20873174
Change-Id: I09058798faa8f79d154cee5008017b03700df8db

Merge "Skip compilation when falling-through out of method code."

Merge "Fix typo in comments."

Simplify floating-point comparisons with NaN in Optimizing.

This change was suggested by Ian.

Also, simplify some art::HFloatConstant and
art::HDoubleConstant methods.

Change-Id: I7908df23581a7f61c8ec79c290fe5f70798ac3be

Merge "ART: Fix debug output for bad reused MemMap."

ART: Fix debug output for bad reused MemMap.

Log the error_msg instead of a pointer to it.
Raise the log level to ERROR for the process maps.

Change-Id: I95f82f013d1c6360f40e6315652d1a2dda0f39e2

Merge "ART: Refactor 082-inline-execute"

ART: Refactor 082-inline-execute

Refactor the indexOf intrinsics tests so that the optimizing
compiler would actually compile them.

Bug: 20889065
Change-Id: I69bfda7fa3eb4ce42c593203731e3ddd61f7e1ed

Insert a read barrier in the JNI bridge.

Bug: 20720510
Bug: 12687968
Change-Id: Ia9edb614853bad45ec25d6e2142361a8bda1eccf

Fix typo in comments.

Change-Id: I6569c0fdc878f797b37509432b59b0d5c9bc5533

Merge "Quick: Fix DCE to rename SSA regs for degenerate Phis."

Merge "Quick: Abolish kMirOpCheckPart2."

[optimizing] Tune some x86_64 moves

Generate Moves of constant FP values by loading from the constant table.

Use 'movl' to load a 64 bit register for positive 32-bit values, saving
a byte in the generated code by taking advantage of the implicit
zero extension.

Change a couple of xorq(reg, reg) to xorl to (potentially) save a byte
of code per xor.

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