Re: [patch 79279] combine/simplify_set issue

2017-02-02 Thread Aurelien Buhrig
Hi Segher, This patch fixes a combiner bug in simplify_set which calls CANNOT_CHANGE_MODE_CLASS with wrong mode params. It occurs when trying to simplify paradoxical subregs of hw regs (whose natural mode is lower than a word). In fact, changing from (set x:m1

Re: [patch 79279] combine/simplify_set issue

2017-02-01 Thread Aurelien Buhrig
On 31/01/2017 22:15, Segher Boessenkool wrote: > Hello, > > On Mon, Jan 30, 2017 at 10:43:23AM +0100, Aurelien Buhrig wrote: >> This patch fixes a combiner bug in simplify_set which calls >> CANNOT_CHANGE_MODE_CLASS with wrong mode params. >> It occurs when trying to s

[patch 79279] combine/simplify_set issue

2017-01-30 Thread Aurelien Buhrig
is the most relevant for this patch though ... OK to commit? Aurélien Changelog: 2017-01-27 Aurelien Buhrig <aurelien.buhrig@gmail.com> * combine.c (simplify_set): Fix call to REG_CANNOT_CHANGE_MODE_CLASS_P Index: gcc/com

Re: [PATCH] Bug fix in store_bit_field_1 for big endian targets (issue 51893)

2012-03-26 Thread Aurelien Buhrig
Passes bootstrap and regression test powerpc64-linux. Thanks a lot, Alan! So, Aurelien, you only need to adjust the formatting of the patch and post a ChangeLog entry along with it. TIA. Thanks Alan! Bootstrap and regression test for m68k-elf ok, but I have trouble cross compiling

[PATCH] Bug fix in store_bit_field_1 for big endian targets (issue 51893)

2012-03-21 Thread Aurelien Buhrig
Hi, This patch (for 4.6) fixes a wrong subword index computation in store_bit_field_1 for big endian targets when value is at least 4 times bigger than a word (DI REG value with HI words). It fixes a regression on gcc.c-torture/execute/bitfld-3.c for my current backend port.

Re: Bug store_bit_field_1 + patch

2012-01-23 Thread Aurelien Buhrig
Le 21/01/2012 03:37, Alan Modra a écrit : The problem occurs when value is a REG and bitsize BITS_PER_WORD. This is because wordnum, which is used to get the subword of value, is incorrectly computed, in BIG_ENDIAN, wrt the number of words needed by bitsize instead of the number of words

Bug store_bit_field_1 + patch

2012-01-18 Thread Aurelien Buhrig
Hi, I've found a bug in store_bit_field_1 for BIG_ENDIAN targets whose words are HI. The testcase is execute.exp=bitfld-3.c for my target (which is not public). It occurs on 4.6.1 release, but seem to be present in trunk (looking at the code, not executed). The problem occurs when value is a