kedynamic` instructions. This mimics the
> JVM's handling of `invokedynamic` instructions, creating dynamic call sites
> that are linked at runtime.
>
> It currently uses ad-hoc code to process class files and intends to migrate
> to ASM; but since we have the Classfile API, we can
On Tue, 25 Jun 2024 13:49:31 GMT, Adam Sotona wrote:
>> Transforming the classModel back to the bytes array in order to write the
>> transformed classfile
>
> And what is the purpose of parsing a model from bytes and transforming it
> back to bytes without a change?
This transformation is call
On Tue, 25 Jun 2024 13:17:13 GMT, Adam Sotona wrote:
>> Oussama Louati has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> remove: removed unnecessary logging
>
> test/jdk/java/lang/invoke/indify/Indify.jav
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
remove: removed unnecessary logging
-
Changes:
- all: https://git.openjdk.org/jd
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
fix: review addressing
-
Changes:
- all: https://git.openjdk.org/jdk/pull/18841/fi
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
fix: review addressing
-
Changes:
- all: https://git.openjdk.org/jdk/pull/18841/fi
On Mon, 17 Jun 2024 08:33:17 GMT, Adam Sotona wrote:
>> We need a boolean value to determine if we should proceed after replacing
>> the appropriate "pop" instruction with an "invokedynamic" instruction.
>> However, instead of using just a boolean field, we use a stack. The reason
>> for this
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with six additional
commits since the last revision:
- Update test/jdk/java/lang/invoke/indify/Indify.java
Co-authored-by: ExE B
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with eight additional
commits since the last revision:
- fix: work done
- fix: work in progress
- fix: one pass transformation per method
- fix: one p
On Mon, 17 Jun 2024 08:12:08 GMT, Adam Sotona wrote:
>> Oussama Louati has updated the pull request incrementally with 19 additional
>> commits since the last revision:
>>
>> - fix: fixed whitespaces
>> - fix: fixed whitespaces
>> - chore: used Class::
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
fix: fix comments
-
Changes:
- all: https://git.openjdk.org/jdk/pull/18841/fi
On Wed, 22 May 2024 22:38:16 GMT, Oussama Louati wrote:
>> test/jdk/java/lang/invoke/CallStaticInitOrder.java line 54:
>>
>>> 52:
>>> 53: static int Init1Tick;
>>> 54: public static class Init1 {
>>
>> Is there a reason to change
On Fri, 24 May 2024 17:57:28 GMT, Oussama Louati wrote:
>> test/jdk/java/lang/invoke/indify/Indify.java line 660:
>>
>>> 658: *
>>> 659: * @return true if any marks were changed, false otherwise.
>>> 660: */
>>
>> Th
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
chore: changed pool marking initialization to be done in one pass
-
Chang
On Wed, 22 May 2024 22:37:35 GMT, Oussama Louati wrote:
>> test/jdk/java/lang/invoke/CallSiteTest.java line 96:
>>
>>> 94: }
>>> 95:
>>> 96: public static class MyCCS extends ConstantCallSite {
>>
>> Any reason for this change?
>
On Wed, 22 May 2024 08:57:23 GMT, Adam Sotona wrote:
>> Oussama Louati has updated the pull request incrementally with 19 additional
>> commits since the last revision:
>>
>> - fix: fixed whitespaces
>> - fix: fixed whitespaces
>> - chore: used Class::
On Tue, 21 May 2024 11:37:26 GMT, Adam Sotona wrote:
> Indify invocation on Microbenchmarks as a part of the JDK build (after
> application of the above patch) still fails with:
>
> ```
> Failure on
> /Applications/XcodeJIB.app/dev/github/jdk/build/macosx-aarch64/support/test/micro/classes
> E
On Fri, 24 May 2024 17:57:28 GMT, Oussama Louati wrote:
>> An indify tool in j.l.i tests (also in vmTestBase) convert some source-code
>> private static methods with MT_ MH_, and INDY_ prefixes into MethodHandle,
>> MethodType, and CallSite constants.
>> It curre
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with 19 additional
commits since the last revision:
- fix: fixed whitespaces
- fix: fixed whitespaces
- chore: used Class::descriptorString
- remove:
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
update: Added javadoc documentation
-
Changes:
- all: https://git.openjdk.org/jd
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
update: updates indify
-
Changes:
- all: https://git.openjdk.org/jdk/pull/18841/fi
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
update: updates indify
-
Changes:
- all: https://git.openjdk.org/jdk/pull/18841/fi
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
Update test/jdk/java/lang/invoke/indify/Indify.java
Co-authored-by: liach <7806504+li...@use
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
Update test/jdk/java/lang/invoke/indify/Indify.java
Co-authored-by: Glavo
-
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
remove: remove unnecessary ClassModel fields
-
Changes:
- all: https://git.openjdk.
M; but since we have the Classfile API, we can migrate to Classfile API
> instead.
Oussama Louati has updated the pull request incrementally with one additional
commit since the last revision:
fix: enhance code
-
Changes:
- all: https://git.openjdk.org/jdk/pull/18841/fi
On Thu, 18 Apr 2024 13:53:34 GMT, Oussama Louati wrote:
> An indify tool in j.l.i tests (also in vmTestBase) convert some source-code
> private static methods with MT_ MH_, and INDY_ prefixes into MethodHandle,
> MethodType, and CallSite constants.
> It currently uses ad-hoc cod
On Thu, 18 Apr 2024 14:15:12 GMT, Chen Liang wrote:
> This is going to be a difficult task because of the sheer size of indify
> tool; to migrate you really need in-depth understanding of both indify and CF
> API. Good luck!
Thank you
> test/jdk/java/lang/invoke/indify/Indify.java line 1247:
An indify tool in j.l.i tests (also in vmTestBase) convert some source-code
private static methods with MT_ MH_, and INDY_ prefixes into MethodHandle,
MethodType, and CallSite constants.
It currently uses ad-hoc code to process class files and intends to migrate to
ASM; but since we have the Cla
29 matches
Mail list logo
