On 4/06/2015 10:38 PM, Magnus Ihse Bursie wrote:
On 2015-06-04 07:41, David Holmes wrote:
Magnus,
You missed the hotspot side of this:
./solaris/makefiles/gcc.make: DEBUG_CFLAGS += -fstack-protector-all
--param ssp-buffer-size=1
./bsd/makefiles/gcc.make:DEBUG_CFLAGS += -fstack-protector-a
On 2015-06-04 07:41, David Holmes wrote:
Magnus,
You missed the hotspot side of this:
./solaris/makefiles/gcc.make: DEBUG_CFLAGS += -fstack-protector-all
--param ssp-buffer-size=1
./bsd/makefiles/gcc.make:DEBUG_CFLAGS += -fstack-protector-all
--param ssp-buffer-size=1
./linux/makefiles/g
Magnus,
You missed the hotspot side of this:
./solaris/makefiles/gcc.make: DEBUG_CFLAGS += -fstack-protector-all
--param ssp-buffer-size=1
./bsd/makefiles/gcc.make:DEBUG_CFLAGS += -fstack-protector-all
--param ssp-buffer-size=1
./linux/makefiles/gcc.make:DEBUG_CFLAGS += -fstack-protec
Magnus:
Looks good to me as well.
Tim
On 06/03/15 06:12, Erik Joelsson wrote:
Looks good to me.
/Erik
On 2015-06-03 14:33, Magnus Ihse Bursie wrote:
Not all versions of gcc support -fstack-protector. We should check
that it is accepted as an argument before adding it to the flags.
Bug: ht
Looks good to me.
/Erik
On 2015-06-03 14:33, Magnus Ihse Bursie wrote:
Not all versions of gcc support -fstack-protector. We should check
that it is accepted as an argument before adding it to the flags.
Bug: https://bugs.openjdk.java.net/browse/JDK-8081692
WebRev inline:
diff --git a/common
