RFR: 8343685: C2 SuperWord: refactor VPointer with MemPointer [v6]
Emanuel Peter
epeter at openjdk.org
Thu Jan 16 06:31:04 UTC 2025
On Wed, 15 Jan 2025 12:43:15 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>>
>> More fixes for vnkozlov
>
> src/hotspot/share/opto/vectorization.hpp line 949:
>
>> 947:
>> 948: // Vector element size statistics for loop vectorization with vector masks
>> 949: class VectorElementSizeStats {
>
> Just noticed this when scrolling through the class in my IDE: All methods of this class could be made `const`. You could probably squeeze this in here as well as part of this bigger refactoring - probably not worth a separate RFE.
Turns out it is dead code!
emanuel at emanuel-oracle:/oracle-work/jdk-fork5/open$ grep VectorElementSizeStats src/hotspot/share/ -r
src/hotspot/share/opto/vectorization.hpp:class VectorElementSizeStats {
src/hotspot/share/opto/vectorization.hpp: VectorElementSizeStats(Arena* a) : _stats(NEW_ARENA_ARRAY(a, int, 4)) {
Binary file src/hotspot/share/opto/.vectorization.hpp.swp matches
emanuel at emanuel-oracle:/oracle-work/jdk-fork5/open$ git log -S VectorElementSizeStats
commit 96781ba33d4717c8df2b9ba4cd5d66858cf5c2d1
Author: Pengfei Li <pli at openjdk.org>
Date: Fri Sep 15 01:02:44 2023 +0000
8312332: C2: Refactor SWPointer out from SuperWord
Reviewed-by: epeter, kvn
commit a38582e941c0234e76d1dbea60c731c83d2c9977
Author: Pengfei Li <pli at openjdk.org>
Date: Thu Jul 13 01:45:52 2023 +0000
8311691: C2: Remove legacy code related to PostLoopMultiversioning
Reviewed-by: kvn, sviswanathan
commit 741be46138c4a02f1d9661b3acffb533f50ba9cf
Author: Pengfei Li <pli at openjdk.org>
Date: Tue Apr 5 23:50:13 2022 +0000
8183390: Fix and re-enable post loop vectorization
Reviewed-by: roland, thartmann, kvn
The last occurance was removed with https://github.com/openjdk/jdk/pull/14824
I'll just rip it out now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21926#discussion_r1917811724
More information about the hotspot-compiler-dev
mailing list