Re: [PATCH] Fix ipa references removal

2014-01-16 Thread Ilya Enkovich
2014/1/16 Jeff Law : > On 01/16/14 07:24, Ilya Enkovich wrote: >> >> Hi, >> >> I found ipa_remove_stmt_references incorrectly iterates over >> references after reference removal. When reference is removed, the >> last one takes it's place and is not checked at all. It means we do >> not remove all

Re: [PATCH] Fix ipa references removal

2014-01-16 Thread Jeff Law
On 01/16/14 07:24, Ilya Enkovich wrote: Hi, I found ipa_remove_stmt_references incorrectly iterates over references after reference removal. When reference is removed, the last one takes it's place and is not checked at all. It means we do not remove all required references in some cases. Here

[PATCH] Fix ipa references removal

2014-01-16 Thread Ilya Enkovich
Hi, I found ipa_remove_stmt_references incorrectly iterates over references after reference removal. When reference is removed, the last one takes it's place and is not checked at all. It means we do not remove all required references in some cases. Here is a patch to fix it. Bootstrapped and tes