am b2a7ec2a: Merge "Disable 114-ParallelGC for GC stress testing."

* commit 'b2a7ec2ad3f24c4094185cbf87bd0a39f727ffe7':
  Disable 114-ParallelGC for GC stress testing.

Merge "Disable 114-ParallelGC for GC stress testing."

Disable 114-ParallelGC for GC stress testing.

Bug: 17387969
Change-Id: Ic50e9642468646c36afa5cfbe4050d24e9190ae6

am a4fee49a: Don\'t call dlsym from signal context in signal chain

* commit 'a4fee49a5023a3cf8431597950ee87c711d0b81c':
  Don't call dlsym from signal context in signal chain

am acfbbd4d: Merge "Cleanup baseline register allocator."

* commit 'acfbbd4df2fc1c79a7102587bebf398f95b5e5de':
  Cleanup baseline register allocator.

Merge "Cleanup baseline register allocator."

Cleanup baseline register allocator.

- Use three arrays for blocking regsters instead of
  one and computing offsets in that array.]
- Don't pass blocked_registers_ to methods, just use the field.

Change-Id: Ib698564c31127c59b5a64c80f4262394b8394dc6

am b76c5495: Merge "Fix code generation of materialized conditions."

* commit 'b76c5495c4879fcfa0866b1490031a3123baf9ee':
  Fix code generation of materialized conditions.

Merge "Fix code generation of materialized conditions."

am 4471609d: Merge "Only watch location for BREAKPOINT event"

* commit '4471609d86b7e846b26bebe3373707a10c222e71':
  Only watch location for BREAKPOINT event

Merge "Only watch location for BREAKPOINT event"

am 6b0518f6: Merge "Enable -Wimplicit-fallthrough."

* commit '6b0518f6cfa79b35c067d8bc4d50d054cfc6656c':
  Enable -Wimplicit-fallthrough.

Merge "Enable -Wimplicit-fallthrough."

Enable -Wimplicit-fallthrough.

Falling through switch cases on a clang build must now annotate the fallthrough
with the FALLTHROUGH_INTENDED macro.
Bug: 17731372

Change-Id: I836451cd5f96b01d1ababdbf9eef677fe8fa8324

Don't call dlsym from signal context in signal chain

It is dangerous to call dlsym from within a signal context
since it takes a lock and can lead to a mutex reentry attempt if
timing is bad.

This change adds an initialization function to the signal chain
that calls dlsym for sigaction and sigprocmask from outside the
signal context (from Runtime::Init()).  The results are cached
in a static variable and used from within the signal context if
necessary.

However, tests don't necessarily call Runtime::Init() so we also
need to deal with the case where the signal chain is not initialized
and perform a lazy initialization from inside sigaction or sigprocmask.
This is always outside a signal context since we have not initialized
the runtime.

Bug: 17498571, 17896006

(cherry picked from commit cefcea838729287a04174664a76514dd793dd77d)

Change-Id: I9bf8540a1250eadf977ff9af249dbe1c73b5ac63

am 8fa8c904: Merge "Fix object header size for baker read barrier."

* commit '8fa8c904f7c783204a1dc9438429391d256658da':
  Fix object header size for baker read barrier.

Merge "Fix object header size for baker read barrier."

Fix object header size for baker read barrier.

Change-Id: I2b4a61488b9017d22210d79a07f485f9afef3b42

am 182c9f66: Merge "stdint types all the way!"

* commit '182c9f660993abf05825a2d532fe5bba420f11f0':
  stdint types all the way!

Merge "stdint types all the way!"

stdint types all the way!

Change-Id: I4e4ef3a2002fc59ebd9097087f150eaf3f2a7e08

am 25b18bbd: Merge "Simplify instance reference offsets."

* commit '25b18bbdaa36ff936eb44f228f0518d4223e9d52':
  Simplify instance reference offsets.

Merge "Simplify instance reference offsets."

am 98672681: Merge "Fix race in thread list destructor"

* commit '9867268143038e6037c80d4aaaca9a94d538ac01':
  Fix race in thread list destructor

Simplify instance reference offsets.

Don't encode Object's class. Use trailing rather than leading zeroes to give
offset position.

Change-Id: I1ae74e7a01f63696429644adf81cdf6ee58832fe

am b90132cb: Fix race in thread list destructor

* commit 'b90132cb5132eaeb4dbfca1e63d79b4005dacec5':
  Fix race in thread list destructor

Merge "Fix race in thread list destructor"

Fix race in thread list destructor

The ThreadList::Contains wasn't being guarded by the thread list
lock, which could cause a corrupted thread list if another thread
unregistered itself while std::find was searching the thread list.

Bug: 17896374

(cherry picked from commit b90132cb5132eaeb4dbfca1e63d79b4005dacec5)

Change-Id: I8f6f979f365ca00ac0655e04eb26020d2ad7d6ee

Fix race in thread list destructor

The ThreadList::Contains wasn't being guarded by the thread list
lock, which could cause a corrupted thread list if another thread
unregistered itself while std::find was searching the thread list.

Bug: 17896374
Change-Id: I629b1eac3473e3cf7355ebde9080dfd7c58af73a

am bbb1fc6e: Merge changes Ia0044523,Ieefe83cf

* commit 'bbb1fc6e95f46470979936450b0d3f1e020c5904':
  Fix ARM64 build.
  ARM64: Update code after the VIXL 1.6 release.

Merge changes Ia0044523,Ieefe83cf

* changes:
  Fix ARM64 build.
  ARM64: Update code after the VIXL 1.6 release.

Fix ARM64 build.

Change-Id: Ia0044523473d7a93ef3f0eed8e6c28602e767c17

am 349e486f: Merge "Fix arm64 build: move macro usage after definition."

* commit '349e486f5e6c8f8dd482641a27f9e7cce04ea623':
  Fix arm64 build: move macro usage after definition.

am 4ecaf87a: Merge "ART: X86: GenLongArith should handle overlapped VRs"

* commit '4ecaf87a2b62f4e51ee4c1f8b32aaea1473187a2':
  ART: X86: GenLongArith should handle overlapped VRs

Merge "Fix arm64 build: move macro usage after definition."

Merge "ART: X86: GenLongArith should handle overlapped VRs"

Fix arm64 build: move macro usage after definition.

The build was broken by
    https://android-review.googlesource.com/110341

Change-Id: I137b4bc694af15465bf0db3ffe337950064d2a53

am a1d1fc2b: Merge "Restore valgrind-test-art-host"

* commit 'a1d1fc2bed0d11a380bc9bdad5f7f3c69d75bd6d':
  Restore valgrind-test-art-host

Merge "Restore valgrind-test-art-host"

Restore valgrind-test-art-host

Change-Id: I2586e16664dabfc543bc9d0dab471ec8d3810f34

ART: X86: GenLongArith should handle overlapped VRs

In a case, when src and dest VRs are overlapped when we called
GenLongArith it may cause the incorrect use of regs.

The solution is to map src to an physical reg and work with this
reg instead of mem.

Renamed BadOverlap() to PartiallyIntersects() for consistency.

Change-Id: Ia3fc7f741f0a92556e1b2a1b084506662ef04c9d
Signed-off-by: Katkov, Serguei I <serguei.i.katkov@intel.com>
Signed-off-by: Alexei Zavjalov <alexei.zavjalov@intel.com>

am 88dd36d2: Merge "Makefile cleanups."

* commit '88dd36d2d1f0df6d8263ada619497742a4dd21a8':
  Makefile cleanups.

Merge "Makefile cleanups."

Makefile cleanups.

Also add the DEX2OAT_FLAGS to the generation of core.oat/core.art.

Change-Id: I5d113fad460b2e14fc652816069adba29ca744b8

Only watch location for BREAKPOINT event

This CL ensures we watch a location for BREAKPOINT event only. Other
events (single-step, method entry/exit, ...) are handled differently
and LocationOnly modifier is used as an event filter in this case.

This prevents from failing a check when we need to deoptimize for
non-breakpoint event.

Bug: 17908144
Change-Id: Ib413d62fa31480fec8d750543c0605ba52188350

ARM64: Update code after the VIXL 1.6 release.

We now leave the assembler buffer management to VIXL.

Change-Id: Ieefe83cf5cf5e1ab8c924b0e7dc03af6a55053ae

am 9e878d50: Merge "Tidy ELF builder."

* commit '9e878d50567f624094f3c4940ac3aedbc5eff3b9':
  Tidy ELF builder.

Merge "Tidy ELF builder."

am 51205042: Merge "Quick: Reduce memory usage and improve compile time."

* commit '51205042343fa4852e01919de01fdcf261af510b':
  Quick: Reduce memory usage and improve compile time.

am 2da74f2d: Merge "Quick: Separate null check elimination and type inference."

* commit '2da74f2de211b7ecb39464e641fa9794dc1b67f4':
  Quick: Separate null check elimination and type inference.

Merge "Quick: Reduce memory usage and improve compile time."

Merge "Quick: Separate null check elimination and type inference."

Tidy ELF builder.

Don't do "if (ptr)". Use const. Use DISALLOW_COPY_AND_ASSIGN. Avoid public
member variables.
Move ValueObject to base and use in ELF builder.
Tidy VectorOutputStream to not use non-const reference arguments.

Change-Id: I2c727c3fc61769c3726de7cfb68b2d6eb4477e53

Quick: Reduce memory usage and improve compile time.

Move the def-block-matrix from Arena to ScopedArena. Remove
BasicBlockDataFlow::ending_check_v and use a temporary bit
matrix instead. Remove unused BasicBlockDataFlow::phi_v.
Avoid some BitVector::Copy() at the end of null and clinit
check elimination passes when the contents of the source
BitVector is no longer needed.

Change-Id: I8111b2f8a51e63075aa124b528d61b79b6933274

Quick: Separate null check elimination and type inference.

Change-Id: I4566ae9354c91ca935481cb4f5b729bba05c1592

Fix code generation of materialized conditions.

Move the logic for knowing if a condition needs to be materialized
in an optimization pass (so that the information does not change
as a side effect of another optimization).

Also clean-up arm and x86_64 codegen:
- arm: ldr and str are for power-users when a constant is
  in play. We should use LoadFromOffset and StoreToOffset.
- x86_64: fix misuses of movq instead of movl.

Change-Id: I01a03b91803624be2281a344a13ad5efbf4f3ef3

am 2072c465: Merge "Stop converting from Location to ManagedRegister."

* commit '2072c465cfff077da257bdf14f1f1b2690c946c8':
  Stop converting from Location to ManagedRegister.

Merge "Stop converting from Location to ManagedRegister."

Stop converting from Location to ManagedRegister.

Now the source of truth is the Location object that knows
which register (core, pair, fpu) it needs to refer to.

Change-Id: I62401343d7479ecfb24b5ed161ec7829cda5a0b1

am b6404fe0: Disable kEnableJavaStackTraceHandler

* commit 'b6404fe0bb782cbbccda88903901fe0096a741d5':
  Disable kEnableJavaStackTraceHandler

am a3c4d722: Merge "Disable kEnableJavaStackTraceHandler"

* commit 'a3c4d72210de174552f47b2d117b1946f274af1e':
  Disable kEnableJavaStackTraceHandler

Merge "Disable kEnableJavaStackTraceHandler"

Disable kEnableJavaStackTraceHandler

Bug: 17890306

(cherry picked from commit b6404fe0bb782cbbccda88903901fe0096a741d5)

Change-Id: Ie13d6af1d384c6ebf87e8f055120609e2bff3338

am a9b09744: Merge "Don\'t copy fill array data to quick literal pool."

* commit 'a9b097443f26a6df1a322a8fef50fdd1f26a410b':
  Don't copy fill array data to quick literal pool.

Merge "Don't copy fill array data to quick literal pool."

Don't copy fill array data to quick literal pool.

Currently quick copies the fill array data from the dex file to the literal
pool. It then has to go through hoops to pass this PC relative address down
to out-of-line code. Instead, pass the offset of the table to the out-of-line
code and use the CodeItem data associated with the ArtMethod. This reduces
the size of oat code while greatly simplifying it.
Unify the FillArrayData implementation in quick, portable and the interpreters.

Change-Id: I9c6971cf46285fbf197856627368c0185fdc98ca

am 47798949: Merge "Dump mutator lock for thread suspend timeout"

* commit '477989496bfb8d1d9687d7ad8ee5b9ede80dc410':
  Dump mutator lock for thread suspend timeout

Merge "Dump mutator lock for thread suspend timeout"

Dump mutator lock for thread suspend timeout

Should help us see if thread suspend timeouts are due to someone
holding on the mutator lock for too long or if it is a missed
futex wake.

(cherry picked from commit f924d2381a5ea53967ba1e279766d601b9be05ea)

Bug: 17837911
Change-Id: I3417156a78f574076d5da17c86836c5a7c76084a

Disable kEnableJavaStackTraceHandler

Bug: 17890306
Change-Id: I76d1d1b709cfcb0cf8fe02f5d28fb746909d682f

am f1f05d30: Merge "Really fix --gdb when running tests on target."

* commit 'f1f05d303988a5c071c87b760056be8358276c94':
  Really fix --gdb when running tests on target.

Merge "Really fix --gdb when running tests on target."

am 175b2732: Merge "Header file clean up."

* commit '175b2732f7e55a07094c30aa08d946189b4af97e':
  Header file clean up.

Merge "Header file clean up."

Header file clean up.

Remove runtime.h from object.h.
Move TypeStaticIf to its own header file to avoid bringing utils.h into
allocator.h.
Move Array::DataOffset into -inl.h as it now has a utils.h dependency.
Fix include issues arising from this.

Change-Id: I4605b1aa4ff5f8dc15706a0132e15df03c7c8ba0

am edc34c88: Merge "ART: fix NullCheckElimination to preserve MIR_IGNORE_NULL_CHECK"

* commit 'edc34c88b8f8abd04f9c4668787403608cf0b2d4':
  ART: fix NullCheckElimination to preserve MIR_IGNORE_NULL_CHECK

am 254aa0d3: Merge "ART: Fix SelectKind to work with nullptr"

* commit '254aa0d3f58b56b227077ef32b4606ebc25eaae4':
  ART: Fix SelectKind to work with nullptr

am 1293b6b2: Merge "ART: Fix issues with SplitBlock"

* commit '1293b6b2adba33e8b5eca5632dd90c9835b2ef42':
  ART: Fix issues with SplitBlock

am 893e8558: Merge "Fix to MirGraph::GetSSANameWithConst()"

* commit '893e855864b7535f7da92f3942d3f88aaa584dd8':
  Fix to MirGraph::GetSSANameWithConst()

Merge "ART: fix NullCheckElimination to preserve MIR_IGNORE_NULL_CHECK"

Merge "ART: Fix SelectKind to work with nullptr"

Merge "ART: Fix issues with SplitBlock"

Merge "Fix to MirGraph::GetSSANameWithConst()"

am 663796fb: Merge "Fix and improve static evaluation of constant expressions."

* commit '663796fb14accb722e9b37c4fe8c3d2588680d7e':
  Fix and improve static evaluation of constant expressions.

Merge "Fix and improve static evaluation of constant expressions."

Fix and improve static evaluation of constant expressions.

- Fix the definition of art::HSub::Evaluate.
- Qualify Evaluate methods as OVERRIDE.
- Evaluate comparisons in a deterministic way: if a
  comparison is true, always return 1 (instead of letting
  the compiler return any non-null value).
- Better exercise static evaluation of constant expressions
  in compiler/optimizing/constant_propagation_test.cc.

Change-Id: I13d0862e5f4eba1275016fb8c3c17e9aff54408b

ART: Fix issues with SplitBlock

The SplitBlock is a public interface from MIRGraph to be able to create two
blocks from one. This interface can be called not just during the control
flow graph building phase, but later. At that later point, extended instructions
may have already been inserted.

For example, SpecialMethodInliner can insert kMirOpNullCheck and kMirOpNop,
and then another pass tries to split block that contains this. The DCHECK
will fail even though the split is semantically correct.

Signed-off-by: Mathew Zaleski <mathew.zaleski@intel.com>
Conflicts:

compiler/dex/mir_graph.cc

Change-Id: I8bb3551407907909cda444a3379da2eb9331b9ff

am 50940ea8: Merge "Implement method calls using relative BL on ARM64."

* commit '50940ea8fe668a80c15920d98cae228851ceb990':
  Implement method calls using relative BL on ARM64.

Merge "Implement method calls using relative BL on ARM64."

Really fix --gdb when running tests on target.

Change-Id: Ic96edb066d345839a4af371e7c10327713a045c7

am 1000e69b: Merge "Add trivial register hints to the register allocator."

* commit '1000e69b7e11348f2e1d3ba67339616a647f53d7':
  Add trivial register hints to the register allocator.

Merge "Add trivial register hints to the register allocator."

Add trivial register hints to the register allocator.

- Add hints for phis, same as first input, and expected registers.
- Make the if instruction accept non-condition instructions.

Change-Id: I34fa68393f0d0c19c68128f017b7a05be556fbe5

am a9f29042: Merge "Add support for floats and doubles."

* commit 'a9f2904263581f606a5704f2bb74efcecf7e9f97':
  Add support for floats and doubles.

Merge "Add support for floats and doubles."

Add support for floats and doubles.

- Follows Quick conventions.
- Currently only works with baseline register allocator.

Change-Id: Ie4b8e298f4f5e1cd82364da83e4344d4fc3621a3

am 4a1b4679: Merge "Inserting a node must also update its inputs users."

* commit '4a1b4679cda2f0d2893b8e3f910c21231849291c':
  Inserting a node must also update its inputs users.

Merge "Inserting a node must also update its inputs users."

Inserting a node must also update its inputs users.

Change-Id: I55357564b81efcc0cf52fffdf23289696fe27dd1

am 0d6102a8: Merge "Propagate optimization flags to kMirOpCheck."

* commit '0d6102a80ec741d701788decfb701a657d7a98f7':
  Propagate optimization flags to kMirOpCheck.