Re: [patch][x86] Fix PR83618

2018-01-17 Thread Uros Bizjak 
On Wed, Jan 17, 2018 at 9:56 AM, Koval, Julia  wrote:
> Fix bug, when rdpid intrinsic used eax instead of rax in 64bit mode. Ok for 
> trunk?
>
> gcc/
> * config/i386/i386.c (ix86_expand_builtin): Handle IX86_BUILTIN_RDPID.
> * config/i386/i386.md (rdpid_rex64) New.
> (rdpid): Make 32bit only.
>
> gcc/testsuite/
> * gcc.target/i386/rdpid.c: Remove "eax".

OK, but please fix the comment:

+  /* mode is VOIDmode if __builtin_rd* has been called
+ without lhs.  */

We have __builtin_rdpid here.

Uros.

[patch][x86] Fix PR83618

2018-01-17 Thread Koval, Julia 
Fix bug, when rdpid intrinsic used eax instead of rax in 64bit mode. Ok for 
trunk?

gcc/
* config/i386/i386.c (ix86_expand_builtin): Handle IX86_BUILTIN_RDPID.
* config/i386/i386.md (rdpid_rex64) New.
(rdpid): Make 32bit only.

gcc/testsuite/
* gcc.target/i386/rdpid.c: Remove "eax".

Thanks,
Julia


0001-fix.patch
Description: 0001-fix.patch