RFR: 8205091: AIX: build errors in hotspot after 8203641: Refactor String Deduplication into shared
René Schünemann
rene.schuenemann at gmail.com
Fri Jun 15 09:08:06 UTC 2018
Hi Matthias,
the name SI seems also quite "common" and may result in other naming
clashes in the future.
Maybe something more readable like STAT_IMPL?
Please also change the name in the comment:
// STAT: String Dedup Stat implementation
Regards,
Rene
On Fri, Jun 15, 2018 at 9:47 AM, Baesken, Matthias
<matthias.baesken at sap.com> wrote:
> Please review this small change that fixes the AIX build after "8203641: Refactor String Deduplication into shared" .
>
> We are getting this compilation error :
> /build_ci_jdk_jdk_rs6000_64/src/hotspot/share/gc/shared/stringdedup/stringDedup.hpp", line 107.38: 1540-0063 (S) The text "1" is unexpected.
>
>
> Looks like the name of the second template parameter (STAT)
>
> template <typename QUEUE, typename STAT>
> static void initialize_impl();
>
> is clashing with defines from the AIX system headers (where I find #define STAT 1 ) .
> Renaming STAT to something else fixes the build on AIX .
>
> Webrev :
>
> http://cr.openjdk.java.net/~mbaesken/webrevs/8205091/
>
> Bug :
>
> https://bugs.openjdk.java.net/browse/JDK-8205091
>
>
> Thanks, Matthias
More information about the build-dev
mailing list