RFR: 8304089: Convert TraceDependencies to UL [v3]

[Coleen Phillimore]

On Fri, 17 Mar 2023 04:53:38 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> See the function log_dependency()
>> 
>> https://github.com/openjdk/jdk/blob/421b4ee33c652cc7c444fbbf298bbc23d052c2fe/src/hotspot/share/code/dependencies.cpp#L845
>> 
>> called from here (as one place).
>> 
>> https://github.com/openjdk/jdk/blob/421b4ee33c652cc7c444fbbf298bbc23d052c2fe/src/hotspot/share/code/dependencies.cpp#L2071
>> 
>> When TraceDependencies was true, the log file would be non-null and log_dependency would write to it.  Keeping this with -Xlog:dependencies=debug retains what TraceDependencies did.
>
> Hmmm this seems broken to me then. We output the general dependency logging to one place based on UL configuration, but then we output `log_dependency` to the log file. If these are meant to be related and always reported together then we have lost that. If they are actually unrelated then this may still need the TraceDependencies flag to control it.
> 
> @iwanowww can you comment on this please?

Maybe we don't need this -Xlog test in this place, because this is also what PrintDependencies does.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13007#discussion_r1143368172