Re: [PATCH] PR62120

2014-09-15 Thread Ilya Tocar
On 01 Sep 18:38, Ilya Tocar wrote: > > Please mention the PR in the ChangeLog entry and add some testcases > > (can be gcc.target/i386/, but we should have it tested). > > Does this change anything on say register short sil __asm ("sil"); in 32-bit > > mode (when it IMHO should be rejected too?)? >

Re: [PATCH] PR62120

2014-09-01 Thread Ilya Tocar
> Please mention the PR in the ChangeLog entry and add some testcases > (can be gcc.target/i386/, but we should have it tested). > Does this change anything on say register short sil __asm ("sil"); in 32-bit > mode (when it IMHO should be rejected too?)? > Do we support "sil" at all? In i386.h i se

Re: [PATCH] PR62120

2014-09-01 Thread Jakub Jelinek
On Mon, Sep 01, 2014 at 02:43:14PM +0400, Ilya Tocar wrote: > Hi, this patch adds checks for registers availability, when > alternative/numeric name is used. > Bootstraps/passes make-check on x86-64. > Ok for trunk? > > ChangeLog: > > gcc/ > > 2014-09-01 Ilya Tocar > > * varasm.c (deco

[PATCH] PR62120

2014-09-01 Thread Ilya Tocar
Hi, this patch adds checks for registers availability, when alternative/numeric name is used. Bootstraps/passes make-check on x86-64. Ok for trunk? ChangeLog: gcc/ 2014-09-01 Ilya Tocar * varasm.c (decode_reg_name_and_count): Check availability for registers from ADDITIONAL_R