Re: [PATCH PR84969]Don't reorder builtin memsets if they set different rhs values

2018-03-21 Thread Richard Biener
On Tue, Mar 20, 2018 at 7:18 PM, Bin Cheng wrote: > Hi, > As noted in PR84969, fuse_memset_builtins breaks dependence between different > memsets. > Specifically, it reorders different builtin memset partitions though it > doesn't merge > them in the end. This simple patch fixes this wrong code

Re: [PATCH PR84969]Don't reorder builtin memsets if they set different rhs values

2018-03-20 Thread Martin Liška
On 03/20/2018 07:18 PM, Bin Cheng wrote: Hi, As noted in PR84969, fuse_memset_builtins breaks dependence between different memsets. Specifically, it reorders different builtin memset partitions though it doesn't merge them in the end. This simple patch fixes this wrong code issue by checking i

[PATCH PR84969]Don't reorder builtin memsets if they set different rhs values

2018-03-20 Thread Bin Cheng
Hi, As noted in PR84969, fuse_memset_builtins breaks dependence between different memsets. Specifically, it reorders different builtin memset partitions though it doesn't merge them in the end. This simple patch fixes this wrong code issue by checking if any two builtin memsets set the same rhs