Re: genattrab.c generate switch

2016-02-01 Thread Jesper Broge Jørgensen
On 01/02/16 20:19, Patrick Palka wrote: On Tue, Jan 12, 2016 at 7:53 PM, Jesper Broge Jørgensen wrote: Hello genattrab.c can generate if statements that have very deep bracket nesting causing clang to produce errors (when target=arm-none-eabi) as explained at https://gcc.gnu.org/ml/gcc/2014-0

Re: genattrab.c generate switch

2016-02-01 Thread Patrick Palka
On Tue, Jan 12, 2016 at 7:53 PM, Jesper Broge Jørgensen wrote: > Hello > > genattrab.c can generate if statements that have very deep bracket nesting > causing clang to produce errors (when target=arm-none-eabi) as explained at > https://gcc.gnu.org/ml/gcc/2014-05/msg00032.html > At the above link

Re: genattrab.c generate switch

2016-02-01 Thread Jesper Broge Jørgensen
On 19/01/16 13:18, Bernd Schmidt wrote: On 01/18/2016 11:44 PM, Jesper Broge Jørgensen wrote: I found a formatting tool called uncrustify that comes with a gnu style config https://github.com/bengardner/uncrustify/blob/master/etc/gnu-indent.cfg that needed a few tweaks to format code that look

Re: genattrab.c generate switch

2016-01-19 Thread Bernd Schmidt
On 01/18/2016 11:44 PM, Jesper Broge Jørgensen wrote: I found a formatting tool called uncrustify that comes with a gnu style config https://github.com/bengardner/uncrustify/blob/master/etc/gnu-indent.cfg that needed a few tweaks to format code that looked what is already in gcc/genattrtab.c The

Re: genattrab.c generate switch

2016-01-18 Thread Jesper Broge Jørgensen
On 18/01/16 18:39, Manuel López-Ibáñez wrote: On 18/01/16 14:39, Jesper Broge Jørgensen wrote: No i have not gone through copyright assignment. This is my first time trying to contribute to a GNU project so i have tried following the "Contributing to GCC"@ https://gcc.gnu.org/contribute.html

Re: genattrab.c generate switch

2016-01-18 Thread Manuel López-Ibáñez
On 18/01/16 14:39, Jesper Broge Jørgensen wrote: No i have not gone through copyright assignment. This is my first time trying to contribute to a GNU project so i have tried following the "Contributing to GCC"@ https://gcc.gnu.org/contribute.html There i followed the advice to run the patch throu

Re: genattrab.c generate switch

2016-01-18 Thread Jesper Broge Jørgensen
On 18/01/16 15:15, Bernd Schmidt wrote: On 01/13/2016 01:53 AM, Jesper Broge Jørgensen wrote: genattrab.c can generate if statements that have very deep bracket nesting causing clang to produce errors (when target=arm-none-eabi) as explained at https://gcc.gnu.org/ml/gcc/2014-05/msg00032.html A

Re: genattrab.c generate switch

2016-01-18 Thread Jakub Jelinek
On Mon, Jan 18, 2016 at 03:15:08PM +0100, Bernd Schmidt wrote: > Secondly, we're currently in a development phase where we only accept bug > fixes for gcc-6. You should resubmit/ping the patch once stage1 opens again. I think this is a bug fix, it is a workaround for a broken compiler that some pe

Re: genattrab.c generate switch

2016-01-18 Thread Bernd Schmidt
On 01/13/2016 01:53 AM, Jesper Broge Jørgensen wrote: genattrab.c can generate if statements that have very deep bracket nesting causing clang to produce errors (when target=arm-none-eabi) as explained at https://gcc.gnu.org/ml/gcc/2014-05/msg00032.html At the above link it was suggested that gen