RFR(M): 8233787: Break cycle in vm_version* includes

[Doerr, Martin]

Hi Lutz,

I've looked over the complete webrev .03 and it looks good to me.
I appreciate having the abstract version in separate files and the regular vm_version basically include the platform stuff.

Best regards,
Martin


> -----Original Message-----
> From: hotspot-dev <hotspot-dev-bounces at openjdk.java.net> On Behalf Of
> Kim Barrett
> Sent: Samstag, 16. November 2019 00:00
> To: Schmidt, Lutz <lutz.schmidt at sap.com>
> Cc: hotspot-dev at openjdk.java.net
> Subject: Re: RFR(M): 8233787: Break cycle in vm_version* includes
> 
> > On Nov 15, 2019, at 7:19 AM, Schmidt, Lutz <lutz.schmidt at sap.com> wrote:
> >
> > Hi Kim,
> >
> > thanks for reviewing - I understand your comments that way. One more
> review to go. :-)
> >
> > I made abstract_vm_version.cpp #include vm_version.hpp, and I updated
> the copyrights. See the webrev#03:
> >  http://cr.openjdk.java.net/~lucy/webrevs/8233787.03/
> >
> > I ran the initial webrev iteration through dev-submit and had it active SAP-
> internally. The current webrev is active since last night SAP-internally. All
> builds are green. The test show only unrelated issues (some JIT compiler
> asserts). Of course I will run the final webrev through dev-submit.
> >
> > Re test coverage: we do not cover 32-bit platforms. And we do not have
> zero or minimal builds.
> 
> Looks good.