[Bug lto/85248] [6/7/8 Regression] Miscompilation when using error function that returns

2018-04-10 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85248 --- Comment #9 from Martin Liška --- Author: marxin Date: Tue Apr 10 07:24:59 2018 New Revision: 259265 URL: https://gcc.gnu.org/viewcvs?rev=259265&root=gcc&view=rev Log: Be more carefull about DECL merging in LTO (PR lto/85248). 2018-04-10 Ri

[Bug lto/85248] [6/7/8 Regression] Miscompilation when using error function that returns

2018-04-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85248 --- Comment #8 from Martin Liška --- Created attachment 43884 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43884&action=edit Patch for GCC 9

[Bug lto/85248] [6/7/8 Regression] Miscompilation when using error function that returns

2018-04-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85248 --- Comment #7 from Martin Liška --- (In reply to Richard Biener from comment #6) > I think we want to merge the cgraph nodes but _not_ merge decls (thus not > call lto_symtab_prevail_decl). Iff that is supported somehow...? Probably > we expec

[Bug lto/85248] [6/7/8 Regression] Miscompilation when using error function that returns

2018-04-09 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85248 --- Comment #6 from Richard Biener --- I think we want to merge the cgraph nodes but _not_ merge decls (thus not call lto_symtab_prevail_decl). Iff that is supported somehow...? Probably we expect the decl <-> cgraph node be a 1:1 relation, not

[Bug lto/85248] [6/7/8 Regression] Miscompilation when using error function that returns

2018-04-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85248 --- Comment #5 from Martin Liška --- Unfortunately the test ICE for couple of MPX test-cases compiled with LTO (I wish we had MPX already removed). So: $ gcc /home/marxin/Programming/gcc/gcc/testsuite/gcc.dg/lto/pr66221_*.c -flto -mmpx -fcheck-p

[Bug lto/85248] [6/7/8 Regression] Miscompilation when using error function that returns

2018-04-06 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85248 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #4

[Bug lto/85248] [6/7/8 Regression] Miscompilation when using error function that returns

2018-04-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85248 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |NEW CC|

[Bug lto/85248] [6/7/8 Regression] Miscompilation when using error function that returns

2018-04-06 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85248 --- Comment #2 from Martin Liška --- (In reply to Andrew Pinski from comment #1) > I suspect this is because error is marked as noreturn in the header file and > not a gcc issue. No, it's not. It's properly distinguished: extern void __error_al

[Bug lto/85248] [6/7/8 Regression] Miscompilation when using error function that returns

2018-04-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85248 Richard Biener changed: What|Removed |Added Priority|P1 |P2 Version|unknown

[Bug lto/85248] [6/7/8 Regression] Miscompilation when using error function that returns

2018-04-06 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85248 --- Comment #1 from Andrew Pinski --- I suspect this is because error is marked as noreturn in the header file and not a gcc issue.

[Bug lto/85248] [6/7/8 Regression] Miscompilation when using error function that returns

2018-04-06 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85248 Martin Liška changed: What|Removed |Added Priority|P3 |P1 Last reconfirmed|