RFR: 8265101: Remove unnecessary functions in os*.inline.hpp [v2]

Ioi Lam iklam at openjdk.java.net
Thu Apr 22 05:48:30 UTC 2021


On Wed, 14 Apr 2021 02:46:57 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
>> 
>>  - Merge branch 'master' into 8265101-remove-unnecessary-os-inline-functions
>>  - Merge branch 'master' into 8265101-remove-unnecessary-os-inline-functions
>>  - fixed copyright
>>  - removed unnecessary include os.hpp
>>  - remove unnecessary include os.inline.hpp
>>  - step1
>
> Hi Ioi,
> 
> This all seems fine to me. I agree these seem unlikely to need to be inlined for performance.
> 
> Thanks,
> David

Thanks @dholmes-ora and @kimbarrett for the review.
Tested again in mach5 build tiers 1-5.

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

PR: https://git.openjdk.java.net/jdk/pull/3475


More information about the hotspot-dev mailing list