Pitchumani Sivanupandi schrieb:
Hi,
This patch introduces new flags for warning 'misspelled interrupt/
signal handler'. Flag -Wmisspelled-isr is enabled by default and it
will warn user if the interrupt/ signal handler is without '__vector'
prefix. Flag -Wno-misspelled-isr shall be enabled by user to allow
custom names, i.e. without __vector prefix.
// avr-gcc -c test.c
void custom_interruption(void) __attribute__((signal));
void custom_interruption(void) {}
Behavior after applying this patch:
$ avr-gcc test.c
test.c: In function 'custom_interruption':
test.c:2:6: warning: 'custom_interruption' appears to be a misspelled
signal handler
void custom_interruption(void) {}
^~~~~~~~~~~~~~~~~~~
$ avr-gcc test.c -Wmisspelled-isr
test.c: In function
'custom_interruption':
test.c:2:6: warning: 'custom_interruption'
appears to be a misspelled signal handler
void
custom_interruption(void) {}
^~~~~~~~~~~~~~~~~~~
$ avr-gcc test.c -Wno-misspelled-isr
$
If OK, could someone commit please? I do not have commit access.
Regards,
Pitchumani
gcc/ChangeLog
2016-06-10 Pitchumani Sivanupandi <pitchuman...@atmel.com>
Missing PR target/67353
* config/avr/avr.c (avr_set_current_function): Warn misspelled
interrupt/ signal handler if warn_misspelled_isr flag is set.
* config/avr/avr.opt (Wmisspelled-isr): New warning flag. Enabled
by default to warn misspelled interrupt/ signal handler.
Shouldn't it also be documented in doc/invoke.texi?
Johann
