On Thu, 29 Aug 2024 21:46:52 GMT, Chen Liang <li...@openjdk.org> wrote:

>> `CodeBuilder::loadConstant(Opcode, ConstantDesc)` is error-prone and 
>> confusing. Users should almost always use `loadConstant(ConstantDesc)` for 
>> optimized instructions, or use specific factories `iconst_0` etc. or 
>> `bipush` with arguments or `LoadConstantInstruction.of` if they want to 
>> specify the exact type of LDC opcode.
>
> Chen Liang has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Move bipush and sipush fix from Opcode cleanup to this patch

Marked as reviewed by abdelhak-za...@github.com (no known OpenJDK username).

-------------

PR Review: https://git.openjdk.org/jdk/pull/20779#pullrequestreview-2274458870

Reply via email to