RFR: 8350952: Remove some non present files from OPT_SPEED_SRC list

Matthias Baesken mbaesken at openjdk.org
Wed Mar 5 08:43:56 UTC 2025


On Tue, 4 Mar 2025 16:11:22 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

> JvmFeatures.gmk contains a file list OPT_SPEED_SRC , the list contains files that have to be optimized for speed when the other files are optimized for binary size.
> However some non present files are mentioned and should be removed e.g.
> genCollectedHeap.cpp
> hashtable.cpp
> markSweep.cpp
> Maybe they were present in old releases.

Hi David, thanks for the review !
I wonder how useful this OPT_SPEED_SRC list is these days ; from what I heard it was setup a long time ago and also some tests/benchmarks were done back then. But in the meantime probably a lot of new files were added to the hs codebase that are also performance sensitive.
On the other hand , the compilers were switched a few times between introduction of the OPT_SPEED_SRC  list and now, so  who knows how valid the list is even for the files in it currently.

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

PR Comment: https://git.openjdk.org/jdk/pull/23902#issuecomment-2700234710


More information about the build-dev mailing list