* [Bug rtl-optimization/61641] [4.9 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
@ 2014-06-28 22:45 ` danglin at gcc dot gnu.org
2014-06-28 22:46 ` danglin at gcc dot gnu.org
` (14 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu.org @ 2014-06-28 22:45 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
--- Comment #1 from John David Anglin <danglin at gcc dot gnu.org> ---
Created attachment 33028
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33028&action=edit
opt_vectorize.ii.240r.barriers
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug rtl-optimization/61641] [4.9 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
2014-06-28 22:45 ` [Bug rtl-optimization/61641] " danglin at gcc dot gnu.org
@ 2014-06-28 22:46 ` danglin at gcc dot gnu.org
2014-07-01 1:42 ` danglin at gcc dot gnu.org
` (13 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu.org @ 2014-06-28 22:46 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
--- Comment #2 from John David Anglin <danglin at gcc dot gnu.org> ---
Created attachment 33029
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33029&action=edit
opt_vectorize.ii.241r.dbr
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug rtl-optimization/61641] [4.9 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
2014-06-28 22:45 ` [Bug rtl-optimization/61641] " danglin at gcc dot gnu.org
2014-06-28 22:46 ` danglin at gcc dot gnu.org
@ 2014-07-01 1:42 ` danglin at gcc dot gnu.org
2014-07-07 11:10 ` [Bug rtl-optimization/61641] [4.9/4.10 " rguenth at gcc dot gnu.org
` (12 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu.org @ 2014-07-01 1:42 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
John David Anglin <danglin at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |steven at gcc dot gnu.org
--- Comment #3 from John David Anglin <danglin at gcc dot gnu.org> ---
Introduced in r197266:
https://gcc.gnu.org/ml/gcc-cvs/2013-03/msg00895.html
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug rtl-optimization/61641] [4.9/4.10 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (2 preceding siblings ...)
2014-07-01 1:42 ` danglin at gcc dot gnu.org
@ 2014-07-07 11:10 ` rguenth at gcc dot gnu.org
2014-07-16 13:30 ` jakub at gcc dot gnu.org
` (11 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: rguenth at gcc dot gnu.org @ 2014-07-07 11:10 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Priority|P3 |P2
Target Milestone|--- |4.9.1
Summary|[4.9 Regression] undefined |[4.9/4.10 Regression]
|label in jump_table_data |undefined label in
| |jump_table_data
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug rtl-optimization/61641] [4.9/4.10 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (3 preceding siblings ...)
2014-07-07 11:10 ` [Bug rtl-optimization/61641] [4.9/4.10 " rguenth at gcc dot gnu.org
@ 2014-07-16 13:30 ` jakub at gcc dot gnu.org
2014-08-07 23:32 ` danglin at gcc dot gnu.org
` (10 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-07-16 13:30 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|4.9.1 |4.9.2
--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.9.1 has been released.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug rtl-optimization/61641] [4.9/4.10 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (4 preceding siblings ...)
2014-07-16 13:30 ` jakub at gcc dot gnu.org
@ 2014-08-07 23:32 ` danglin at gcc dot gnu.org
2014-08-08 18:52 ` danglin at gcc dot gnu.org
` (9 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu.org @ 2014-08-07 23:32 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
--- Comment #5 from John David Anglin <danglin at gcc dot gnu.org> ---
Created attachment 33271
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33271&action=edit
Patch
Maybe this will fix.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug rtl-optimization/61641] [4.9/4.10 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (5 preceding siblings ...)
2014-08-07 23:32 ` danglin at gcc dot gnu.org
@ 2014-08-08 18:52 ` danglin at gcc dot gnu.org
2014-08-08 23:37 ` [Bug target/61641] " danglin at gcc dot gnu.org
` (8 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu.org @ 2014-08-08 18:52 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
--- Comment #6 from John David Anglin <danglin at gcc dot gnu.org> ---
The patch does what it's supposed to but it's not right:
.LVL58:
comib,<> 0,%r28,.L70
nop
.L74:
.L80:
.begin_brtab
.word .L73-.L80
.word .L81-.L80
.word .L74-.L80
.word .L84-.L80
.word .L74-.L80
.word .L74-.L80
.word .L74-.L80
.word .L72-.L80
.end_brtab
.L72:
.LBE104:
.LBE103:
.LBB105:
.LBB106:
.loc 2 228 0
ldi 3,%r28
The branch table is in the text section. It appears from the above
that the execution flow can drop into the branch table. So, either
removal of the associated casesi insn was wrong or the branch table
needs to be removed when the table label is removed.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug target/61641] [4.9/4.10 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (6 preceding siblings ...)
2014-08-08 18:52 ` danglin at gcc dot gnu.org
@ 2014-08-08 23:37 ` danglin at gcc dot gnu.org
2014-08-11 16:54 ` danglin at gcc dot gnu.org
` (7 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu.org @ 2014-08-08 23:37 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
John David Anglin <danglin at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Component|rtl-optimization |target
--- Comment #7 from John David Anglin <danglin at gcc dot gnu.org> ---
The brtab markers break delete_related_insns(). On SOM, we need
to use ASM_OUTPUT_ADDR_DIFF_VEC to generate the markers, instead
of pa_reorg.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug target/61641] [4.9/4.10 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (7 preceding siblings ...)
2014-08-08 23:37 ` [Bug target/61641] " danglin at gcc dot gnu.org
@ 2014-08-11 16:54 ` danglin at gcc dot gnu.org
2014-08-16 3:12 ` [Bug target/61641] [4.9/5 " danglin at gcc dot gnu.org
` (6 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu.org @ 2014-08-11 16:54 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
John David Anglin <danglin at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #33271|0 |1
is obsolete| |
--- Comment #8 from John David Anglin <danglin at gcc dot gnu.org> ---
Created attachment 33292
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33292&action=edit
Patch
The attached patch fixes the testcase. However, delay-slot-2.c
now fails when compiling 32-bit non PIC code.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug target/61641] [4.9/5 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (8 preceding siblings ...)
2014-08-11 16:54 ` danglin at gcc dot gnu.org
@ 2014-08-16 3:12 ` danglin at gcc dot gnu.org
2014-08-16 16:37 ` danglin at gcc dot gnu.org
` (5 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu.org @ 2014-08-16 3:12 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
--- Comment #9 from John David Anglin <danglin at gcc dot gnu.org> ---
With the patch from comment #8, I find the failure of delay-slot-2.c
was introduced in r209897:
2014-03-21 James Greenhalgh <james.greenhalgh@arm.com>
* calls.c (initialize_argument_information): Always treat
PUSH_ARGS_REVERSED as 1, simplify code accordingly.
(expand_call): Likewise.
(emit_library_call_calue_1): Likewise.
* expr.c (PUSH_ARGS_REVERSED): Do not define.
(emit_push_insn): Always treat PUSH_ARGS_REVERSED as 1, simplify
code accordingly.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug target/61641] [4.9/5 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (9 preceding siblings ...)
2014-08-16 3:12 ` [Bug target/61641] [4.9/5 " danglin at gcc dot gnu.org
@ 2014-08-16 16:37 ` danglin at gcc dot gnu.org
2014-08-16 16:39 ` danglin at gcc dot gnu.org
` (4 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu.org @ 2014-08-16 16:37 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
--- Comment #10 from John David Anglin <danglin at gcc dot gnu.org> ---
Author: danglin
Date: Sat Aug 16 16:36:33 2014
New Revision: 214064
URL: https://gcc.gnu.org/viewcvs?rev=214064&root=gcc&view=rev
Log:
PR target/61641
* config/pa/pa-protos.h (pa_output_addr_vec, pa_output_addr_diff_vec):
Declare.
* config/pa/pa.c (pa_reorg): Remove code to insert brtab marker insns.
(pa_output_addr_vec, pa_output_addr_diff_vec): New.
* config/pa/pa.h (ASM_OUTPUT_ADDR_VEC, ASM_OUTPUT_ADDR_DIFF_VEC):
Define.
* config/pa/pa.md (begin_brtab): Delete insn.
(end_brtab): Likewise.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/pa/pa-protos.h
trunk/gcc/config/pa/pa.c
trunk/gcc/config/pa/pa.h
trunk/gcc/config/pa/pa.md
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug target/61641] [4.9/5 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (10 preceding siblings ...)
2014-08-16 16:37 ` danglin at gcc dot gnu.org
@ 2014-08-16 16:39 ` danglin at gcc dot gnu.org
2014-09-01 9:23 ` jakub at gcc dot gnu.org
` (3 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu.org @ 2014-08-16 16:39 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
--- Comment #11 from John David Anglin <danglin at gcc dot gnu.org> ---
Author: danglin
Date: Sat Aug 16 16:39:17 2014
New Revision: 214065
URL: https://gcc.gnu.org/viewcvs?rev=214065&root=gcc&view=rev
Log:
PR target/61641
* config/pa/pa-protos.h (pa_output_addr_vec, pa_output_addr_diff_vec):
Declare.
* config/pa/pa.c (pa_reorg): Remove code to insert brtab marker insns.
(pa_output_addr_vec, pa_output_addr_diff_vec): New.
* config/pa/pa.h (ASM_OUTPUT_ADDR_VEC, ASM_OUTPUT_ADDR_DIFF_VEC):
Define.
* config/pa/pa.md (begin_brtab): Delete insn.
(end_brtab): Likewise.
Modified:
branches/gcc-4_9-branch/gcc/ChangeLog
branches/gcc-4_9-branch/gcc/config/pa/pa-protos.h
branches/gcc-4_9-branch/gcc/config/pa/pa.c
branches/gcc-4_9-branch/gcc/config/pa/pa.h
branches/gcc-4_9-branch/gcc/config/pa/pa.md
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug target/61641] [4.9/5 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (11 preceding siblings ...)
2014-08-16 16:39 ` danglin at gcc dot gnu.org
@ 2014-09-01 9:23 ` jakub at gcc dot gnu.org
2014-09-01 15:57 ` dave.anglin at bell dot net
` (2 subsequent siblings)
15 siblings, 0 replies; 17+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-09-01 9:23 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jakub at gcc dot gnu.org
--- Comment #12 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
So fixed?
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug target/61641] [4.9/5 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (12 preceding siblings ...)
2014-09-01 9:23 ` jakub at gcc dot gnu.org
@ 2014-09-01 15:57 ` dave.anglin at bell dot net
2014-10-30 10:41 ` jakub at gcc dot gnu.org
2015-01-18 17:46 ` danglin at gcc dot gnu.org
15 siblings, 0 replies; 17+ messages in thread
From: dave.anglin at bell dot net @ 2014-09-01 15:57 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
--- Comment #13 from dave.anglin at bell dot net ---
On 1-Sep-14, at 5:23 AM, jakub at gcc dot gnu.org wrote:
> So fixed?
I left it open because of the following fail on hppa-unknown-linux-gnu:
FAIL: gcc.dg/delay-slot-2.c scan-assembler printf
The table jump and related insns are incorrectly removed. Haven't had
time
to determine why.
Dave
--
John David Anglin dave.anglin@bell.net
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug target/61641] [4.9/5 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (13 preceding siblings ...)
2014-09-01 15:57 ` dave.anglin at bell dot net
@ 2014-10-30 10:41 ` jakub at gcc dot gnu.org
2015-01-18 17:46 ` danglin at gcc dot gnu.org
15 siblings, 0 replies; 17+ messages in thread
From: jakub at gcc dot gnu.org @ 2014-10-30 10:41 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|4.9.2 |4.9.3
--- Comment #14 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.9.2 has been released.
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Bug target/61641] [4.9/5 Regression] undefined label in jump_table_data
2014-06-28 22:44 [Bug rtl-optimization/61641] New: [4.9 Regression] undefined label in jump_table_data danglin at gcc dot gnu.org
` (14 preceding siblings ...)
2014-10-30 10:41 ` jakub at gcc dot gnu.org
@ 2015-01-18 17:46 ` danglin at gcc dot gnu.org
15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu.org @ 2015-01-18 17:46 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61641
John David Anglin <danglin at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |RESOLVED
Resolution|--- |FIXED
--- Comment #15 from John David Anglin <danglin at gcc dot gnu.org> ---
The failure mentioned in comment #13 is gone.
^ permalink raw reply [flat|nested] 17+ messages in thread