am 4c015c74: Merge "Don\'t hardcode the location of the caller."

* commit '4c015c747417fd719d937260fdb6b04021562f50':
  Don't hardcode the location of the caller.

Merge "Don't hardcode the location of the caller."

Don't hardcode the location of the caller.

This is to avoid shooting ourselves in the foot when
dealing with inlined frames. Instead, use common methods
for fetching the caller and its dex pc.

Change-Id: I3467a7b50cf163022d332e80356f0aab747de252

am 27e542bd: Merge "ART: Fix broken Checker test"

* commit '27e542bd3c5430e4bf6fe265eb7e15eb4e401b77':
  ART: Fix broken Checker test

am 3e736b12: Merge "ART: Stricter matching of Checker lines"

* commit '3e736b12d62ea539b71bd9e6ce953347c0f09a3e':
  ART: Stricter matching of Checker lines

Merge "ART: Fix broken Checker test"

ART: Fix broken Checker test

MemoryBarrier test checked for Return instead of ReturnVoid. Checker
does not allow that after its recent update.

Change-Id: Id90f4ccc8a1416e24550b1f7cbf7fdae21a33783

Merge "ART: Stricter matching of Checker lines"

am bc842221: Merge "Mark CheckCast\'s and InstanceOf\'s input as !CanBeNull if used before in a NullCheck"

* commit 'bc84222168dcc4057e594ecdf8e76559fd07b706':
  Mark CheckCast's and InstanceOf's input as !CanBeNull if used before in a NullCheck

Merge "Mark CheckCast's and InstanceOf's input as !CanBeNull if used before in a NullCheck"

ART: Stricter matching of Checker lines

This patch refactors the MatchLines function of Checker and changes it
so that regular expressions must match entire words of C1vis output.
Previously, this could lead to false positives because assertion 'xyz'
translated to 'xyz.*' and hence testing for id 'i12' could be satisfied
by the presence of 'i123'.

Change-Id: Iaeb486c53519b450ea0cd40042ff3048b38d5a7b

am cc8bda03: Merge "Test and fix bug checking status of secondary multidex files."

* commit 'cc8bda030d7a18715d2c63bbb2528b61472bf3d5':
  Test and fix bug checking status of secondary multidex files.

Merge "Test and fix bug checking status of secondary multidex files."

am b258ce51: Merge "Fix build - large frame size of ElfWriterQuick<ElfTypes>::Write."

* commit 'b258ce51247e775079a73805ed17e2a6255bb0e6':
  Fix build - large frame size of ElfWriterQuick<ElfTypes>::Write.

Merge "Fix build - large frame size of ElfWriterQuick<ElfTypes>::Write."

Fix build - large frame size of ElfWriterQuick<ElfTypes>::Write.

Change-Id: I0d1b7f6ca18a992bb7619a08a2cb0ed538578410

am dc23a388: Merge "Eliminate redundant constructor barriers when inlining."

* commit 'dc23a3882b82a7a3dd5078cda3f4b4706e172617':
  Eliminate redundant constructor barriers when inlining.

am 9652135e: Merge "Support generation of CFI in .debug_frame format."

* commit '9652135e5c23767098d7c7960f9d7d06f7951f10':
  Support generation of CFI in .debug_frame format.

Merge "Eliminate redundant constructor barriers when inlining."

Eliminate redundant constructor barriers when inlining.

Bug: 20410297
Change-Id: I2097743d00eb795d050d390b1918e38c7f41d506

Merge "Support generation of CFI in .debug_frame format."

Support generation of CFI in .debug_frame format.

.debug_frame section is almost identical to .eh_frame section.
There are only minor differences in the CIE and FDE headers.
The main difference is that .eh_frame is intended to be used
at runtime for exception handling and is therefore allocated
within the running program whereas .debug_frame is not.

This makes .debug_frame easier to remove using standard tools.
"objcopy --strip-debug" removes .debug_frame, but not .eh_frame.
(although objcopy can be still be forced to remove .eh_frame)

Similarly, we might want to separate the CFI to separate debug
file or include it as compressed .gnu_debugdata section.
It is more appropriate to use .debug_frame for this purpose.

Bug:20556771
Change-Id: I9d91a333b9fb37523fd6fafccfad89b21d2477af

am cfee0507: Merge "Fix debuggable compiler flag detection for secondary dex files"

* commit 'cfee0507f26375fe048ed35f402e61f904db0095':
  Fix debuggable compiler flag detection for secondary dex files

Merge "Fix debuggable compiler flag detection for secondary dex files"

Fix debuggable compiler flag detection for secondary dex files

Compiles secondary dex files like the primary dex file: if it has
been compiled with the --debuggable flag, compile secondary dex files
with the --debuggable flag too.

Therefore, dex files loaded at runtime are compiled the same way as
dex files compiled at install time on the classpath (excluding the
boot image that is not compiled debuggable).

Also adds debuggable key in the oat header and bump the oat version.

Bug: 20944228
Change-Id: I59119f3468adb27ab1d6026f2cefbebbd814224c

am 48fb0b7b: Merge "Don\'t do a null test in MarkGCCard if the value cannot be null."

* commit '48fb0b7be18a783de9cd1b246042c1ec5b732c49':
  Don't do a null test in MarkGCCard if the value cannot be null.

Merge "Don't do a null test in MarkGCCard if the value cannot be null."

am 45970a4c: Merge "ART: Change huge-method warning in verifier"

* commit '45970a4cde2fb12e1cb1515aaf0d9cb9869c5116':
  ART: Change huge-method warning in verifier

Merge "ART: Change huge-method warning in verifier"

am be910330: Merge "ART: Add verbose:deopt logging"

* commit 'be910330a25117e0c739330747f7f6806c00acaf':
  ART: Add verbose:deopt logging

Merge "ART: Add verbose:deopt logging"

ART: Add verbose:deopt logging

Add verbose logging for deoptimization. Can be enabled with the
regular command line option "-verbose:_".

Change-Id: Iab63e02f3a4e0a213fc61efed1118fe829feeaa4

ART: Change huge-method warning in verifier

Change the logging to be included in the took-a-long-time warning,
instead of being a non-descriptive standalone. This should reduce
the noise.

Bug: 21149450
Change-Id: Ia4493dd12f54a8452a46541f6da86530584016ea

am 9a85fe78: Merge changes Ia1f2269a,I2ac93038

* commit '9a85fe78f45768fbb4d3e5e138716d107d399b6c':
  Print more info on mark sweep invalid ref crash.
  Enable tlab by default for the read barrier config.

Merge changes Ia1f2269a,I2ac93038

* changes:
  Print more info on mark sweep invalid ref crash.
  Enable tlab by default for the read barrier config.

Print more info on mark sweep invalid ref crash.

- Add extra info about the field holder.
- Suspend threads when verifying roots.

(cherry pick commit 1e1a47204984e3b72c4a753a2beb704350f38373)

Bug: 20557050
Change-Id: Ia1f2269aaa3b1a81d0594e781e439a5decfb82f9

Enable tlab by default for the read barrier config.

Bug: 12687968
Change-Id: I2ac930386aebe5aaacbccf1b950c376df0d9c81b

Don't do a null test in MarkGCCard if the value cannot be null.

Change-Id: I45687f6d3505178e2fc3689eac9cb6ab1b2c1e29

Mark CheckCast's and InstanceOf's input as !CanBeNull if used before in a NullCheck

Change-Id: Ied0412a01922b40a3f5d89bed49707498582abc1

am a5511adc: Merge "ART: Rephrase test comments, update Checker README"

* commit 'a5511adcd42bfef442ac6e4990bde26366253bbf':
  ART: Rephrase test comments, update Checker README

Merge "ART: Rephrase test comments, update Checker README"

am 2f9d1379: Merge "Opt compiler: ARM64: Use ldp/stp on arm64 for slow paths."

* commit '2f9d1379fdebcdeeac52eaeff25ad5697c6b6ffb':
  Opt compiler: ARM64: Use ldp/stp on arm64 for slow paths.

Test and fix bug checking status of secondary multidex files.

We were accidentally returning 'false' instead of 'true' to indicate a native
multidex file with an up-to-date primary dex file checksum but out-of-date
secondary dex file checksum is out of date.

Bug: 21022815
(cherry picked from commit a5a1c13a74225a1db3c54dc108fcd161b867bb00)

Change-Id: I0cc56d2d49858a4af91ca8b05aa8bd3c17bb44be

Merge "Opt compiler: ARM64: Use ldp/stp on arm64 for slow paths."

am 5969307a: Merge "ART: Mark constructor \'explicit\'"

* commit '5969307a254fb731a464119506b2cef9404871b9':
  ART: Mark constructor 'explicit'

Merge "ART: Mark constructor 'explicit'"

ART: Mark constructor 'explicit'

Fixes a cpp-lint error.

Change-Id: Ibd2c84a6f171a5ddb9aa1a3b9e23c1e5a30537f0

am 468137a7: Merge "ART: Revert change to LiveRange::Dump"

* commit '468137a73b35b3ed1c457983937fb4979d6b69c9':
  ART: Revert change to LiveRange::Dump

Merge "ART: Revert change to LiveRange::Dump"

am 58b1654e: Merge "Stack walking of inlined frames."

* commit '58b1654e657bc8dfc611fabe831baece3feae6c5':
  Stack walking of inlined frames.

Merge "Stack walking of inlined frames."

ART: Revert change to LiveRange::Dump

Changes back the LiveRange printing format to "[start,end)" for better
clarity. However, it removes the space after comma due to b/1189305
and prints the "ranges" attribute with curly brackets to improve
readability.

This is a resubmission of CL Ic83025fa78d6f1edb5e0e39d66160182b0198ab8
which fixes a compilation issue on target.

Bug: 21189305
Change-Id: Ic232c02ba19a710ead67793a039f99c0345353c7

am 4b49a861: Merge "Revert "ART: Revert change to LiveRange::Dump""

* commit '4b49a861e69580206988024e6987029340667628':
  Revert "ART: Revert change to LiveRange::Dump"

Stack walking of inlined frames.

Be "true" to the stack visitor design, which seems to be:
1) Carry minimal state, so this CL just adds an inlining depth to it.
2) Compute needed information for each method, which is what GetDexPc,
   GetMethod, and GetVRegFromOptimized code do.

Change-Id: I3d14126766f12216a7b448ce856259f1e7f37395

Merge "Revert "ART: Revert change to LiveRange::Dump""

Revert "ART: Revert change to LiveRange::Dump"

This reverts commit 294a0509cf811061b8009ade61e76a1f94e248b2.

Need to investigate a compilation error.

Change-Id: I62020f6e546c9faf5f4a2ed6cc1f7e1499c2fa30

ART: Rephrase test comments, update Checker README

Addressing minor comments on a previous CL.

Change-Id: Ic011adc578111fc162480ed09723ff5d48690fe5

am ce63c60a: Merge "ART: Revert change to LiveRange::Dump"

* commit 'ce63c60ad824495043c15f3fe9e649d88c15d60f':
  ART: Revert change to LiveRange::Dump

Merge "ART: Revert change to LiveRange::Dump"

ART: Revert change to LiveRange::Dump

Changes back the LiveRange printing format to "[start,end)" for better
clarity. However, it removes the space after comma due to b/1189305
and prints the "ranges" attribute with curly brackets to improve
readability.

Bug: 21189305
Change-Id: Ic83025fa78d6f1edb5e0e39d66160182b0198ab8

am 17f1bc53: Merge "Support InlineInfo in StackMap."

* commit '17f1bc531ea2f8c1a6fac3def13dee1b901949dd':
  Support InlineInfo in StackMap.

Merge "Support InlineInfo in StackMap."

am 9ee371e2: Merge "Fix libcore buildbot failure."

* commit '9ee371e2324d979ff7d11ac58b8201f29888c682':
  Fix libcore buildbot failure.

Merge "Fix libcore buildbot failure."

Fix libcore buildbot failure.

... due to a duplicate suppression. test_PacketSocketAddress has been
moved to libcore/knownfailures.txt.

Change-Id: I5c9cc98938d46607cba5a799e9c45b864ec22bf8

Support InlineInfo in StackMap.

Change-Id: I9956091775cedc609fdae7dec1433fcb8858a477

am 682876a1: Merge "Don\'t emit bkpt, takes extra space for nothing."

* commit '682876a1fc3f215e60766952968dd4dc1f66397d':
  Don't emit bkpt, takes extra space for nothing.

am 81b2591f: Merge "ART: Removed redundant Phis as part of DCE"

* commit '81b2591f7be6005b553d12a7cfa7ce2cdc08f22d':
  ART: Removed redundant Phis as part of DCE

Merge "Don't emit bkpt, takes extra space for nothing."

Merge "ART: Removed redundant Phis as part of DCE"

ART: Removed redundant Phis as part of DCE

Removing dead blocks can render some phis redundant but they do not
get removed. This patch adds running SsaRedundantPhiElimination to
adress that.

Change-Id: Iec9a16f3c20b2d5b5607cd5aaf802917c155a0f1

am d30ad93c: Merge "Callee/caller save logic in register allocator."

* commit 'd30ad93c9bd31dc03c1e08b92c4733db2dfad849':
  Callee/caller save logic in register allocator.

Merge "Callee/caller save logic in register allocator."

am c88b09fc: Merge "ART: Print C1vis lists as [v1,...,vN]"

* commit 'c88b09fc1d67413e814a10d20752e46aa28bc0e4':
  ART: Print C1vis lists as [v1,...,vN]

Callee/caller save logic in register allocator.

Prevent intervals that do not span a 'will-call' safepoint
to allocate a callee-save register when caller-saves
are available.

Change-Id: I6e613ab54b087f433bbc433aa62847fbca423377

Merge "ART: Print C1vis lists as [v1,...,vN]"

Don't emit bkpt, takes extra space for nothing.

Change-Id: Id091dcdc950b590e5be55c4fa31d7779308e1a2c

am aae4334a: Merge "JDWP: more GC safety"

* commit 'aae4334a3f5215edbfe90ab00f8e1e9ade542007':
  JDWP: more GC safety

Merge "JDWP: more GC safety"

am 03e4351d: Merge "Add new style String operations during compilation init, part 2"

* commit '03e4351df62ade9fa0e9139ece28f23ba85c49c2':
  Add new style String operations during compilation init, part 2

Merge "Add new style String operations during compilation init, part 2"

Add new style String operations during compilation init, part 2

This allows some class initializers that deal with Strings to run during
compilation. However, java.lang.Character.toUpperCase() is blocking
initialization for some parts.

Bug: 21036900
Change-Id: Ia969c6a643f510bc798f94fb10bca1a68fe2ae67

am 74ef4424: Merge "Stop wrapping close(2) with TEMP_FAILURE_RETRY in art."

* commit '74ef4424040ce38945825ac7b75e74de96093145':
  Stop wrapping close(2) with TEMP_FAILURE_RETRY in art.

Merge "Stop wrapping close(2) with TEMP_FAILURE_RETRY in art."

ART: Print C1vis lists as [v1,...,vN]

Checker interprets whitespace as a don't-care placeholder, making it
easier to write assertions which test only parts of the output (e.g.
"//CHECK: Add liveness:44" does not test the inputs or any other
attributes apart from "liveness").

However, since the GraphVisualizer prints lists with elements
separated by spaces ("[ v1 ... vN ]"), this allows for false positives
caused by an occurrence elsewhere in the output. For example, the
assertion: "//CHECK: [ x y ]" will match "[ x y ]" but also
"[ x a y b ]" or even "[ x ] abc [ y ]".

Switching to comma-separated lists works around this issue.

This patch updates all test files, fixes one false positive that this
change revealed (test 442, line 337) and two occurrences of a wrong
match (test 462, lines 121, 149).

Bug: 21189305
Change-Id: I3b22503be3d92529dac0b13f66bccbcfabea6721

am 99df2b15: Merge "Opt compiler: Speedup div/rem by constants on arm32 and arm64."

* commit '99df2b150039fa2a3e1132a8cd5e35201ae97aee':
  Opt compiler: Speedup div/rem by constants on arm32 and arm64.

Stop wrapping close(2) with TEMP_FAILURE_RETRY in art.

Note that all the TEMP_FAILURE_RETRY([A-Z] calls in this file are
wrong. The member functions return -errno, so only an EPERM failure
will return -1 (but errno should be EPERM, so you won't loop).

Bug: http://b/20501816
Change-Id: Ibde6a2c7bf92469d008d6d510f9391669ae37a84

Merge "Opt compiler: Speedup div/rem by constants on arm32 and arm64."

Opt compiler: Speedup div/rem by constants on arm32 and arm64.

This patch also includes:
1. Add java test for div/rem negative constants.
2. Fix a thumb2 encoding issue where the last operand is
   "reg, shift #amount" in some instructions.
3. Support a simple filter in arm32 assembler test to filter out
   unsupported cases, such as "smull r0, r0, r1, r2".
4. Add smull arm32 assembler test.
5. Add smull/umull thumb2 test.
6. Add test for the thumb2 encoding issue which is fixed in this
   patch.

Change-Id: I1601bc9c38f70f11909f2816fe3ec105a158951e

am 06675720: Merge "ART: Fix a minor lint error"

* commit '06675720cc274eb9d91a4ecd3fb1eef94a2d88f0':
  ART: Fix a minor lint error

Merge "ART: Fix a minor lint error"

ART: Fix a minor lint error

Change-Id: I956656fa4c36f20d80ef885c03487970f707be8e

am 590d5767: Merge "ART: Change Checker syntax for variables"

* commit '590d5767104757399c956400df9c5edc3a2c379e':
  ART: Change Checker syntax for variables

Merge "ART: Change Checker syntax for variables"

ART: Change Checker syntax for variables

This patch changes the syntax for defining and referencing variables
from square brackets '[[VAR]]' to angle brackets '<<VAR>>'. This is
done in order to avoid clashes when matching against lists enclosed
in square brackets.

Change-Id: Ib9560b07bdc7bd641fd1eb6143c0063e0783f1eb

am 1cad536d: Merge "ART: Refactor GraphVisualizer attribute printing"

* commit '1cad536d675846ac9c110b5b3a412dfc55ace3ed':
  ART: Refactor GraphVisualizer attribute printing

Merge "ART: Refactor GraphVisualizer attribute printing"

ART: Refactor GraphVisualizer attribute printing

This patch unifies the way GraphVisualizer prints instruction
attributes in preparation of changes to the Checker syntax.

Change-Id: I44e91e36c660985ddfe039a9f410fedc48b496ec

am 38b06de7: Merge "Fix minor checker bug"

* commit '38b06de7de49400d83bfe3221649a0e1a1a96dec':
  Fix minor checker bug

Merge "Fix minor checker bug"