am 24898d44: Merge "Disable test, it fails on target and read-barrier config."

* commit '24898d447640c3a45e6fc9f404b049eddc5a2709':
  Disable test, it fails on target and read-barrier config.

am 911b4be5: Merge "Fix transaction aborting"

* commit '911b4be50c379d052dd9e5f9b59ce91df1340453':
  Fix transaction aborting

Merge "Disable test, it fails on target and read-barrier config."

Disable test, it fails on target and read-barrier config.

Change-Id: Ib1ae28b1e81a1355196c32c43765a6f5466ca728

Merge "Fix transaction aborting"

Fix transaction aborting

During compilation, a java.lang.InternalError is used to indicate
that class initialization failed and the enclosing transaction
should be aborted and the changes rolled back. However there is
nothing preventing the code executed from a class initializer from
catching that exception (like catching Throwable and ignore it).
Therefore we may return from the class initializer with no pending
exception, even if the transaction was aborted, and not rollback
the changes properly.

To fix this, we now rely on the new Transaction::aborted_ field to
know whether a transaction aborted. When returning from the class
initializer without pending exception, we now check wether we aborted
the enclosing transaction. If that's the case, we set the status of
the class to kStatusError and throw a new java.lang.InternalError
with the original abort message.

This CL also contains some cleanup:
- Renames Transaction::Abort to Transaction::Rollback which is less
ambiguous and more reflect what is done.
- Moves the code throwing the java.lang.InternalError exception into
the Transaction::ThrowInternalError method so we do not duplicate
code. Now we may abort transaction more than once (because we may
have caught the java.lang.InternalError then execute code causing
new transaction abort), we only keep the first abort message to
throw the exception.
- Updates transaction_test with more cases and more checks.
- Bumps oat version to force recompilation with this fix.

Bug: 19202032
Change-Id: Iedc6969528a68bbdf3123146e990df4dbc57834b

am 7e6a918e: Merge "Clear thread local freed bits in RosAlloc::Run::InspectAllSlots"

* commit '7e6a918ecfbe786c060bce0eeddd55c4c70f819d':
  Clear thread local freed bits in RosAlloc::Run::InspectAllSlots

Merge "Clear thread local freed bits in RosAlloc::Run::InspectAllSlots"

am 19929654: Merge "art: Try to fix parsed_options_test on the build server"

* commit '19929654ddf3bce1b89db92a9b7fb47d6e91e2f8':
  art: Try to fix parsed_options_test on the build server

am edf45678: Merge "Fix broken volantis build"

* commit 'edf45678f133ce51d7aff43be68d9be3936ffca2':
  Fix broken volantis build

Clear thread local freed bits in RosAlloc::Run::InspectAllSlots

Previously we didn't take these bits into consideration. This could
cause RosAlloc::Run::InspectAllSlots to inspect recently freed
allocations as allocated.

Bug: 19193521
Change-Id: I56b3c089e2a36098423261cda623fc834069f832

Merge "art: Try to fix parsed_options_test on the build server"

art: Try to fix parsed_options_test on the build server

(It works fine locally because environment variables are defined
differently)

Also some minor cleanup elsewhere pointed out in the previous code
review.

Change-Id: I80650d2be6ce467b838db85eb130a0020d7c1480

Merge "Fix broken volantis build"

Fix broken volantis build

Change-Id: Iedc1a94fd1ec4224b605b984c11cfcc8c1c510e3

am 1bb6cb0b: Merge "art: Refactor RuntimeOptions/ParsedOptions"

* commit '1bb6cb0b7e10698f12213f7a4fabc845057d9ab9':
  art: Refactor RuntimeOptions/ParsedOptions

Merge "art: Refactor RuntimeOptions/ParsedOptions"

art: Refactor RuntimeOptions/ParsedOptions

Refactor the RuntimeOptions to be a
type-safe map (VariantMap, see runtime_options.h) and the ParsedOptions
to delegate the parsing to CmdlineParser (see cmdline/cmdline_parser.h).

This is the start of a command line parsing refactor, and may include
more in the future (dex2oat, patchoat, etc).

For more details of the command line parsing generator usage see cmdline/README.md

Change-Id: Ic67c6bca5e1f33bf2ec60e2e3ff8c366bab91563

am a8fb460f: Merge "ART: Fix old run scripts"

* commit 'a8fb460f0a5c96881c433f9586bdf1f705c97d35':
  ART: Fix old run scripts

Merge "ART: Fix old run scripts"

ART: Fix old run scripts

The run scripts had not been updated when the central command script
was unified, so they were always choosing target mode.

Also update the run scripts so they abort when wrong options are
used.

Also strip superfluous whitespace from commands.

Change-Id: I13cc59f43734a86062fc3ce5ff4ea308c301d4d3

am f30c00fc: Merge "Fix incorrect static const"

* commit 'f30c00fc436264014338744f9902346f0642df3b':
  Fix incorrect static const

Merge "Fix incorrect static const"

Fix incorrect static const

Bug: 19265183
Change-Id: I573df60fe90952246d2126ced010f2bdac006bca

am 7d318624: Merge "ART: Pass Options should be strings instead of integers"

* commit '7d3186247cd382c4f7cf8093961795e8565b00e0':
  ART: Pass Options should be strings instead of integers

Merge "ART: Pass Options should be strings instead of integers"

am c470aefe: Merge "Fix occasional long ThreadSuspendSleep"

* commit 'c470aefe5abb9ab0b209c7c99139734efe97b177':
  Fix occasional long ThreadSuspendSleep

Merge "Fix occasional long ThreadSuspendSleep"

am a4ddb549: Merge "Fix a bug in the inliner."

* commit 'a4ddb5493e2426488074cfeccf1b1e779d5385ee':
  Fix a bug in the inliner.

am a6919983: Merge "ART: Dump environment to cfg file."

* commit 'a6919983bfc7e2c1eb69a4408f940146a2b78d59':
  ART: Dump environment to cfg file.

am c2024144: Merge "Create HGraph outside Builder, print timings"

* commit 'c2024144da54504d819869dc928ce4ac975634d4':
  Create HGraph outside Builder, print timings

am 53150eb4: Merge "Fix a bug in combination of intrinsics and kNoOutputOverlap."

* commit '53150eb4fb5c3bbcb0138b8dccb481726299f6ab':
  Fix a bug in combination of intrinsics and kNoOutputOverlap.

am f4b24f7e: Merge "ART: Change x86 long param ABI (Quick/JNI/Opt)"

* commit 'f4b24f7ea507ff41db39154a1e6a681a582e224a':
  ART: Change x86 long param ABI (Quick/JNI/Opt)

am 6bc17805: Merge "Fix test after register allocator changes."

* commit '6bc17805d2df2678de186e3b4c3e2f959a48555f':
  Fix test after register allocator changes.

am b73e4447: Merge "Finally implement Location::kNoOutputOverlap."

* commit 'b73e4447736996f7cc8b193d7e3f5c38d5739d22':
  Finally implement Location::kNoOutputOverlap.

Merge "Fix a bug in the inliner."

Fix a bug in the inliner.

Code did not work in the presence of multiple returns.
Spotted by Mark P. Mendell.

Change-Id: I237050a0d79c0cfaa479e9b886f7450879e84713

Merge "ART: Dump environment to cfg file."

ART: Dump environment to cfg file.

Environment shows additional dependency and instructions with environment
should usually be a safepoint as well.

Change-Id: I04606624e6ea212ab1dc7502cdb87b5d5d0e3ac2

Merge "Create HGraph outside Builder, print timings"

Create HGraph outside Builder, print timings

This patch refactors the way HGraph objects are created, moving the
instantiation out of the Builder class and creating the CodeGenerator
earlier. The patch uses this to build a single interface for printing
timings info and dumping the CFG.

Change-Id: I2eb63eabf28e2d0f5cdc7affaa690c3a4b1bdd21

Merge "Fix a bug in combination of intrinsics and kNoOutputOverlap."

Fix a bug in combination of intrinsics and kNoOutputOverlap.

In case we need to go in the slow path for an intrinsic call,
we can't have the output be the same as the input: the current
liveness analysis considers the input to be live at the point of the call.

Change-Id: I5cbdc7f50dd06b4fefcbd3c213274fa645bd3fa0

Merge "ART: Change x86 long param ABI (Quick/JNI/Opt)"

Merge "Fix test after register allocator changes."

Fix test after register allocator changes.

Change-Id: I22b8faf7926d33ab19033deaba77001d164abd2a

Merge "Finally implement Location::kNoOutputOverlap."

Finally implement Location::kNoOutputOverlap.

The [i, i + 1) interval scheme we chose for representing
lifetime positions is not optimal for doing this optimization.
It however doesn't prevent recognizing a non-split interval
during the TryAllocateFreeReg phase, and try to re-use
its inputs' registers.

Change-Id: I80a2823b0048d3310becfc5f5fb7b1230dfd8201

Fix occasional long ThreadSuspendSleep

Since usleep can wait much longer than the specified us, we use
sched_yield instead for more predictable behavior.

Bug: 18379850
Change-Id: I729212251db1e86e470240b7a042bbf2d4e981cd

am 69d69ea4: Merge "ART: checkpoint mechanism optimization"

* commit '69d69ea40fe64ff2e70daffc365a2fffe5964fcc':
  ART: checkpoint mechanism optimization

Merge "ART: checkpoint mechanism optimization"

ART: Pass Options should be strings instead of integers

This patch modifies the pass options to handle string
values as well as integers. This will allow for more
verbose pass customization.

Change-Id: Iaf9507ceaae2cef317a23f0783404bacfdcad023
Signed-off-by: Jean-Philippe Halimi <jean-philippe.halimi@intel.com>

am 900e7b36: Merge "ARM/ARM64: Dump thread offset."

* commit '900e7b36057410cb12cc3c1a066dca04db3c2d45':
  ARM/ARM64: Dump thread offset.

am f9af1941: Merge "Inline methods with multiple blocks."

* commit 'f9af19413333c271192c3b11425f865bd8054c0c':
  Inline methods with multiple blocks.

Merge "ARM/ARM64: Dump thread offset."

Merge "Inline methods with multiple blocks."

Inline methods with multiple blocks.

Change-Id: I3431af60e97fae230e0b6e98bcf0acc0ee9abf8c

am bd17c3b4: Merge "Use a different block order when not compiling baseline."

* commit 'bd17c3b46b035861ced434d6ed4d4fe183736e65':
  Use a different block order when not compiling baseline.

Merge "Use a different block order when not compiling baseline."

Use a different block order when not compiling baseline.

Use the linearized order instead, as it puts blocks logically
next to each other in a better way. Also, it does not contain
dead blocks.

Change-Id: Ie65b56041a093c8155e6c1e06351cb36a4053505

am bb2f65e6: Merge "Fix build by putting ATTRIBUTE_UNUSUED."

* commit 'bb2f65e64844bfca8d05d4c37991d858aa748648':
  Fix build by putting ATTRIBUTE_UNUSUED.

am 0b87c470: Merge "Fix thumb2 assembler\'s ShifterOperandCanHold."

* commit '0b87c47033081d4b86e6611d947d49926df6b577':
  Fix thumb2 assembler's ShifterOperandCanHold.

Merge "Fix build by putting ATTRIBUTE_UNUSUED."

Fix build by putting ATTRIBUTE_UNUSUED.

Change-Id: If894fa1343a7c8a6456d707e815d803404a171d7

Merge "Fix thumb2 assembler's ShifterOperandCanHold."

Fix thumb2 assembler's ShifterOperandCanHold.

Change-Id: Ica3c5f39d52722950725ba622fc67bd7f634918a

am 08029544: Merge "ART: Fix to X86Mir2Lir::GenReduceVector"

* commit '08029544d72bd9bec162956978afcb59204ea97b':
  ART: Fix to X86Mir2Lir::GenReduceVector

Merge "ART: Fix to X86Mir2Lir::GenReduceVector"

am 09fbde3b: Merge "Allow nested inlining."

* commit '09fbde3b2b4cad32059b0fd36524c11a808b2885':
  Allow nested inlining.

am 515d828f: Merge "Go back to -O2."

* commit '515d828fd51f55bc71a6bcd268a7311f614d28b0':
  Go back to -O2.

am d82b172b: Merge "Revert "Revert "ART: Make 074-gc-thrash more OOME tolerant"""

* commit 'd82b172b2b5f68f224ead383d35200123fa71119':
  Revert "Revert "ART: Make 074-gc-thrash more OOME tolerant""

am 63aa9aec: Merge "Revert "ART: Make 074-gc-thrash more OOME tolerant""

* commit '63aa9aec310dd0d49e66a6bba7777e56d544de01':
  Revert "ART: Make 074-gc-thrash more OOME tolerant"

Merge "Allow nested inlining."

Allow nested inlining.

Also run optimizations before iterating over
the instructions to get rid of, e.g., null checks.

Change-Id: I6e52e61ce4d0ccb63d687afea09bb4722453bb6a

ARM/ARM64: Dump thread offset.

Dump thread offset in compiler verbose log for arm32/arm64 and
oatdump for arm64.

Before patch :
0x4e: ldr      lr, [rSELF, #604]
After patch :
0x4e: ldr      lr, [rSELF, #604]  ; pTestSuspend

Change-Id: I514e69dc44b1cf4c8a8fa085b31f93cf6a1b7c91

Merge "Go back to -O2."

Go back to -O2.

Stack space suffers otherwise.

Change-Id: I890a9714acd8500981c3d41df0f5f7fd63d48f8f

Merge "Revert "Revert "ART: Make 074-gc-thrash more OOME tolerant"""

Revert "Revert "ART: Make 074-gc-thrash more OOME tolerant""

This reverts commit ed8b53d679d46891d8d405b0ce4a48c282d872f7.

This wasn't the reason for breakage. Instead, it was the Makefile change in:
https://android-review.googlesource.com/#/c/128250/

Change-Id: Ib94870c1eccbe821e3b4945ea92ca35ca0a8a9b9

ART: checkpoint mechanism optimization

GC thread and trim thread are both using checkpoint mechanism. GC thread
will request java threads to mark their thread roots by themselves. Trim
thread will request java threads to trim their jni local reference
tables by themselves.

The checkpint mechanism semantics is that the runnable java threads will
run checkpoint function itself at safepoint, and finally the java
threads and gc thread or trim thread is synchronized via barrier. If the
java threads are not runnable, gc thread or trim thread will suspend
them and then run their checkpoint functions one by one on behalf of
them. If all the java threads are not runnable, then gc thread or trim
thread will do all the work itself. In this case, there is no need
synchronization. This will save unnecessary synchronization and thread
state transitions.

Change-Id: If55940946cb3f8b1af42c7237c334f09c8ec7a9f

Merge "Revert "ART: Make 074-gc-thrash more OOME tolerant""

Revert "ART: Make 074-gc-thrash more OOME tolerant"

Redness on the boards.

This reverts commit 2f6046edd7da9bb67e30b377955ff6faa8bff67e.

Change-Id: I8e169c537aaa76e34a7027460e3e9bff6f75a581

am 5a7c6344: Merge "ART: Make 074-gc-thrash more OOME tolerant"

* commit '5a7c634406b2acc4917009b43dcc7def2178a79b':
  ART: Make 074-gc-thrash more OOME tolerant

am c398b058: Merge "ART: Fix assembler_thumb_test"

* commit 'c398b0580fc7a7be768dcd6786d5ab6e7a892c9e':
  ART: Fix assembler_thumb_test

Merge "ART: Make 074-gc-thrash more OOME tolerant"

Merge "ART: Fix assembler_thumb_test"

ART: Fix assembler_thumb_test

Fix the inconsistent calls. This was bad practice, requiring that
shape meant arm32 and thumb were not interchangeable.

Also fix an error in the assembler itself for tst and teq.

Change-Id: I50242801002aa81cc17bfaa53eb8b7c1d9488fbc

ART: Make 074-gc-thrash more OOME tolerant

The strong and weak array need to stay in sync. Don't set the
strong reference when the weak reference can't be allocated.

Change-Id: I6c8f245627d206d89a378099cea97e4f1cd1d9fa

am a1a0ffb7: Merge "ART: Fix thumb2 utils assembler"

* commit 'a1a0ffb72c4d7170dee9d46f055ca56932eedbae':
  ART: Fix thumb2 utils assembler

Merge "ART: Fix thumb2 utils assembler"

ART: Fix thumb2 utils assembler

It didn't correctly put down the 16b variants, as it was not
checking the ShifterOperand correctly.

Add one simple test. TODO: exhaustive testing.

Change-Id: Ied08da115f8417a3ca92bd9967c6dcdc102e7510

am 7687b250: Merge "ART: Fix run-tests for emulator"

* commit '7687b250a565f465e3222c8b5adda1427f10c5f6':
  ART: Fix run-tests for emulator

Merge "ART: Fix run-tests for emulator"

ART: Fix to X86Mir2Lir::GenReduceVector

When generating the result to memory, the existing code didn't set the
aliasing correctly.

Mark the result as going to a Dalvik VR, and mark it as only a write.

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

am 4336b97f: Merge "Fix broken gtests after SuspendCheck optimization"

* commit '4336b97fc291d28f883607c88feaff202a684a59':
  Fix broken gtests after SuspendCheck optimization

Merge "Fix broken gtests after SuspendCheck optimization"

Fix broken gtests after SuspendCheck optimization

Fixes hardcoded graph dumps in pretty printer tests and an assumption
that non-zero branch offset in dex implies presence of HInstructions.

Change-Id: Iee273c06b7b36410b4621107bef2f3592ece2f5b

ART: Fix run-tests for emulator

On a standard emulator, there is no sdcard emulation, so trying to
create a temp file in /sdcard will fail - it will try to create in
the root file system, which is read-only.

Change-Id: If7d1ad82db156177a5be58c2f79ed730cf7ab6b9

am ec57f287: Merge "No SuspendChecks when branching to return block"

* commit 'ec57f287ba0475e9f9e6dad749b67625d0c0d92e':
  No SuspendChecks when branching to return block

Merge "No SuspendChecks when branching to return block"