Hi Andrew, thanks for the quick reply.
I wanted to use the syntax I know from hotspot/make/linux/makefiles/gcc.make I think it's easier to understand if it's everywhere noted down similarly. Best regards, Goetz > -----Original Message----- > From: Andrew Haley [mailto:a...@redhat.com] > Sent: Mittwoch, 13. Juli 2016 11:00 > To: Lindenmaier, Goetz <goetz.lindenma...@sap.com>; jdk7u- > d...@openjdk.java.net > Subject: Re: RFR(XS): 8161262: Fix jdk build with gcc 4.1.2: -fno-strict- > overflow not known. > > On 13/07/16 09:53, Lindenmaier, Goetz wrote: > > we are building the jdk7u ppc port on gcc 4.1.2. This fails since > > "8147466: Add -fno-strict-overflow to > IndicRearrangementProcessor{,2}.cpp" > > as the gcc flag introduced there is not known to gcc 4.1.2. > > I just guard it by the compiler version. > > > > Please review this change. > > http://cr.openjdk.java.net/~goetz/wr16/8161262-jdkGcc412/webrev.01/ > > This is OK, but do we really need to do it this way? If you're going > to launch a shell and test the result, you might as well launch GCC > with -fno-strict-overflow and test the result of that instead. > > Andrew.
