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