RFR(S): 8019926 : PPC64 (part 106): Make hsdis build and work on Linux/PPC64
vladimir.kozlov at oracle.com
Thu Jul 25 11:04:47 PDT 2013
Very good. Yes, you can push it into stage.
On 7/25/13 9:32 AM, Volker Simonis wrote:
> On Wed, Jul 24, 2013 at 8:47 PM, Vladimir Kozlov
> <vladimir.kozlov at oracle.com> wrote:
>> Looks good.
>> Please, fix indention in hsdis.c.
>> Could you add workaround info to the src/share/tools/hsdis/README?
> See: http://cr.openjdk.java.net/~simonis/webrevs/8019926.v3/
> I've also added a link to the Wiki page and updated the copyright
> years appropriately.
> Can I push this right into 'stage' (I don't think it will have any
> impact on JPRT:)
>> On 7/24/13 10:39 AM, Volker Simonis wrote:
>>> could you please review the following small patch which enables
>>> building the hsdis library on Linux/PPC64 and AIX/PPC64:
>>> I've tested that this change will not break the Linux/x86/x86_64 and
>>> Solaris/sparc/sparcv9 platforms.
>>> Notice that building the hs-dis library against a recent source
>>> version of binutils may stop because of a dependency bug in the
>>> binutils configure/make system with an error like:
>>> WARNING: `makeinfo' is missing on your system. You should only need it if
>>> you modified a `.texi' or `.texinfo' file, or any other file
>>> indirectly affecting the aspect of the manual. The spurious
>>> call might also be the consequence of using a buggy `make' (AIX,
>>> DU, IRIX). You might want to install the `Texinfo' package or
>>> the `GNU make' package. Grab either from any GNU archive site.
>>> make: *** [bfd.info] Error 1
>>> The easiest way to work around this problem is by doing a "touch
>>> $BINUTILS/bfd/doc/bfd.info". I've documented this in the HotSpot Wiki
>>> Thank you and best regards,
More information about the hotspot-dev