Re: [Patch] PR 61662

2014-07-16 Thread Gerald Pfeifer
On Wed, 16 Jul 2014, David Wohlferd wrote: >>> 2014-07-09 David Wohlferd >>> >>> PR target/61662 >>> * config/i386/ia32intrin.h: Use __LP64__ to determine size of >>> long >> This is OK for mainline and 4.9 backport. > Thank you for reviewing and approving this. However while

Re: [Patch] PR 61662

2014-07-16 Thread David Wohlferd
On 7/15/2014 11:10 AM, Uros Bizjak wrote: Hello! The detailed description and examples can be found in pr61662, but in short: using "#ifdef __x86_64__" to determine the size of a 'long' does not reliably yield the correct result. This causes _lrotl and _lrotr to return incorrect results on L

Re: [Patch] PR 61662

2014-07-15 Thread Uros Bizjak
Hello! > The detailed description and examples can be found in pr61662, but in short: > using "#ifdef > __x86_64__" to determine the size of a 'long' does not reliably yield the > correct result. This > causes _lrotl and _lrotr to return incorrect results on LLP64 systems (like > Windows). > >

Re: [Patch] PR 61662

2014-07-11 Thread David Wohlferd
Doh! Since the component here is 'Target', I probably should have included the x86-64 Port Maintainer in the TO line. Jan, while I have a release on file with the FSF, I don't have SVN write access. dw On 7/9/2014 12:51 AM, David Wohlferd wrote: As requested, I am posting this patch to gcc