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