am ba14a447: am e6caf046: Merge "Fix issue with Thread.setName hanging after Thread.start" into lmp-dev

* commit 'ba14a447405980211cc68203fa8845a75d97e929':
  Fix issue with Thread.setName hanging after Thread.start

am e6caf046: Merge "Fix issue with Thread.setName hanging after Thread.start" into lmp-dev

* commit 'e6caf04639aec134f7b668bf71cd6797a4759dd9':
  Fix issue with Thread.setName hanging after Thread.start

Merge "Fix issue with Thread.setName hanging after Thread.start" into lmp-dev

am 5e894e1f: am 7654123b: Merge "Ensure proxy constructor is in dex cache" into lmp-dev

* commit '5e894e1f84c267f1a259f0831a5cdd5d2b9d6fa3':
  Ensure proxy constructor is in dex cache

am 7654123b: Merge "Ensure proxy constructor is in dex cache" into lmp-dev

* commit '7654123b2fd7abeb40522dac791fa26a1364d030':
  Ensure proxy constructor is in dex cache

am 0bf01ef1: am 679c3ced: Merge "Prevent exception bugs in class linker" into lmp-dev

* commit '0bf01ef1c5ac31ff35cb044b3fc48c3ea90b04be':
  Prevent exception bugs in class linker

Merge "Ensure proxy constructor is in dex cache" into lmp-dev

am 679c3ced: Merge "Prevent exception bugs in class linker" into lmp-dev

* commit '679c3cedaf1550113104f5113b23f0fdec6175c0':
  Prevent exception bugs in class linker

Merge "Prevent exception bugs in class linker" into lmp-dev

Ensure proxy constructor is in dex cache

Bug: 17262039

(cherry picked from commit ae94e350f100207359b8345d8d21e78e7cfb44c4)

Change-Id: I1cfc8dac1f63b9012fa098804135be8847b1daee

Fix issue with Thread.setName hanging after Thread.start

When suspending a thread by peer the invariant that only attached threads are
suspended must be maintained. Add a ThreadList::Contains check which requires
making this method non-static.
Add some extra thread logging.

Bug: 17302037
Change-Id: I89af9840e398c2ceadeb736c431dfe3be8f3ba06

Prevent exception bugs in class linker

There were some places that could throw exceptions but still succeed.
This caused the allocation entrypoints to occasionally allocate a
heap object with a pending exception.
Also added some additional AssertNoExceptionPending.

Bug: 17164348

Change-Id: Ic6dd3b0cce9955349176503dd7f6c3da7ab0a6f1

am aad269d7: Merge "Actually ask the pthread implementation for the stack guard size."

* commit 'aad269d708cb1dd18fe3a319a862f91f01696435':
  Actually ask the pthread implementation for the stack guard size.

am 044489b4: am 5b41c2a8: Merge "Actually ask the pthread implementation for the stack guard size." into lmp-dev

* commit '044489b463722e1377b49477b1c555f569389336':
  Actually ask the pthread implementation for the stack guard size.

am 1b761a95: am d1e4e735: Merge "Revert "Avoid using NanoTime for compiler and verifier"" into lmp-dev

* commit '1b761a957961329d845a55f2dac41200ac9b244f':
  Revert "Avoid using NanoTime for compiler and verifier"

am dbc0ca70: Merge "Ensure proxy constructor is in dex cache"

* commit 'dbc0ca70bcee22561c2f0ab7129a7721369e9aea':
  Ensure proxy constructor is in dex cache

am 5b41c2a8: Merge "Actually ask the pthread implementation for the stack guard size." into lmp-dev

* commit '5b41c2a8f8e4a745b18685413b51c883cd993403':
  Actually ask the pthread implementation for the stack guard size.

am d1e4e735: Merge "Revert "Avoid using NanoTime for compiler and verifier"" into lmp-dev

* commit 'd1e4e735cdf98e2ece7fa9a5dfeaad98d0c37cd1':
  Revert "Avoid using NanoTime for compiler and verifier"

Merge "Actually ask the pthread implementation for the stack guard size."

Actually ask the pthread implementation for the stack guard size.

(cherry-pick 307091dc306c34ce9e4ee6cc3b467807b3a3bd12.)

Bug: 17111575
Change-Id: I23919b1e8aeff627a65daf57f1109bec60b196cc

Merge "Actually ask the pthread implementation for the stack guard size." into lmp-dev

Merge "Revert "Avoid using NanoTime for compiler and verifier"" into lmp-dev

Merge "Ensure proxy constructor is in dex cache"

Ensure proxy constructor is in dex cache

Bug: 17262039

Change-Id: I879341fa9c66f18ec64d6d4bf6a222a452c27739

am 782e743e: am cd7a048c: Merge "Support booting without functioning boot.oat/art patchoat." into lmp-dev

* commit '782e743e949240ffb97a523801f9dc5997e2b61b':
  Support booting without functioning boot.oat/art patchoat.

am cd7a048c: Merge "Support booting without functioning boot.oat/art patchoat." into lmp-dev

* commit 'cd7a048cfac23e6ec3e1ba3ab4c1dddb322eaadf':
  Support booting without functioning boot.oat/art patchoat.

Merge "Support booting without functioning boot.oat/art patchoat." into lmp-dev

Actually ask the pthread implementation for the stack guard size.

Bug: 17111575
Change-Id: I23919b1e8aeff627a65daf57f1109bec60b196cc

am 9c188588: Merge "Fix gcstress and gcverify test targets"

* commit '9c18858899738f79e75b5a31a96b91c8bfa8bdd7':
  Fix gcstress and gcverify test targets

Merge "Fix gcstress and gcverify test targets"

Fix gcstress and gcverify test targets

Make the build system look at the right variables to determine if these
should be on.

Change-Id: Iede993714eee773716cdc99181b10b280e96ca2a

am 2602e205: am 074c8c6a: Merge "Avoid using NanoTime for compiler and verifier" into lmp-dev

* commit '2602e205729fe90cffff9adeb25d7f65a59bdf21':
  Avoid using NanoTime for compiler and verifier

am 074c8c6a: Merge "Avoid using NanoTime for compiler and verifier" into lmp-dev

* commit '074c8c6aac3bd42a173befd1a4eafd479ef12dc9':
  Avoid using NanoTime for compiler and verifier

am 70644461: Merge "ART: Change temporaries to positive names"

* commit '70644461909a68eb5cd032027ae0eb528143d99c':
  ART: Change temporaries to positive names

Revert "Avoid using NanoTime for compiler and verifier"

This reverts commit c89e6edbdce97c7d199b54bf63aed7ea01ebf1ff.

Change-Id: Id3b89121a19e64bf3c457c39b375f3d3c0fcc579

Merge "Avoid using NanoTime for compiler and verifier" into lmp-dev

am 03de1f07: am 52fb2a63: Merge "Handle nested signals" into lmp-dev

* commit '03de1f07f5bdc2896d447415f2c3526556fa20f7':
  Handle nested signals

am 90364c5f: Merge "Handle nested signals"

* commit '90364c5f94069cf96e507b40076ea8eb2a6f303b':
  Handle nested signals

Merge "ART: Change temporaries to positive names"

Avoid using NanoTime for compiler and verifier

Target timings before with -j2 --compiler-filter=interpret-only:
Approximate speedup 3%.

Before:
real    0m36.852s
real    0m36.050s
real    0m36.246s

After:
real    0m35.667s
real    0m35.779s
real    0m35.458s

Bug: 16853450

Change-Id: Id996172d8283639fe293ddfc26d01fc1a0beed0b

resolved conflicts for merge of a42d23a7 to lmp-dev-plus-aosp

Change-Id: I0258f55e8199c06948878ba0e9bf7941bf60a435

Merge "Handle nested signals"

am 52fb2a63: Merge "Handle nested signals" into lmp-dev

* commit '52fb2a636a082cf5dc4c669a4b34085135709645':
  Handle nested signals

Support booting without functioning boot.oat/art patchoat.

Bug: 17000769

(cherry picked from commit 84d7605f93f1e6e86a16e02017e305c90e93117a)

Change-Id: I89c26a905af12ea288742368c2c038afd57a879a

Handle nested signals

This allows for signals to be raised inside the ART signal handler.
This can occur when the JavaStackTraceHandler attempts to generate
a stack trace and something goes wrong.

It also fixes an issue where the fault manager was not being
correctly shut down inside the signal chaining code.  In this
case the signal handler was not restored to the original.

Bug: 17006816
Bug: 17133266

(cherry picked from commit fabe91e0d558936ac26b98d2b4ee1af08f58831d)

Change-Id: I10730ef52d5d8d34610a5293253b3be6caf4829e

Merge "Handle nested signals" into lmp-dev

am a0a0da29: Merge "Really fix tests."

* commit 'a0a0da29e7d4d5c1bd471c49f1a4b6ec98fb767a':
  Really fix tests.

Merge "Really fix tests."

am b68c6e57: Reduce interpret-only compile time.

* commit 'b68c6e578a28a9717d78dfd522d9d9b8befaedf2':
  Reduce interpret-only compile time.

Really fix tests.

Remove extra line in Android.run-test.mk.
Update junit code in 082 and 021.
Set correct bootclasspath for --no-image.
Make host core.art depend on dex files being installed.
Make 118 pass in the correct bootclasspath.

Bug: 17290452

Change-Id: I415eddfa3632ec7eda927abe95925202de193749

am 469f2ee9: Merge "Various run-test makefile fixes."

* commit '469f2ee9a3c3d77f0352faed19d8c83c7397a638':
  Various run-test makefile fixes.

Merge "Various run-test makefile fixes."

Various run-test makefile fixes.

Reduce number of test combinations based on global flags, rather than just
specific tests, to keep known broken list as small as possible.
Fix dist goal known broken definition.
Fix naming of rules in define-test-art-run-test to match the make target.
Undefine more variables after use.

Change-Id: I4b4eadd37fc1f816bc61d78c5866bfcee7b60244

Reduce interpret-only compile time.

Before:
39.04user 5.18system 0:29.24elapsed 151%CPU (0avgtext+0avgdata 164176maxresident)k
38.87user 5.16system 0:29.14elapsed 151%CPU (0avgtext+0avgdata 164144maxresident)k

After:
36.26user 3.25system 0:27.00elapsed 146%CPU (0avgtext+0avgdata 162592maxresident)k
36.25user 3.28system 0:26.28elapsed 150%CPU (0avgtext+0avgdata 162688maxresident)k

Disabled implicit stack protection for the compiler, this reduces page faults.

Added support for not timing every method compilation and verification. NanoTime is
slow and adds ~2 seconds of real time. This is currently enabled since people want
to know which methods are slow to compile.

Bug: 16853450

(cherry picked from commit 8e219ae27624116b6d23e858fb21e93342f81d66)

Change-Id: I349ffb3f36db8c437137387aa6914dc17d743f09

Handle nested signals

This allows for signals to be raised inside the ART signal handler.
This can occur when the JavaStackTraceHandler attempts to generate
a stack trace and something goes wrong.

It also fixes an issue where the fault manager was not being
correctly shut down inside the signal chaining code.  In this
case the signal handler was not restored to the original.

Bug: 17006816
Bug: 17133266
Change-Id: I9c25bf4f6921e6a107aefbdf47d2c0db9f41508f

am 270c3e6f: Merge "Fix builds where dist_goal is defined."

* commit '270c3e6f95f047595a37ac15eb0734573377347d':
  Fix builds where dist_goal is defined.

Merge "Fix builds where dist_goal is defined."

Fix builds where dist_goal is defined.

Change-Id: I93b3fb468cf760aa84d792b2651cdde366fc6033

am 3d159704: Merge "Fix test build targets."

* commit '3d159704d1f03d120cef8ad67a7a65f436fe8d00':
  Fix test build targets.

Merge "Fix test build targets."

Fix test build targets.

Missing includes for libarttest.
Missed merge for adding IMAGE_TYPES to known broken tests.

Change-Id: I64227b3410bac9a1de8e0efd3b89eab78cd3cf5b

am ff092c2d: Merge changes Id3113019,I89c26a90

* commit 'ff092c2d5f672979c97f4c3b9d165e609ccf7ce0':
  Add run-test support for running without a working patchoat/dex2oat
  Support booting without functioning boot.oat/art patchoat.

am 709368e6: Merge "Reduce interpret-only compile time."

* commit '709368e616791209b02d39adb6da5e55782cb45f':
  Reduce interpret-only compile time.

am 2cdd2f2f: Merge "ART: Added dirty flag to optimizations"

* commit '2cdd2f2f4c18de88c0daeddec362c0471d823b0b':
  ART: Added dirty flag to optimizations

am ab66df8b: Merge "ART: Add capability for a pass to have options"

* commit 'ab66df8bc9e32133cec45c56f2c17d6d705bfb21':
  ART: Add capability for a pass to have options

ART: Change temporaries to positive names

Changes compiler temporaries to have positive names. The numbering now
puts them above the code VRs (locals + ins, in that order). The patch also
introduces APIs to query the number of temporaries, locals and ins.

The compiler temp infrastructure suffered from several issues
which are also addressed by this patch:
-There is no longer a queue of compiler temps. This would be polluted
with Method* when post opts were called multiple times.
-Sanity checks have been added to allow requesting of temps from BE
and to prevent temps after frame is committed.
-None of the structures holding temps can overflow because they are
allocated to allow holding maximum temps. Thus temps can be requested
by BE with no problem.
-Since the queue of compiler temps is no longer maintained, it is no
longer possible to refer to a temp that has invalid ssa (because it
was requested before ssa was run).
-The BE can now request temps after all ME allocations and it is guaranteed
to actually receive them.
-ME temps are now treated like normal VRs in all cases with no special
handling. Only the BE temps are handled specially because there are no
references to them from MIRs.
-Deprecated and removed several fields in CompilationUnit that saved
register information and updated callsites to call the new interface from
MIRGraph.

Change-Id: Ia8b1fec9384a1a83017800a59e5b0498dfb2698c
Signed-off-by: Razvan A Lupusoru <razvan.a.lupusoru@intel.com>
Signed-off-by: Udayan Banerji <udayan.banerji@intel.com>

Merge changes Id3113019,I89c26a90

* changes:
  Add run-test support for running without a working patchoat/dex2oat
  Support booting without functioning boot.oat/art patchoat.

Merge "Reduce interpret-only compile time."

Merge "ART: Added dirty flag to optimizations"

Add run-test support for running without a working patchoat/dex2oat

Also add it to the test-art target.

Bug: 17262039

Change-Id: Id31130194c46df66fc48852d6f8884f14cb08db4

Merge "ART: Add capability for a pass to have options"

ART: Add capability for a pass to have options

This patch adds capability to have pass options. These are needed when a pass
has multiple flags that can be tweaked. The user is now allowed to pass those
options via command line.

Since passes are treated as singletons and they are immutable, the overridden
options provided by user are set on the compilation unit. Doing this way also
allows a selectivity system to tweak the option per compilation instead of
doing it globally (due to the single pass existing).

The following command line flags have been added:
--print-pass-options - This prints all passes that have options along with
their defaults.
--pass-options= - This is used to pass the overridden options in format of
PassName:PassOption:PassOptionSetting

Change-Id: Ib5156f5d2ff51a0c64c4ea0fa050bd2170663417
Signed-off-by: Razvan A Lupusoru <razvan.a.lupusoru@intel.com>
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>

am a9ffc181: Merge "Use loops to build combinations of run-tests."

* commit 'a9ffc181c61f5634753bf1039ed50bf22c800fde':
  Use loops to build combinations of run-tests.

ART: Added dirty flag to optimizations

- Added a dirty flag to the PassMEDataHolder and use it to determine
    if we want to perform the post-optimization passes or not.

Change-Id: I59a85f5198f9224d5fe8893397bfaf68a978d8f1
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>

Support booting without functioning boot.oat/art patchoat.

Bug: 17000769

Change-Id: I89c26a905af12ea288742368c2c038afd57a879a

Merge "Use loops to build combinations of run-tests."

am 061d3103: Merge "ART: Add non-temporal store support"

* commit '061d3103991ab4c9a98a14228cbd4c8654da1ac8':
  ART: Add non-temporal store support

Use loops to build combinations of run-tests.

Add a JNI force copy test.
Split tracing from GC, relocation from prebuilding.
As the combinations of tests is sufficiently large to cause make serious work,
only build rules for tests that will be executed. Default to small number of
tests rather than full. Use ART_TEST_FULL=true to override.

Change-Id: Ic25431f29071b840d2e2295be6853511c0b52cfa

Merge "ART: Add non-temporal store support"

Reduce interpret-only compile time.

Before:
39.04user 5.18system 0:29.24elapsed 151%CPU (0avgtext+0avgdata 164176maxresident)k
38.87user 5.16system 0:29.14elapsed 151%CPU (0avgtext+0avgdata 164144maxresident)k

After:
36.26user 3.25system 0:27.00elapsed 146%CPU (0avgtext+0avgdata 162592maxresident)k
36.25user 3.28system 0:26.28elapsed 150%CPU (0avgtext+0avgdata 162688maxresident)k

Disabled implicit stack protection for the compiler, this reduces page faults.

Added support for not timing every method compilation and verification. NanoTime is
slow and adds ~2 seconds of real time. This is currently enabled since people want
to know which methods are slow to compile.

Bug: 16853450

Change-Id: I349ffb3f36db8c437137387aa6914dc17d743f09

ART: Add non-temporal store support

Added non-temporal store support as a hint from the ME.
Added the implementation of the memory barrier
extended instruction that supports non-temporal stores
by explicitly serializing all previous store-to-memory instructions.

Change-Id: I8205a92083f9725253d8ce893671a133a0b6849d
Signed-off-by: Jean Christophe Beyler <jean.christophe.beyler@intel.com>
Signed-off-by: Chao-ying Fu <chao-ying.fu@intel.com>

am 0038fcaa: Merge "Fix race condition in TransitionCollector."

* commit '0038fcaa6f2457f5de5544fc93cd57a305e466de':
  Fix race condition in TransitionCollector.

am 652e3a16: am f59b605f: Merge "Fix race condition in TransitionCollector." into lmp-dev

* commit '652e3a164fe1348de573dcdb4d79926592117bcd':
  Fix race condition in TransitionCollector.

am d6a4d7d8: am ccb1a5d4: ART: Reject field accesses to non-reference vregs.

* commit 'd6a4d7d8856785c79ba24426abcc0c684dc901f1':
  ART: Reject field accesses to non-reference vregs.

am 0b05ec67: Merge "ART: Improve error message on oat writer failure"

* commit '0b05ec674139c2b2e0914b0bb0e1898f93a9c852':
  ART: Improve error message on oat writer failure

am f59b605f: Merge "Fix race condition in TransitionCollector." into lmp-dev

* commit 'f59b605f82c0e2b7bfc6019ee7bd650628f50d3a':
  Fix race condition in TransitionCollector.

am ccb1a5d4: ART: Reject field accesses to non-reference vregs.

* commit 'ccb1a5d46bc1b4ee888e74775f5c1291d600ff54':
  ART: Reject field accesses to non-reference vregs.

Merge "Fix race condition in TransitionCollector."

Fix race condition in TransitionCollector.

There was a minor race condition that could happen if multiple
threads called TransitionCollector at the same time. Starting out
in CMS:

Thread 1: TransitionCollector(SS).
Thread 2: TransitionCollector(CMS) sees that it is already CMS so
has copying_transition == false. But then thread 2 changes to SS.
Thread 1 resumes but incorrectly doesn't check the
disable_moving_gc_count_ possibly resulting in errors if
disable_moving_gc_count_ != 0 due to JNI since we are going from
SS -> CMS and are goign to move objects.

(cherry picked from commit de2233bbf64c7dde96513a729f6a97aebc2f0424)

Change-Id: I2fdda85f7cb7cbb0df3d1466a3c74853c5304c1b

Merge "Fix race condition in TransitionCollector." into lmp-dev

Merge "ART: Improve error message on oat writer failure"

ART: Improve error message on oat writer failure

Try to log the pending exception to help resolve issues.

Bug: 16406811
Change-Id: I035ae9e59a5ee02b9e90c35c0644ec088d3c7d12

am 3b6711fa: Merge "Fix usage of literals within a macro."

* commit '3b6711faf7b0b10eaa6c48ba854160bcecd00166':
  Fix usage of literals within a macro.

Merge "Fix usage of literals within a macro."

Fix race condition in TransitionCollector.

There was a minor race condition that could happen if multiple
threads called TransitionCollector at the same time. Starting out
in CMS:

Thread 1: TransitionCollector(SS).
Thread 2: TransitionCollector(CMS) sees that it is already CMS so
has copying_transition == false. But then thread 2 changes to SS.
Thread 1 resumes but incorrectly doesn't check the
disable_moving_gc_count_ possibly resulting in errors if
disable_moving_gc_count_ != 0 due to JNI since we are going from
SS -> CMS and are goign to move objects.

Bug: 17189964

Change-Id: I4f0d002717516b81355d66f26e0e8ebe3958348d

ART: Reject field accesses to non-reference vregs.

The verifier will now reject any iget/iput insts
that do not have a reference object in vB.

Bug: 17207857
Signed-off-by: Stuart Monteith <stuart.monteith@arm.com>
(cherry picked from commit 695c5982eeddee18c643cc2f94c9d78fa75599ab)

Change-Id: I3790b8687673643bb142549d74f36a3e2978bf13

Fix usage of literals within a macro.

Macs handle arguments in macros as $num meaning literals must be $$num.

Change-Id: Ia30c7ef4e089561e9f23f353d7080e800b117d70

am 2f1f3b26: Merge "Map heap maps at a lower address."

* commit '2f1f3b26e360ed5362a3e6655ed0acdccea3a6a7':
  Map heap maps at a lower address.

am b7fbafda: Merge "ART: kSuper also has IncompatibleClassChangeError"

* commit 'b7fbafdac8cf0ff8f860e6e85646d6c88d3914ed':
  ART: kSuper also has IncompatibleClassChangeError

am 654881ff: Merge "ART: Fix IncompatibleClassChange check for ArtMethod"

* commit '654881ffd780066fe1160320059ae629416b4652':
  ART: Fix IncompatibleClassChange check for ArtMethod