[COMMITTED][PATCH] Fix register corruption bug in ree

2014-09-12 Thread Jiong Wang
On 11/09/14 21:10, Jeff Law wrote: On 09/08/14 11:21, Wilco Dijkstra wrote: Anyway here is the modified check: Thanks. Just needs an updated ChangeLog and it's good for the trunk. committed to trunk on behalf of Wilco as r215205 after pass bootstrapping. 2014-09-12 Wilco Dijkstra

Re: [PATCH] Fix register corruption bug in ree

2014-09-11 Thread Jeff Law
On 09/08/14 11:21, Wilco Dijkstra wrote: Thanks! Jakub noticed a potential problem in this area a while back, but I never came up with any code to trigger and have kept that issue on my todo list ever since. Rather than ensuring the inserted copy write a single register, it seems to me we're be

RE: [PATCH] Fix register corruption bug in ree

2014-09-08 Thread Wilco Dijkstra
> Thanks! Jakub noticed a potential problem in this area a while back, > but I never came up with any code to trigger and have kept that issue on > my todo list ever since. > > Rather than ensuring the inserted copy write a single register, it seems > to me we're better off ensuring that the numb

Re: [PATCH] Fix register corruption bug in ree

2014-09-05 Thread Jeff Law
On 09/04/14 08:05, Wilco Dijkstra wrote: Hi, While changing register costs for AArch64 (patches to follow), the test gcc.target/aarch64/vect-mult.c fails. This is caused by ree inserting a TI mode copy of a DI register. Since TI mode requires 2 registers, this results in silent corruption of t

[PATCH] Fix register corruption bug in ree

2014-09-04 Thread Wilco Dijkstra
Hi, While changing register costs for AArch64 (patches to follow), the test gcc.target/aarch64/vect-mult.c fails. This is caused by ree inserting a TI mode copy of a DI register. Since TI mode requires 2 registers, this results in silent corruption of the 2nd register. After split2: (insn 149