RFR: 8333840: C2 SuperWord: wrong result for MulAddS2I when inputs permuted [v5]

Emanuel Peter epeter at openjdk.org
Thu Jun 13 14:39:17 UTC 2024


On Thu, 13 Jun 2024 14:23:17 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 13 commits:
>> 
>>  - manual merge with master
>>  - review changes for Vladimir
>>  - more refactoring
>>  - improve naming
>>  - move code
>>  - rm isa_pack_input_or_null, leave that to a refactoring
>>  - refactoring / simplification
>>  - refactor into is_muladds2i_pack_with_pack_inputs
>>  - fix done, need refactoring
>>  - more partial fix and more test
>>  - ... and 3 more: https://git.openjdk.org/jdk/compare/5d2a19de...2155d18b
>
> Small comments, otherwise, looks good!

@chhagedorn thanks for the review! I applied all your suggestions.

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

PR Comment: https://git.openjdk.org/jdk/pull/19619#issuecomment-2165863266


More information about the hotspot-compiler-dev mailing list