RFR(M): 8233787: Break cycle in vm_version* includes
Kim Barrett
kim.barrett at oracle.com
Mon Nov 11 20:52:39 UTC 2019
> On Nov 11, 2019, at 11:56 AM, Schmidt, Lutz <lutz.schmidt at sap.com> wrote:
>
> Oh, oh,
> looks like I stepped into a beehive... Found JDK-8202579 and JDK-8145956 talking about the unwanted use of Abstract_VM_Version.
>
> My intended change would not tackle that "mess", as you call it. But it would make potential future cleanups a little bit easier by ensuring all of hotspot code only includes vm_version.hpp. I'm in the process of modifying my initial change to reflect Kim's suggestions. I'll send it out Tuesday (hopefully), Wednesday the latest.
JDK-7041262 was fixed a while ago, though I found that a new reference to Abstract_VM_Version
was recently added: https://bugs.openjdk.java.net/browse/JDK-8233943
I agree that JDK-8202579 looks like it might be easier to deal with after a change to break include cycles.
(BTW, I hadn’t noticed the include cycles before; good find!)
More information about the hotspot-dev
mailing list