RFR: 8220784: hsdis cannot be built with MinGW64
yasuenag at gmail.com
Wed Mar 20 01:07:36 UTC 2019
Hi Erik, David,
I checked this change on Linux x64 and MinGW for Windows.
According to hsdis README, we need to use MinGW cross compiler to
build hsdis . So I think Cygwin is not required.
I do not have macOS and AIX. So I cannot check this change on them.
BTW is hsdis included Java SE spec?
hsdis seems not to be included jtreg tests, and it is not contained in
I think it is allowed even if we lack some test for hsdis if hsdis is
not required for Java SE.
If not so, I want sponsor(s) for this change.
2019年3月20日(水) 6:54 David Holmes <david.holmes at oracle.com>:
> CC'ing hotspot-dev. I agree this needs to be checked on every platform
> affected. I can't comment on the fix itself.
> On 20/03/2019 2:36 am, Erik Joelsson wrote:
> > I think this needs to be reviewed by at least someone in hotspot who
> > regularly builds hsdis. I can't really comment on the validity of the
> > patch as I'm unfamiliar with both hsdis as well as this makefile. Have
> > you at least verified the build on all the platforms which you affect
> > with this change (which would be at least Macos, AIX and Windows in a
> > normal Cygwin VS env)?
> > /Erik
> > On 2019-03-18 17:56, Yasumasa Suenaga wrote:
> >> Hi all,
> >> Please review this change:
> >> JBS: https://bugs.openjdk.java.net/browse/JDK-8220784
> >> webrev: http://cr.openjdk.java.net/~ysuenaga/JDK-8220784/webrev.00/
> >> I attempt to build hsdis for Windows on WSL Ubuntu 18.04 with
> >> gcc-mingw-w64-x86-64, but I saw error messages that some functions
> >> which are provided by zlib are unresolved.
> >> We need to link to zlib.
> >> Thanks,
> >> Yasumasa
More information about the build-dev