[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-11-19 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 Andrew Pinski changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #15 from Hans-Peter Nilsson --- (In reply to David B. Robins from comment #12) > Note that it does still need -fno-inline. I almost missed. > I know why it doesn't crash with -O0: there is a text-section constant > inserted after th

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 Hans-Peter Nilsson changed: What|Removed |Added Target Milestone|--- |7.0

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #14 from Hans-Peter Nilsson --- Author: hp Date: Mon Jun 20 20:02:03 2016 New Revision: 237616 URL: https://gcc.gnu.org/viewcvs?rev=237616&root=gcc&view=rev Log: PR target/71571 * g++.dg/torture/pr71571.C: New test.

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #13 from Hans-Peter Nilsson --- Author: hp Date: Mon Jun 20 20:01:20 2016 New Revision: 237615 URL: https://gcc.gnu.org/viewcvs?rev=237615&root=gcc&view=rev Log: PR target/71571 * config/cris/cris.c (cris_asm_output_m

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 Hans-Peter Nilsson changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread gcc at davidrobins dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #12 from David B. Robins --- Note that it does still need -fno-inline. I know why it doesn't crash with -O0: there is a text-section constant inserted after the thunk, and it's inlined at higher optimization levels. The instruction s

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #11 from Hans-Peter Nilsson --- (In reply to David B. Robins from comment #10) > My apologies, I saw the example you pointed at was from torture/ but > missed/forgot that you had also wanted the new test there. I moved the file > to t

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread gcc at davidrobins dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #10 from David B. Robins --- My apologies, I saw the example you pointed at was from torture/ but missed/forgot that you had also wanted the new test there. I moved the file to torture and removed -O2 -fno-inline and I didn't get the

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #9 from Hans-Peter Nilsson --- (In reply to Hans-Peter Nilsson from comment #8) > all that's "stopping" optimizations. Bad choice of words, I meant "all that's needed to stop optimizations from making choices that would not expose th

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #8 from Hans-Peter Nilsson --- (In reply to David B. Robins from comment #7) > I have verified that the pr71571.C test case attached does expose the bug on > trunk and that it passes with the above fix. Great, thanks for the report a

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread gcc at davidrobins dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #7 from David B. Robins --- I have verified that the pr71571.C test case attached does expose the bug on trunk and that it passes with the above fix. (Note "-O2 -fno-inline" are required to ensure a crash; see earlier comment; in 4.3.

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-20 Thread gcc at davidrobins dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #6 from David B. Robins --- Created attachment 38734 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38734&action=edit Test case (for gcc/testsuite/g++.dg/inherit)

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-19 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #5 from Hans-Peter Nilsson --- (In reply to David B. Robins from comment #0) > Runs of "make check-gcc-c++ RUNTESTFLAGS=--target_board=cris-sim" > with/without the fix do not differ in passes/failures. But that doesn't use -fpic or -

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-19 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #4 from Hans-Peter Nilsson --- Created attachment 38728 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38728&action=edit fix to be committed, pending test Here's the patch I'll eventually test and commit. Unfortunately, it may

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-17 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 Hans-Peter Nilsson changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-17 Thread hp at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 Hans-Peter Nilsson changed: What|Removed |Added CC||hp at gcc dot gnu.org Ass

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-17 Thread gcc at davidrobins dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 David B. Robins changed: What|Removed |Added Target||crisv32-axis-linux-gnu H

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-17 Thread gcc at davidrobins dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #2 from David B. Robins --- Created attachment 38721 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38721&action=edit Larger repro - crashes reliably - requires additional -O2 -fno-inline

[Bug target/71571] [CRIS] Multiple inheritance non-virtual PIC thunk causes crash

2016-06-17 Thread gcc at davidrobins dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71571 --- Comment #1 from David B. Robins --- Created attachment 38720 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38720&action=edit Small repro - emits branch (ba) without delay-slot instruction