On 04/25/2016 03:11 AM, Bernd Schmidt wrote:
On 03/02/2016 10:53 PM, Vladimir Makarov wrote:
The patch is ok for me. But I'd wait for GCC7. People are sensitive to
their code performance degradation. Even in most cases the patch
improves performance, in some case it can worsen code and people might
fill new PRs after such change.
I've retested now, and it turns out that it turns two guality/sra-1.c
tests into UNSUPPORTED. Apparently a variable is now destructively
overwritten. Is that something we worry about (I'm inclined to think no)?
We'd like to not go backwards if we can.
AFAICT the sra-1.c expects to see the incremented value and I'm at a
loss to understand what's really going on here. Can you give more details?
jeff
