[Bug target/34299] [avr] ICE on function attribute syntax for main()

2009-01-24 Thread eric dot weddington at atmel dot com


--- Comment #6 from eric dot weddington at atmel dot com  2009-01-24 23:35 
---
*** Bug 38954 has been marked as a duplicate of this bug. ***


-- 

eric dot weddington at atmel dot com changed:

   What|Removed |Added

 CC||charles at chaoslizard dot
   ||org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2009-02-25 Thread pinskia at gcc dot gnu dot org


--- Comment #7 from pinskia at gcc dot gnu dot org  2009-02-25 18:50 ---
*** Bug 39307 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||regehr at cs dot utah dot
   ||edu


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2009-02-25 Thread regehr at cs dot utah dot edu


--- Comment #8 from regehr at cs dot utah dot edu  2009-02-25 20:01 ---
Sorry for the dup.

This bug prevents compilation of almost all TinyOS apps using 4.3.3!  Is there
a workaround?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2009-02-25 Thread eric dot weddington at atmel dot com


--- Comment #9 from eric dot weddington at atmel dot com  2009-02-26 01:00 
---
(In reply to comment #8)
> Sorry for the dup.
> 
> This bug prevents compilation of almost all TinyOS apps using 4.3.3!  Is there
> a workaround?
> 

See comment #4 of this bug.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2009-02-26 Thread jxyang at cs dot utah dot edu


--- Comment #10 from jxyang at cs dot utah dot edu  2009-02-26 23:24 ---
Created an attachment (id=17370)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17370&action=view)
patch for bug 34299

DECL_ASSEMBLER_NAME has a side effect of setting assembler_name for functions
if that attribute is missing, which causes the assertion failure in c-decl.c.

Change it to less harmful DECL_NAME.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2009-02-26 Thread regehr at cs dot utah dot edu


--- Comment #11 from regehr at cs dot utah dot edu  2009-02-26 23:29 ---
Subject: Re:  [avr] ICE on function attribute syntax for
 main()

Great!

John


jxyang at cs dot utah dot edu wrote:
> --- Comment #10 from jxyang at cs dot utah dot edu  2009-02-26 23:24 
> ---
> Created an attachment (id=17370)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17370&action=view)
>  --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17370&action=view)
> patch for bug 34299
> 
> DECL_ASSEMBLER_NAME has a side effect of setting assembler_name for functions
> if that attribute is missing, which causes the assertion failure in c-decl.c.
> 
> Change it to less harmful DECL_NAME.
> 
> 


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2009-02-27 Thread aesok at gcc dot gnu dot org


--- Comment #12 from aesok at gcc dot gnu dot org  2009-02-27 16:25 ---
2007-04-05  Anatoly Sokolov 

PR target/25448
* config/avr/avr.c (avr_handle_fndecl_attribute): Use the
DECL_ASSEMBLER_NAME, not the DECL_NAME.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2009-03-02 Thread aesok at gcc dot gnu dot org


--- Comment #13 from aesok at gcc dot gnu dot org  2009-03-02 21:54 ---
*** Bug 38558 has been marked as a duplicate of this bug. ***


-- 

aesok at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||jw at suse dot de


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2009-03-14 Thread aesok at gcc dot gnu dot org


-- 

aesok at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |aesok at gcc dot gnu dot org
   |dot org |
 Status|NEW |ASSIGNED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2009-03-15 Thread aesok at gcc dot gnu dot org


--- Comment #14 from aesok at gcc dot gnu dot org  2009-03-15 13:09 ---
Subject: Bug 34299

Author: aesok
Date: Sun Mar 15 13:09:44 2009
New Revision: 144870

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=144870
Log:
PR target/34299
* config/avr/avr.c (avr_handle_fndecl_attribute): Move code for
generate a warning if the function name does not begin with
"__vector" and the function has either the 'signal' or 'interrupt'
attribute, from here to ...
(avr_declare_function_name): ...here. New function.
* config/avr/avr.h (ASM_DECLARE_FUNCTION_NAME): Redefine.
* config/avr/avr-protos.h (avr_declare_function_name): Declare.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/avr/avr-protos.h
trunk/gcc/config/avr/avr.c
trunk/gcc/config/avr/avr.h


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2009-03-15 Thread aesok at gcc dot gnu dot org


--- Comment #15 from aesok at gcc dot gnu dot org  2009-03-15 13:14 ---
Fixed.


-- 

aesok at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED
   Target Milestone|--- |4.4.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2007-12-02 Thread pinskia at gcc dot gnu dot org


--- Comment #1 from pinskia at gcc dot gnu dot org  2007-12-02 23:34 ---
Can you attach the preprocessed source?


-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |WAITING


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2007-12-02 Thread eric dot weddington at atmel dot com


--- Comment #2 from eric dot weddington at atmel dot com  2007-12-03 05:39 
---
The subject line of this bug is the same subject as bug #31331, which is fixed.
Without having the preprocessed source it is difficult to determine if this bug
is a duplicate.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2008-01-10 Thread j at uriah dot heep dot sax dot de


--- Comment #3 from j at uriah dot heep dot sax dot de  2008-01-10 15:56 
---
Some bugs appear to re-appear. :-(

While I get

__attribute__((naked))
int main(void)
{
  // ...
   return 42;
}

to compile with the current compiler, the following piece of code:

__attribute__((signal, naked))
void __vector_42(void)
{
   static unsigned char x;

   x++;
}

still triggers the ICE:

% avr-gcc -mmcu=attiny2313 -S foo.c
foo.c:3: internal compiler error: in start_function, at c-decl.c:6177


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2008-01-10 Thread eric dot weddington at atmel dot com


--- Comment #4 from eric dot weddington at atmel dot com  2008-01-10 17:56 
---
(In reply to comment #3)

> __attribute__((signal, naked))
> void __vector_42(void)
> {
>static unsigned char x;
> 
>x++;
> }
> 
> still triggers the ICE:


However, if there is a function prototype, and the attributes are placed on the
prototype, there is no bug:

__attribute__((signal, naked))
void __vector_42(void);

void __vector_42(void)
{
   static unsigned char x;

   x++;
}

Is it really a bug?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2008-01-10 Thread eric dot weddington at atmel dot com


-- 

eric dot weddington at atmel dot com changed:

   What|Removed |Added

 Status|WAITING |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2008-01-10 18:07:45
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299



[Bug target/34299] [avr] ICE on function attribute syntax for main()

2008-01-10 Thread steven at gcc dot gnu dot org


--- Comment #5 from steven at gcc dot gnu dot org  2008-01-10 18:05 ---
If there is an ICE, there is a bug.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34299