public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* PATCH: Add bootstrap-mpx.mk
@ 2015-03-23 19:27 H.J. Lu
  2015-04-13 10:43 ` Ilya Enkovich
  0 siblings, 1 reply; 4+ messages in thread
From: H.J. Lu @ 2015-03-23 19:27 UTC (permalink / raw)
  To: gcc-patches

Hi,

This patch adds bootstrap-mpx.mk so that we test mpx in gcc build by
configuring GCC with

--enable-libmpx --with-build-config="bootstrap-mpx"

OK to install?


H.J.
---
2015-03-23  H.J. Lu  <hongjiu.lu@intel.com>

	* bootstrap-mpx.mk: New file.

diff --git a/config/bootstrap-mpx.mk b/config/bootstrap-mpx.mk
new file mode 100644
index 0000000..ed97826
--- /dev/null
+++ b/config/bootstrap-mpx.mk
@@ -0,0 +1,9 @@
+# This option enables -fcheck-pointer-bounds -mmpx for stage2 and stage3.
+
+STAGE2_CFLAGS += -fcheck-pointer-bounds -mmpx
+STAGE3_CFLAGS += -fcheck-pointer-bounds -mmpx
+POSTSTAGE1_LDFLAGS += -fcheck-pointer-bounds -mmpx \
+		      -static-libmpx -static-libmpxwrappers \
+		      -B$$r/prev-$(TARGET_SUBDIR)/libmpx \
+		      -B$$r/prev-$(TARGET_SUBDIR)/libmpx/mpxrt/.libs \
+		      -B$$r/prev-$(TARGET_SUBDIR)/libmpx/mpxwrap/.libs

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: PATCH: Add bootstrap-mpx.mk
  2015-03-23 19:27 PATCH: Add bootstrap-mpx.mk H.J. Lu
@ 2015-04-13 10:43 ` Ilya Enkovich
  2015-04-13 20:26   ` H.J. Lu
  0 siblings, 1 reply; 4+ messages in thread
From: Ilya Enkovich @ 2015-04-13 10:43 UTC (permalink / raw)
  To: H.J. Lu; +Cc: gcc-patches

2015-03-23 22:27 GMT+03:00 H.J. Lu <hongjiu.lu@intel.com>:
> Hi,
>
> This patch adds bootstrap-mpx.mk so that we test mpx in gcc build by
> configuring GCC with
>
> --enable-libmpx --with-build-config="bootstrap-mpx"
>
> OK to install?
>
>
> H.J.
> ---
> 2015-03-23  H.J. Lu  <hongjiu.lu@intel.com>
>
>         * bootstrap-mpx.mk: New file.
>
> diff --git a/config/bootstrap-mpx.mk b/config/bootstrap-mpx.mk
> new file mode 100644
> index 0000000..ed97826
> --- /dev/null
> +++ b/config/bootstrap-mpx.mk
> @@ -0,0 +1,9 @@
> +# This option enables -fcheck-pointer-bounds -mmpx for stage2 and stage3.
> +
> +STAGE2_CFLAGS += -fcheck-pointer-bounds -mmpx
> +STAGE3_CFLAGS += -fcheck-pointer-bounds -mmpx
> +POSTSTAGE1_LDFLAGS += -fcheck-pointer-bounds -mmpx \
> +                     -static-libmpx -static-libmpxwrappers \
> +                     -B$$r/prev-$(TARGET_SUBDIR)/libmpx \
> +                     -B$$r/prev-$(TARGET_SUBDIR)/libmpx/mpxrt/.libs \
> +                     -B$$r/prev-$(TARGET_SUBDIR)/libmpx/mpxwrap/.libs

Hi,

Static constructors created by Pointer Bounds Checker get random seed
in their names. -frandom-seed should be used to avoid comparison
errors.

Thanks,
Ilya

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: PATCH: Add bootstrap-mpx.mk
  2015-04-13 10:43 ` Ilya Enkovich
@ 2015-04-13 20:26   ` H.J. Lu
  2015-04-14  4:50     ` Jeff Law
  0 siblings, 1 reply; 4+ messages in thread
From: H.J. Lu @ 2015-04-13 20:26 UTC (permalink / raw)
  To: Ilya Enkovich; +Cc: gcc-patches

[-- Attachment #1: Type: text/plain, Size: 1382 bytes --]

On Mon, Apr 13, 2015 at 3:43 AM, Ilya Enkovich <enkovich.gnu@gmail.com> wrote:
> 2015-03-23 22:27 GMT+03:00 H.J. Lu <hongjiu.lu@intel.com>:
>> Hi,
>>
>> This patch adds bootstrap-mpx.mk so that we test mpx in gcc build by
>> configuring GCC with
>>
>> --enable-libmpx --with-build-config="bootstrap-mpx"
>>
>> OK to install?
>>
>>
>> H.J.
>> ---
>> 2015-03-23  H.J. Lu  <hongjiu.lu@intel.com>
>>
>>         * bootstrap-mpx.mk: New file.
>>
>> diff --git a/config/bootstrap-mpx.mk b/config/bootstrap-mpx.mk
>> new file mode 100644
>> index 0000000..ed97826
>> --- /dev/null
>> +++ b/config/bootstrap-mpx.mk
>> @@ -0,0 +1,9 @@
>> +# This option enables -fcheck-pointer-bounds -mmpx for stage2 and stage3.
>> +
>> +STAGE2_CFLAGS += -fcheck-pointer-bounds -mmpx
>> +STAGE3_CFLAGS += -fcheck-pointer-bounds -mmpx
>> +POSTSTAGE1_LDFLAGS += -fcheck-pointer-bounds -mmpx \
>> +                     -static-libmpx -static-libmpxwrappers \
>> +                     -B$$r/prev-$(TARGET_SUBDIR)/libmpx \
>> +                     -B$$r/prev-$(TARGET_SUBDIR)/libmpx/mpxrt/.libs \
>> +                     -B$$r/prev-$(TARGET_SUBDIR)/libmpx/mpxwrap/.libs
>
> Hi,
>
> Static constructors created by Pointer Bounds Checker get random seed
> in their names. -frandom-seed should be used to avoid comparison
> errors.
>

Updated.  I bootstrapped it on Linux/x86-64.  OK for trunk?

Thanks.


-- 
H.J.

[-- Attachment #2: 0001-Add-bootstrap-mpx.mk.patch --]
[-- Type: text/x-patch, Size: 1000 bytes --]

From c70ad827455a431002d500ec933e71aa910f6f21 Mon Sep 17 00:00:00 2001
From: "H.J. Lu" <hjl.tools@gmail.com>
Date: Thu, 26 Mar 2015 10:06:21 -0700
Subject: [PATCH] Add bootstrap-mpx.mk

	* bootstrap-mpx.mk: New file.
---
 config/bootstrap-mpx.mk | 9 +++++++++
 1 file changed, 9 insertions(+)
 create mode 100644 config/bootstrap-mpx.mk

diff --git a/config/bootstrap-mpx.mk b/config/bootstrap-mpx.mk
new file mode 100644
index 0000000..acc2a2e
--- /dev/null
+++ b/config/bootstrap-mpx.mk
@@ -0,0 +1,9 @@
+# This option enables -fcheck-pointer-bounds -mmpx for stage2 and stage3.
+
+STAGE2_CFLAGS += -fcheck-pointer-bounds -mmpx -frandom-seed=1
+STAGE3_CFLAGS += -fcheck-pointer-bounds -mmpx -frandom-seed=1
+POSTSTAGE1_LDFLAGS += -fcheck-pointer-bounds -mmpx -frandom-seed=1 \
+		      -static-libmpx -static-libmpxwrappers \
+		      -B$$r/prev-$(TARGET_SUBDIR)/libmpx \
+		      -B$$r/prev-$(TARGET_SUBDIR)/libmpx/mpxrt/.libs \
+		      -B$$r/prev-$(TARGET_SUBDIR)/libmpx/mpxwrap/.libs
-- 
1.9.3


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: PATCH: Add bootstrap-mpx.mk
  2015-04-13 20:26   ` H.J. Lu
@ 2015-04-14  4:50     ` Jeff Law
  0 siblings, 0 replies; 4+ messages in thread
From: Jeff Law @ 2015-04-14  4:50 UTC (permalink / raw)
  To: H.J. Lu, Ilya Enkovich; +Cc: gcc-patches

On 04/13/2015 02:26 PM, H.J. Lu wrote:
> On Mon, Apr 13, 2015 at 3:43 AM, Ilya Enkovich <enkovich.gnu@gmail.com> wrote:
>> 2015-03-23 22:27 GMT+03:00 H.J. Lu <hongjiu.lu@intel.com>:
>>> Hi,
>>>
>>> This patch adds bootstrap-mpx.mk so that we test mpx in gcc build by
>>> configuring GCC with
>>>
>>> --enable-libmpx --with-build-config="bootstrap-mpx"
>>>
>>> OK to install?
>>>
>>>
>>> H.J.
>>> ---
>>> 2015-03-23  H.J. Lu  <hongjiu.lu@intel.com>
>>>
>>>          * bootstrap-mpx.mk: New file.
OK.
jeff

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-04-14  4:50 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-23 19:27 PATCH: Add bootstrap-mpx.mk H.J. Lu
2015-04-13 10:43 ` Ilya Enkovich
2015-04-13 20:26   ` H.J. Lu
2015-04-14  4:50     ` Jeff Law

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).