am f06f82dd: am fc298560: Merge "ART: Remove TODO in BitVector"

* commit 'f06f82dd69d274633e5e6378e3fe30d232573554':
  ART: Remove TODO in BitVector

am fc298560: Merge "ART: Remove TODO in BitVector"

* commit 'fc29856047d90d8f446d73b576c0c18de7a1380f':
  ART: Remove TODO in BitVector

Merge "ART: Remove TODO in BitVector"

am 672485f3: am e6b307a8: Merge "Add class table field to class loader"

* commit '672485f3fcfc1e28389e4203b8600df75b8df5c6':
  Add class table field to class loader

ART: Remove TODO in BitVector

Refactor the BitVector constructor: split it up to remove the
possibility to provide contradicting parameters, and add a custom
copying constructor.

Change-Id: Ie943f279baa007db578aea0f2f33fa93311612ee

am e6b307a8: Merge "Add class table field to class loader"

* commit 'e6b307a833d3a0ba4327004c2a286680e01eae3b':
  Add class table field to class loader

am 12a00ef3: am 4b2e98fc: Merge "Fix C++14 bitrot."

* commit '12a00ef305c8ef5b2f2c8e361dd09de77ff90d09':
  Fix C++14 bitrot.

Merge "Add class table field to class loader"

am 4b2e98fc: Merge "Fix C++14 bitrot."

* commit '4b2e98fc8c257a27a95fecbc859fcc617e2fa827':
  Fix C++14 bitrot.

Merge "Fix C++14 bitrot."

Fix C++14 bitrot.

Change-Id: I22f7925a87bca45646c563971342a9a1b2090ba6

Add class table field to class loader

Fixes bug with the class table where the comparator would cause read
barriers and break the map strict ordering properties.

Bug: 22957957

Change-Id: I8dbc042db6e22e2172ab4ec58ddf1db0345dcaaa

am fee5335c: am 3ded7766: Merge "Ensure coherency of call kinds for LocationSummary."

* commit 'fee5335c4253b4f06632033e24233ad4a45853cb':
  Ensure coherency of call kinds for LocationSummary.

am a7808cb3: am a001e3a0: Merge "Revert "Pass the verified method to the inlinee DexCompilationUnit.""

* commit 'a7808cb3e25ff91ac5ce4c8c17bd8757dc23c3f4':
  Revert "Pass the verified method to the inlinee DexCompilationUnit."

am b23623e7: am 4320ff10: Merge "Pass the verified method to the inlinee DexCompilationUnit."

* commit 'b23623e740f0a934e9ded4214d74e4c9a418d0d1':
  Pass the verified method to the inlinee DexCompilationUnit.

resolved conflicts for merge of bf3cc9e0 to mnc-dev-plus-aosp

Change-Id: I6079eac15e82065c437391ea5d7491226925c066

am f1912607: am 33407564: Merge "Introduce arch-specific checker tests."

* commit 'f19126074d22cd16df1ea299e5f3403c52d00eb9':
  Introduce arch-specific checker tests.

am 3ded7766: Merge "Ensure coherency of call kinds for LocationSummary."

* commit '3ded7766981b3105c7686e0c9d75c378c42fa4a0':
  Ensure coherency of call kinds for LocationSummary.

Merge "Ensure coherency of call kinds for LocationSummary."

am d333389c: Tighten default inlining settings when using the space filter.

* commit 'd333389cf635e002658a77e64547631e05004447':
  Tighten default inlining settings when using the space filter.

am a001e3a0: Merge "Revert "Pass the verified method to the inlinee DexCompilationUnit.""

* commit 'a001e3a0200fe137feeaaead3c99026abce914cd':
  Revert "Pass the verified method to the inlinee DexCompilationUnit."

Merge "Revert "Pass the verified method to the inlinee DexCompilationUnit.""

Revert "Pass the verified method to the inlinee DexCompilationUnit."

Crashes testing libcore on host/x86.

Bug:22866717

This reverts commit 08624c50f488ecd845e48683e84726ff8c53e6a7.

Change-Id: Ia8f130d9ea32e171e554bd3e10c44f8453111b52

am 4320ff10: Merge "Pass the verified method to the inlinee DexCompilationUnit."

* commit '4320ff10bb344140b75e029c6a4d46b070399682':
  Pass the verified method to the inlinee DexCompilationUnit.

Merge "Pass the verified method to the inlinee DexCompilationUnit."

Tighten default inlining settings when using the space filter.

(cherry picked from commit a215b95d03cfe713018a245553b74d7eeee813df
and amended)

Bug: 21868508
Change-Id: Ic83813a966cef18e59447083926bf033aa587154

am 62dee441: Merge "Tighten default inlining settings when using the space filter."

* commit '62dee441fff15e951c577b9565f87bb5f27c5ffb':
  Tighten default inlining settings when using the space filter.

Merge "Tighten default inlining settings when using the space filter."

Tighten default inlining settings when using the space filter.

Bug: 21868508
Change-Id: Ic83813a966cef18e59447083926bf033aa587154

am f2ea71cd: ART: Fix the simplifier for add/sub

* commit 'f2ea71cdb3ee4f5198bc0298aa8be1f9e945ee1c':
  ART: Fix the simplifier for add/sub

Ensure coherency of call kinds for LocationSummary.

The coherency is enforced with checks added in the `InvokeRuntime`
helper, that we now also use on x86 and x86_64.

Change-Id: I8cb92b042f25dc3c5fd390e9c61a45b477d081f4

am 33407564: Merge "Introduce arch-specific checker tests."

* commit '33407564904d2186f536107e1ca8d88f2c760c83':
  Introduce arch-specific checker tests.

Merge "Introduce arch-specific checker tests."

Pass the verified method to the inlinee DexCompilationUnit.

Sharpening and other optimizations in compiler driver rely
on having it there.

Also fixes a warning for the intrinsics code.

bug:22866717
Change-Id: I6ac4f55e9e77452a4ea9e0d0dd0ad70124a95a55

am 0c3ddba0: am 7fbaa22e: Merge "ART: Fix the simplifier for add/sub"

* commit '0c3ddba0f3859bd518e07d90ce8936a0b225b228':
  ART: Fix the simplifier for add/sub

am 7fbaa22e: Merge "ART: Fix the simplifier for add/sub"

* commit '7fbaa22e10979cf17b8c1c0c96bef4919e9d4163':
  ART: Fix the simplifier for add/sub

ART: Fix the simplifier for add/sub

Instruction simplifier for add/sub should not proceed with floats
because that might cause the incorrect behavior with signed zero.

Bug: 23001681

Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>
(cherry picked from commit 115b53f609e74672fa93eea1845bb17340d5112a)

Change-Id: I9928724c4158b3961e32e376b9203fe01ba2e442

Merge "ART: Fix the simplifier for add/sub"

ART: Fix the simplifier for add/sub

Instruction simplifier for add/sub should not proceed with floats
because that might cause the incorrect behavior with signed zero.

Change-Id: If0c9bf3931bcbf96b0814f8605a86997aea37145
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>

am ce4b1329: ART: x86_64 RoundDouble/Float intrinsics should initialize out value.

* commit 'ce4b1329ca903d6b98734a27a46b54bb9cfd6d5b':
  ART: x86_64 RoundDouble/Float intrinsics should initialize out value.

am c5e25863: am a27a67c9: Merge "Fix image loading in interpeter-only mode"

* commit 'c5e25863d3acae834399701248840bbf51ceb36e':
  Fix image loading in interpeter-only mode

am 906e0def: am bc7db535: Merge "Fix DDM reply buffer deallocation"

* commit '906e0def4a08d7ae262b03d7d344997c0cf8fae4':
  Fix DDM reply buffer deallocation

am a27a67c9: Merge "Fix image loading in interpeter-only mode"

* commit 'a27a67c98e12cd1a90431704a6d7bcbd546667fd':
  Fix image loading in interpeter-only mode

am bc7db535: Merge "Fix DDM reply buffer deallocation"

* commit 'bc7db5356a68cf44d6497f6956e8a97dc9355b80':
  Fix DDM reply buffer deallocation

Merge "Fix image loading in interpeter-only mode"

Merge "Fix DDM reply buffer deallocation"

Introduce arch-specific checker tests.

- The '.cfg' output is now created on target.
- Arch-specific checker tests can be created by inserting a
  suffix. For example:
      /// CHECK-START-ARM64: int Main.foo(int) register (after)
      /// CHECK-DAG:   <<Arg:i\d+>>     ParameterValue

Change-Id: I55cdb37f8e806c7ffdde6b676c8f44ac30b59051

Fix image loading in interpeter-only mode

Follow-up https://android-review.googlesource.com/162935. We now need
an ArtMethodVisitor to update entrypoints of methods in the image.

Bug: 22832610
Change-Id: I30ca9c369a73c3372694b446e73afa2e37890a65

am f1bb39e8: am 1f3f766d: Merge "Move ArtFields and ArtMethods to be a length prefixed array"

* commit 'f1bb39e81c12a11ed10e960a0efe8d2889de43aa':
  Move ArtFields and ArtMethods to be a length prefixed array

am 1f3f766d: Merge "Move ArtFields and ArtMethods to be a length prefixed array"

* commit '1f3f766d3b365d01f36b85dc19d40f754fa48533':
  Move ArtFields and ArtMethods to be a length prefixed array

Fix DDM reply buffer deallocation

Reply buffer is an uint8_t* allocated with new uint8_t[...] so it
must be deallocated with delete[], not free.

Change-Id: Ib5d6387aa63f1e0393113f82602f661bbada4aa2

Merge "Move ArtFields and ArtMethods to be a length prefixed array"

ART: x86_64 RoundDouble/Float intrinsics should initialize out value.

x86_64 RoundDouble intrinsic should initialize output register for the case of
"inPlusPointFive >= maxLong" as expected. The same for the RoundFloat intrinsic.
Fixed also the out register type in CreateSSE41FPToIntLocations provoking
a DCHECK failure.

Signed-off-by: Pavel Vyssotski <pavel.n.vyssotski@intel.com>
(cherry picked from commit 9ca257196b46fd7629bce0b338580e571e4113a8)

Bug: 22973442
Change-Id: If974e79d33311587d0b541a01ca8a4c9c11b9468

am 50fbbd29: am 275bc5e5: Merge "ART: Extend ISA features option for (host) run tests"

* commit '50fbbd29c444930446615651240e74595d8a5a9c':
  ART: Extend ISA features option for (host) run tests

Move ArtFields and ArtMethods to be a length prefixed array

Fixes race conditions between changing method and fields arrays
being seen in the wrong order by the GC.

Bug: 22832610
Change-Id: Ia21d6698f73ba207a6392c3d6b9be2658933073f

am 275bc5e5: Merge "ART: Extend ISA features option for (host) run tests"

* commit '275bc5e5f318ba6ea1bad6c89c2924d1248b7523':
  ART: Extend ISA features option for (host) run tests

Merge "ART: Extend ISA features option for (host) run tests"

ART: Extend ISA features option for (host) run tests

To allow to test advanced intrinsics only available on (fairly)
modern processors, and because our build requirements are still
too low to enable them in the build system, pipe through the
existing make variables to the run-tests to allow to override
the dex2oat autodetection scheme.

Usage (standalone run-test): use the new --instruction-set-features
command line parameter to set the value for dex2oat (in the prebuild
configuration) or as an -Xcompiler-option (for no-prebuild). Note:
this is general and also works for target tests.

Usage (through make): set DEX2OAT_HOST_INSTRUCTION_SET_FEATURES and
2ND_DEX2OAT_HOST_INSTRUCTION_SET_FEATURES to set instruction set
features, as was possible before for the compilation of core files.
This option is now forwarded to host run-tests.

Change-Id: I9d89c82b6a5c8853748e5d1c153dde5d4050757a

am 8a8cefa3: am d8406781: Merge "ART: Wire up DexToDexCompiler without extern"

* commit '8a8cefa3d0331d07c49d11ea936b33d03d124e73':
  ART: Wire up DexToDexCompiler without extern

am d8406781: Merge "ART: Wire up DexToDexCompiler without extern"

* commit 'd8406781fdb8917ae0c45c1e2194f9a5ab874e78':
  ART: Wire up DexToDexCompiler without extern

am 6541898f: am 02f0771b: Merge "Revert "cleanup: Replace pointers with out-parameters and fix-up formatting""

* commit '6541898f6738794ff59a44b121e4a300d36ec393':
  Revert "cleanup: Replace pointers with out-parameters and fix-up formatting"

am 5970565b: am 94800f46: Merge "Address some comments and clean up"

* commit '5970565b1fafc1b152680b4904ad5be0af4b355e':
  Address some comments and clean up

am df9964ee: am 46e0f460: Merge "Fix mod-union logic for native roots"

* commit 'df9964eecad0821a791c54210d3fab67a06e9a19':
  Fix mod-union logic for native roots

Merge "ART: Wire up DexToDexCompiler without extern"

ART: Wire up DexToDexCompiler without extern

The extern declaration actually had the wrong signature, a parameter
was missing, and we never noticed.

Remove the function field and extern. Add a header for the compiler.
Use the header file.

Change-Id: Ia4d4dd86211da6045709a45c7bf8430471d1b62b

am 02f0771b: Merge "Revert "cleanup: Replace pointers with out-parameters and fix-up formatting""

* commit '02f0771bba69edd34b0b112902bb9132ed099c75':
  Revert "cleanup: Replace pointers with out-parameters and fix-up formatting"

Merge "Revert "cleanup: Replace pointers with out-parameters and fix-up formatting""

am 94800f46: Merge "Address some comments and clean up"

* commit '94800f46da0aded5152545322ad842f89e8a21d8':
  Address some comments and clean up

Merge "Address some comments and clean up"

am 46e0f460: Merge "Fix mod-union logic for native roots"

* commit '46e0f460aa233a4a0ac161bfafde164357fe2b26':
  Fix mod-union logic for native roots

Merge "Fix mod-union logic for native roots"

Fix mod-union logic for native roots

Bug: 22949217
Change-Id: I9e4631870160d5a9412df1eec2a7b79c65668542

am 75fe3e02: am 13394093: Merge "Have constant folding be more flexible."

* commit '75fe3e02fdc0c81b81f8ffbdb982f214b4995691':
  Have constant folding be more flexible.

am 13394093: Merge "Have constant folding be more flexible."

* commit '1339409345cc8b46d64570430ca3197408fda159':
  Have constant folding be more flexible.

Merge "Have constant folding be more flexible."

am edd54eeb: am 902ec264: Merge "Add constant folding for long unary operations in opt. compiler."

* commit 'edd54eeb8a725f121b8332f9b4db0b2774a21f38':
  Add constant folding for long unary operations in opt. compiler.

am 902ec264: Merge "Add constant folding for long unary operations in opt. compiler."

* commit '902ec264e1040cbfd783c44809ed10b261bd78ee':
  Add constant folding for long unary operations in opt. compiler.

Merge "Add constant folding for long unary operations in opt. compiler."

am 0611b853: am d4fd5914: Merge "ART: Move exception clearing into own instruction"

* commit '0611b8538ed8106ef246bdc63d0fe0c5b71054d5':
  ART: Move exception clearing into own instruction

Have constant folding be more flexible.

- Have Evaluate methods take as argument(s) and return value
  instances of HConstant (instead of built-in 32- or 64-bit
  integer values), to let the evaluated instruction choose
  the type of the statically evaluated node; for instance,
  art::HEqual::Evaluate shall return a HIntConstant
  node (as implementation of a Boolean constant) whatever
  the type of its inputs (a pair of HIntConstant or a pair
  of HLongConstant).
- Split the evaluation job from the operation logic: the
  former is addressed by Evaluate methods, while the latter
  is done by a generic Compute method.
- Adress valid BinOp(int, long) and BinOp(long, int) cases.
- Add a constructor to art::HIntConstant to build an integer
  constant from a `bool` value.

Change-Id: If84b6fe8406bb94ddb1aa8b02e36628dff526db3

Add constant folding for long unary operations in opt. compiler.

Add tests to exercise the constant folding of these
instructions.

Also, prevent Java methods from run-tests exercising the
code generation of these instruction from being inlined,
so that they continue to check the generated code (and
not the code produced by the constant folding pass).

Change-Id: I28efca7cdb5142ac2b6d158ba296fb9136d62481

am d4fd5914: Merge "ART: Move exception clearing into own instruction"

* commit 'd4fd591406f356dcb8feaa262c19983cacb9116d':
  ART: Move exception clearing into own instruction

Merge "ART: Move exception clearing into own instruction"

am 2b83ce9e: am 8bfc949a: Merge "Revert "Revert "ART: Use bionic TLS slot for thread-self"""

* commit '2b83ce9eb8306ac1fd52148908faf0bc249c5095':
  Revert "Revert "ART: Use bionic TLS slot for thread-self""

am 8bfc949a: Merge "Revert "Revert "ART: Use bionic TLS slot for thread-self"""

* commit '8bfc949ae075d66d1d6fbf36d7ed774b6da1f62a':
  Revert "Revert "ART: Use bionic TLS slot for thread-self""

Merge "Revert "Revert "ART: Use bionic TLS slot for thread-self"""

am a3b4e939: am 10569198: Merge "ART: x86_64 RoundDouble/Float intrinsics should initialize out value."

* commit 'a3b4e939a910b8d3b627bcb71a0e0e0b6771ab01':
  ART: x86_64 RoundDouble/Float intrinsics should initialize out value.

am 10569198: Merge "ART: x86_64 RoundDouble/Float intrinsics should initialize out value."

* commit '1056919800f5db8d7a9607835fa34254eae51ec3':
  ART: x86_64 RoundDouble/Float intrinsics should initialize out value.

Revert "Revert "ART: Use bionic TLS slot for thread-self""

This reverts commit 50832b6804d15ea5ddfe99a2753a09111b5bfe37.

Fix clearing the TLS on detach.

Change-Id: I75ed5ccccb4f31800e5f893055e96067a9695258

Merge "ART: x86_64 RoundDouble/Float intrinsics should initialize out value."

Address some comments and clean up

Change-Id: I538cf204f1c89d5fc81f8fc5e5800fcf1cf87359

am cbc3382f: am 1cf85515: Merge "Revert "ART: Use bionic TLS slot for thread-self""

* commit 'cbc3382f26ef8165a26afe99be7a6e621013ecf4':
  Revert "ART: Use bionic TLS slot for thread-self"

am 1cf85515: Merge "Revert "ART: Use bionic TLS slot for thread-self""

* commit '1cf85515600d298a2c9e4de137c0926949205097':
  Revert "ART: Use bionic TLS slot for thread-self"

Merge "Revert "ART: Use bionic TLS slot for thread-self""

Revert "ART: Use bionic TLS slot for thread-self"

This reverts commit b81535875ff46d1630c76091fe969a9eb7fc4d16.

Breaking tests, need to investigate.

Change-Id: Iab8c4f6136e6cb9bd6cf3c3495aa33962ef890ee

am 9cc66f1e: am d2606b1b: Merge "Parallel Move Resolver: Perform Stack/Stack first"

* commit '9cc66f1ef5488ba62b01866b5e1112ecc16b1e82':
  Parallel Move Resolver: Perform Stack/Stack first

am d2606b1b: Merge "Parallel Move Resolver: Perform Stack/Stack first"

* commit 'd2606b1b922573d45707931bc387cbf990771238':
  Parallel Move Resolver: Perform Stack/Stack first

Merge "Parallel Move Resolver: Perform Stack/Stack first"

am c5866465: am bab71e2a: Merge "ART: Use bionic TLS slot for thread-self"

* commit 'c586646572150f63df0be92a83dab32b8204e387':
  ART: Use bionic TLS slot for thread-self

am bab71e2a: Merge "ART: Use bionic TLS slot for thread-self"

* commit 'bab71e2a51834dd6986f2166d332f540824687f9':
  ART: Use bionic TLS slot for thread-self