* [Bug bootstrap/63995] Bootstrap error with -mmpx -fcheck-pointer-bounds
2014-11-20 12:33 [Bug bootstrap/63995] New: Bootstrap error with -mmpx -fcheck-pointer-bounds rguenth at gcc dot gnu.org
@ 2014-11-20 16:10 ` enkovich.gnu at gmail dot com
2014-11-20 16:16 ` enkovich.gnu at gmail dot com
` (8 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: enkovich.gnu at gmail dot com @ 2014-11-20 16:10 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63995
--- Comment #1 from Ilya Enkovich <enkovich.gnu at gmail dot com> ---
Created attachment 34052
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34052&action=edit
reproducer
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug bootstrap/63995] Bootstrap error with -mmpx -fcheck-pointer-bounds
2014-11-20 12:33 [Bug bootstrap/63995] New: Bootstrap error with -mmpx -fcheck-pointer-bounds rguenth at gcc dot gnu.org
2014-11-20 16:10 ` [Bug bootstrap/63995] " enkovich.gnu at gmail dot com
@ 2014-11-20 16:16 ` enkovich.gnu at gmail dot com
2014-11-25 13:04 ` rguenther at suse dot de
` (7 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: enkovich.gnu at gmail dot com @ 2014-11-20 16:16 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63995
--- Comment #2 from Ilya Enkovich <enkovich.gnu at gmail dot com> ---
I had a successful bootstrap with instrumentation some time ago but it's not
performed regularly.
We are extending regression testing for instrumentation now and coverage should
become better.
This particular problem may be caused by multiple varpool_node for the same
var. Will check it.
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug bootstrap/63995] Bootstrap error with -mmpx -fcheck-pointer-bounds
2014-11-20 12:33 [Bug bootstrap/63995] New: Bootstrap error with -mmpx -fcheck-pointer-bounds rguenth at gcc dot gnu.org
2014-11-20 16:10 ` [Bug bootstrap/63995] " enkovich.gnu at gmail dot com
2014-11-20 16:16 ` enkovich.gnu at gmail dot com
@ 2014-11-25 13:04 ` rguenther at suse dot de
2014-11-25 15:06 ` enkovich.gnu at gmail dot com
` (6 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: rguenther at suse dot de @ 2014-11-25 13:04 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63995
--- Comment #4 from rguenther at suse dot de <rguenther at suse dot de> ---
On Tue, 25 Nov 2014, enkovich.gnu at gmail dot com wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63995
>
> --- Comment #3 from Ilya Enkovich <enkovich.gnu at gmail dot com> ---
> Patch removing duplicating bounds symbols is in review. With this patch
> applied bootstrap goes till the end but there are lots of stage2 and stage3
> comparison error. I looked into one of them and the difference is caused by
> '-gtoggle' option used for stage2 build and not used for stage3 build.
This means you are generating different code with -g vs. -g0 which is
forbidden.
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug bootstrap/63995] Bootstrap error with -mmpx -fcheck-pointer-bounds
2014-11-20 12:33 [Bug bootstrap/63995] New: Bootstrap error with -mmpx -fcheck-pointer-bounds rguenth at gcc dot gnu.org
` (2 preceding siblings ...)
2014-11-25 13:04 ` rguenther at suse dot de
@ 2014-11-25 15:06 ` enkovich.gnu at gmail dot com
2014-11-25 15:45 ` enkovich.gnu at gmail dot com
` (5 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: enkovich.gnu at gmail dot com @ 2014-11-25 15:06 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63995
--- Comment #5 from Ilya Enkovich <enkovich.gnu at gmail dot com> ---
Created attachment 34112
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34112&action=edit
-g0 problem reproducer
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug bootstrap/63995] Bootstrap error with -mmpx -fcheck-pointer-bounds
2014-11-20 12:33 [Bug bootstrap/63995] New: Bootstrap error with -mmpx -fcheck-pointer-bounds rguenth at gcc dot gnu.org
` (3 preceding siblings ...)
2014-11-25 15:06 ` enkovich.gnu at gmail dot com
@ 2014-11-25 15:45 ` enkovich.gnu at gmail dot com
2014-11-26 8:05 ` enkovich.gnu at gmail dot com
` (4 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: enkovich.gnu at gmail dot com @ 2014-11-25 15:45 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63995
--- Comment #7 from Ilya Enkovich <enkovich.gnu at gmail dot com> ---
In chkpopt pass calls to bndmk are moved down to uses to decrease register
pressure. Debug info introduces new uses and therefore it affects a position
where bndmk calls appear.
-g0 case:
<bb 4>:
r.field = -1;
__bound_tmp.1_13 = __builtin_ia32_bndmk (&r, 4);
test2.chkp (&r, __bound_tmp.1_13);
-g case:
<bb 4>:
# DEBUG c => &r
__bound_tmp.1_13 = __builtin_ia32_bndmk (&r, 4);
# DEBUG __chkp_bounds_of_c => NULL
r.field = -1;
test2.chkp (&r, __bound_tmp.1_13);
Will ignore debug statements when computing a new position for bounds
load/creation (BTW debug statement seems to be damaged by gsi_move_before
called for bndmk). Testing following fix:
diff --git a/gcc/tree-chkp-opt.c b/gcc/tree-chkp-opt.c
index ff390d7..b8d5d0b 100644
--- a/gcc/tree-chkp-opt.c
+++ b/gcc/tree-chkp-opt.c
@@ -1175,7 +1175,9 @@ chkp_reduce_bounds_lifetime (void)
FOR_EACH_IMM_USE_STMT (use_stmt, use_iter, op)
{
- if (dom_bb &&
+ if (is_gimple_debug (use_stmt))
+ continue;
+ else if (dom_bb &&
dominated_by_p (CDI_DOMINATORS,
dom_bb, gimple_bb (use_stmt)))
{
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug bootstrap/63995] Bootstrap error with -mmpx -fcheck-pointer-bounds
2014-11-20 12:33 [Bug bootstrap/63995] New: Bootstrap error with -mmpx -fcheck-pointer-bounds rguenth at gcc dot gnu.org
` (4 preceding siblings ...)
2014-11-25 15:45 ` enkovich.gnu at gmail dot com
@ 2014-11-26 8:05 ` enkovich.gnu at gmail dot com
2014-11-26 14:37 ` ienkovich at gcc dot gnu.org
` (3 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: enkovich.gnu at gmail dot com @ 2014-11-26 8:05 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63995
--- Comment #8 from Ilya Enkovich <enkovich.gnu at gmail dot com> ---
With both patches applied bootstrap is OK
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug bootstrap/63995] Bootstrap error with -mmpx -fcheck-pointer-bounds
2014-11-20 12:33 [Bug bootstrap/63995] New: Bootstrap error with -mmpx -fcheck-pointer-bounds rguenth at gcc dot gnu.org
` (5 preceding siblings ...)
2014-11-26 8:05 ` enkovich.gnu at gmail dot com
@ 2014-11-26 14:37 ` ienkovich at gcc dot gnu.org
2014-12-09 7:54 ` ienkovich at gcc dot gnu.org
` (2 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: ienkovich at gcc dot gnu.org @ 2014-11-26 14:37 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63995
--- Comment #10 from ienkovich at gcc dot gnu.org ---
Author: ienkovich
Date: Wed Nov 26 14:37:07 2014
New Revision: 218085
URL: https://gcc.gnu.org/viewcvs?rev=218085&root=gcc&view=rev
Log:
gcc/
PR bootstrap/63995
* tree-chkp-opt.c (chkp_reduce_bounds_lifetime): Ignore
debug statement when searching for a new position for
bounds load/creation statement.
gcc/testsuite/
PR bootstrap/63995
* gcc.target/i386/pr63995-2.c: New.
Added:
trunk/gcc/testsuite/gcc.target/i386/pr63995-2.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/tree-chkp-opt.c
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug bootstrap/63995] Bootstrap error with -mmpx -fcheck-pointer-bounds
2014-11-20 12:33 [Bug bootstrap/63995] New: Bootstrap error with -mmpx -fcheck-pointer-bounds rguenth at gcc dot gnu.org
` (6 preceding siblings ...)
2014-11-26 14:37 ` ienkovich at gcc dot gnu.org
@ 2014-12-09 7:54 ` ienkovich at gcc dot gnu.org
2014-12-09 10:42 ` enkovich.gnu at gmail dot com
2015-04-16 14:06 ` ienkovich at gcc dot gnu.org
9 siblings, 0 replies; 11+ messages in thread
From: ienkovich at gcc dot gnu.org @ 2014-12-09 7:54 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63995
--- Comment #11 from ienkovich at gcc dot gnu.org ---
Author: ienkovich
Date: Tue Dec 9 07:53:17 2014
New Revision: 218506
URL: https://gcc.gnu.org/viewcvs?rev=218506&root=gcc&view=rev
Log:
gcc/
PR bootstrap/63995
* tree-chkp.c (chkp_make_static_bounds): Share bounds var
between nodes sharing assembler name.
gcc/testsuite/
PR bootstrap/63995
* g++.dg/dg.exp: Add mpx-dg.exp.
* g++.dg/pr63995-1.C: New.
Added:
trunk/gcc/testsuite/g++.dg/pr63995-1.C
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/dg.exp
trunk/gcc/tree-chkp.c
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug bootstrap/63995] Bootstrap error with -mmpx -fcheck-pointer-bounds
2014-11-20 12:33 [Bug bootstrap/63995] New: Bootstrap error with -mmpx -fcheck-pointer-bounds rguenth at gcc dot gnu.org
` (7 preceding siblings ...)
2014-12-09 7:54 ` ienkovich at gcc dot gnu.org
@ 2014-12-09 10:42 ` enkovich.gnu at gmail dot com
2015-04-16 14:06 ` ienkovich at gcc dot gnu.org
9 siblings, 0 replies; 11+ messages in thread
From: enkovich.gnu at gmail dot com @ 2014-12-09 10:42 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63995
--- Comment #12 from Ilya Enkovich <enkovich.gnu at gmail dot com> ---
For r218506 bootstrap with BOOT_CFLAGS="-O2 -g -fcheck-pointer-bounds -mmpx" on
x86_64-unknown-linux-gnu is OK.
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Bug bootstrap/63995] Bootstrap error with -mmpx -fcheck-pointer-bounds
2014-11-20 12:33 [Bug bootstrap/63995] New: Bootstrap error with -mmpx -fcheck-pointer-bounds rguenth at gcc dot gnu.org
` (8 preceding siblings ...)
2014-12-09 10:42 ` enkovich.gnu at gmail dot com
@ 2015-04-16 14:06 ` ienkovich at gcc dot gnu.org
9 siblings, 0 replies; 11+ messages in thread
From: ienkovich at gcc dot gnu.org @ 2015-04-16 14:06 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63995
Ilya Enkovich <ienkovich at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |RESOLVED
CC| |ienkovich at gcc dot gnu.org
Resolution|--- |FIXED
--- Comment #13 from Ilya Enkovich <ienkovich at gcc dot gnu.org> ---
Fixed
^ permalink raw reply [flat|nested] 11+ messages in thread