Re: bswap PRs 69714, 67781

2016-02-14 Thread Thomas Preud'homme
Hi Bernd, First of all, my apologize for the late reply. I was in holidays the past week to celebrate Chinese new year. On Friday, February 12, 2016 05:28:43 PM Bernd Schmidt wrote: > PR69714 is an issue where the bswap pass makes an incorrect > transformation on big-endian targets. The source

Re: bswap PRs 69714, 67781

2016-02-12 Thread Jeff Law
On 02/12/2016 09:28 AM, Bernd Schmidt wrote: PR69714 is an issue where the bswap pass makes an incorrect transformation on big-endian targets. The source has a 32-bit bswap, but PA doesn't have a pattern for that. Still, we recognize that there is a 16-bit bswap involved, and generate code for

bswap PRs 69714, 67781

2016-02-12 Thread Bernd Schmidt
PR69714 is an issue where the bswap pass makes an incorrect transformation on big-endian targets. The source has a 32-bit bswap, but PA doesn't have a pattern for that. Still, we recognize that there is a 16-bit bswap involved, and generate code for that - loading the halfword at offset 2 from

Re: bswap PRs 69714, 67781

2016-02-12 Thread Bernd Schmidt
On 02/13/2016 03:36 AM, John David Anglin wrote: As far as the avcrc.c reduced testcase, it didn't trigger the original bug on hppa-unknown-linux-gnu. Odd, I see the same transformation in the dumps. Do you think you could investigate a bit so we get a useful testcase to add? Bernd

Re: bswap PRs 69714, 67781

2016-02-12 Thread John David Anglin
On 2016-02-12, at 11:28 AM, Bernd Schmidt wrote: > I'm attaching a full patch. John David Anglin is currently running tests on > PA which may take a while. He's confirmed it fixes his testcase, and the > earlier 67781 testcase seems to be cured as well (only looked at generated > assembly,