[Bug target/87883] [ARM] ICE: Segmentation fault in arm_regno_class

2018-11-23 Thread mihail.ionescu at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87883

Mihail Ionescu  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |WONTFIX

--- Comment #3 from Mihail Ionescu  ---
Hi Jozeph,

I will close it for now.
I had a look at your patch and even if the segfault is fixed, it won't work
unless arm_active_target.isa is initialised beforehand which does not seem to
be possible without making some major changes.

Best regards,
Mihail

[Bug target/87883] [ARM] ICE: Segmentation fault in arm_regno_class

2018-11-23 Thread jozef.l at mittosystems dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87883

--- Comment #2 from Jozef Lawrynowicz  ---
Created attachment 45074
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45074=edit
reg class checking patch

Hi Mihail,

Thanks for the info.

I fixed a bug for msp430-elf caused by REGNO_REG_CLASS and REG_CLASS_CONTENTS
being out of sync i.e. REGNO_REG_CLASS[reg] reported one class, but the bit for
that reg in that class was not set in REG_CLASS_CONTENTS.

So I was trying to add some generic warning to the middle-end which would catch
these cases for other targets (see attached patch). However, there were some
other issues with my proposal so I've parked it for now.

Given that I'm not going to proceed with my patch in its current state, it's up
to you whether to close this PR or not.

Kind regards,
Jozef

[Bug target/87883] [ARM] ICE: Segmentation fault in arm_regno_class

2018-11-23 Thread mihail.ionescu at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87883

--- Comment #1 from Mihail Ionescu  ---
Hi Jozef,

The issue is indeed there on trunk/gcc-8/gcc-7, but it is expected. It is
caused by the fact that the "arm_active_target.isa" bitmap is not initialised
by the time "init_reg_sets" is called. This can be fixed by checking if it is
initialised or not in the arm backend, but it is not ideal, as arm_regno_class
(REGNO_REG_CLASS) is not supposed to be queried at that time.

What exactly is your patch trying to do? Is there any way of working around
this within your patch?


Best regards,
Mihail

[Bug target/87883] [ARM] ICE: Segmentation fault in arm_regno_class

2018-11-22 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87883

Thomas Preud'homme  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2018-11-22
 CC||thopre01 at gcc dot gnu.org
   Assignee|unassigned at gcc dot gnu.org  |mihail.ionescu at arm 
dot com
 Ever confirmed|0   |1