[Bug inline-asm/61692] ICE in extract_insn in recog.c for asm with many parameters

2015-11-25 Thread mrs at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61692

--- Comment #5 from mrs at gcc dot gnu.org  ---
Mine.  Mine supports 33. Don't expect it to go past 40.  Most folks won't.  In
10-30 years, maybe we might need to update to 50, but we can do that then.

[Bug inline-asm/61692] ICE in extract_insn in recog.c for asm with many parameters

2015-11-25 Thread gccbugzilla at limegreensocks dot com 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61692

--- Comment #4 from David  ---
(In reply to m...@gcc.gnu.org from comment #3)
> This test case isn't portable.  If upped to 40 then it would be more
> portable.

What platform supports more than 30 operands?

As near as I can see, MAX_RECOG_OPERANDS always resolves to 30.  Testing the
difference between 30 and 31 is the edge case.  

So yeah, we could also try 40, but that seems like an arbitrary number.  Why
not 50?  150?

What am I missing here?

[Bug inline-asm/61692] ICE in extract_insn in recog.c for asm with many parameters

2015-11-24 Thread mrs at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61692

mrs at gcc dot gnu.org  changed:

   What|Removed |Added

 CC||mrs at gcc dot gnu.org

--- Comment #3 from mrs at gcc dot gnu.org  ---
This test case isn't portable.  If upped to 40 then it would be more portable.

[Bug inline-asm/61692] ICE in extract_insn in recog.c for asm with many parameters

2015-02-23 Thread gccbugzilla at limegreensocks dot com 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61692

David gccbugzilla at limegreensocks dot com changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #2 from David gccbugzilla at limegreensocks dot com ---
Patch solves problem.

[Bug inline-asm/61692] ICE in extract_insn in recog.c for asm with many parameters

2014-12-08 Thread law at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61692

--- Comment #1 from Jeffrey A. Law law at gcc dot gnu.org ---
Author: law
Date: Mon Dec  8 21:58:23 2014
New Revision: 218494

URL: https://gcc.gnu.org/viewcvs?rev=218494root=gccview=rev
Log:
PR target/61692
* cfgexpand.c (expand_asm_operands): Count all inline asm params.

PR target/61692
* gcc.dg/pr61692.c: New test.

Added:
trunk/gcc/testsuite/gcc.dg/pr61692.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/cfgexpand.c
trunk/gcc/testsuite/ChangeLog