diff options
author | 2018-06-26 11:02:04 +0100 | |
---|---|---|
committer | 2018-06-29 14:39:00 +0100 | |
commit | 6fd1606a3f3fc2dd53ab4f8b371e420b3e33c74f (patch) | |
tree | 9f944d267ce3616eb969c027665e4a451c2b3879 /compiler/optimizing/intrinsics.h | |
parent | bb089b6bf850c87e0e42917a383cc7298dcb09c5 (diff) |
Implement Integer.valueOf() intrinsic for boot image.
And generate only one "boot image live objects" array rather
than one per boot*.art file.
Test: m test-art-host-gtest
Test: testrunner.py --host --optimizing
Test: Pixel 2 XL boots.
Test: testrunner.py --target --optimizing
Bug: 71526895
Change-Id: I23af7f47fea5150805f801cd2512f2d152ee5b73
Diffstat (limited to 'compiler/optimizing/intrinsics.h')
-rw-r--r-- | compiler/optimizing/intrinsics.h | 25 |
1 files changed, 17 insertions, 8 deletions
diff --git a/compiler/optimizing/intrinsics.h b/compiler/optimizing/intrinsics.h index f2b78239d6..993648f765 100644 --- a/compiler/optimizing/intrinsics.h +++ b/compiler/optimizing/intrinsics.h @@ -129,10 +129,10 @@ class IntrinsicVisitor : public ValueObject { // Temporary data structure for holding Integer.valueOf data for generating code. // We only use it if the boot image contains the IntegerCache objects. struct IntegerValueOfInfo { + static constexpr uint32_t kInvalidReference = static_cast<uint32_t>(-1); + IntegerValueOfInfo(); - // Boot image offset of java.lang.Integer for allocating an instance. - uint32_t integer_boot_image_offset; // Offset of the Integer.value field for initializing a newly allocated instance. uint32_t value_offset; // The low value in the cache. @@ -140,18 +140,27 @@ class IntrinsicVisitor : public ValueObject { // The length of the cache array. uint32_t length; + // Boot image offset of java.lang.Integer for allocating an instance. + uint32_t integer_boot_image_offset; // Set to kInvalidReference when compiling the boot image. + + // This union contains references to the boot image. For app AOT or JIT compilation, + // these are the boot image offsets of the target. For boot image compilation, the + // location shall be known only at link time, so we encode a symbolic reference using + // IntrinsicObjects::EncodePatch(). union { - // Boot image offset of the target Integer object for constant input in the cache range. - // If the input is out of range, this is set to 0u and the code must allocate a new Integer. - uint32_t value_boot_image_offset; + // The target value for a constant input in the cache range. If the constant input + // is out of range (use `low` and `length` to check), this value is bogus (set to + // kInvalidReference) and the code must allocate a new Integer. + uint32_t value_boot_image_reference; - // Boot image offset of the cache array data used for non-constant input in the cache range. + // The cache array data used for a non-constant input in the cache range. // If the input is out of range, the code must allocate a new Integer. - uint32_t array_data_boot_image_offset; + uint32_t array_data_boot_image_reference; }; }; - static IntegerValueOfInfo ComputeIntegerValueOfInfo(HInvoke* invoke); + static IntegerValueOfInfo ComputeIntegerValueOfInfo( + HInvoke* invoke, const CompilerOptions& compiler_options); protected: IntrinsicVisitor() {} |