RFR: 8286117: Remove unnecessary indirection and unused code in UL [v2]

David Holmes dholmes at openjdk.java.net
Tue May 10 02:36:42 UTC 2022


On Mon, 9 May 2022 11:15:40 GMT, Johan Sjölén <duke at openjdk.java.net> wrote:

>> This PR cleans up some minor annoyances in the UL code.
>> 
>> Note that the change from `LogImpl` to `LogTagSet` is equivalent, we're essentially inlining the calls that `LogImpl` would have made. This simplifies understanding ("why do we need `LogImpl` here?" was my question, answer: We don't!)'
>> 
>> *Edit:* Passes tier1.
>
> Johan Sjölén has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Use _tagset

This all seems fine to me now.

Thanks for explaining everything.

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

Marked as reviewed by dholmes (Reviewer).

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


More information about the hotspot-runtime-dev mailing list