This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[PATCH, i386] Fix PR 81294, _subborrow_u64 argument order inconsistent with intrinsic reference

From: Uros Bizjak <ubizjak at gmail dot com>
To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Tue, 4 Jul 2017 22:41:26 +0200
Subject: [PATCH, i386] Fix PR 81294, _subborrow_u64 argument order inconsistent with intrinsic reference
Authentication-results: sourceware.org; auth=none

Hello!

Apparently, Intel changed operand order with the new intrinsic
reference release version. Attached patch updates gcc intrinsic
headers accordingly.

2017-07-04  Uros Bizjak  <ubizjak@gmail.com>

    PR target/81294
    * config/i386/adxintrin.h (_subborrow_u32): Swap _X and _Y
    arguments in the call to __builtin_ia32_sbb_u32.
    (_subborrow_u64): Swap _X and _Y arguments in the call to
    __builtin_ia32_sbb_u64.

testsuite/ChangeLog:

2017-07-04  Uros Bizjak  <ubizjak@gmail.com>

    PR target/81249
    * gcc.target/i386/adx_addcarryx32-2.c (adx_test): Swap
    x and y arguments in the call to _subborrow_u32.
    * gcc.target/i386/adx_addcarryx64-2.c (adx_test): Swap
    x and y arguments in the call to _subborrow_u64.
    * gcc.target/i386/pr81294-1.c: New test.
    * gcc.target/i386/pr81294-2.c: Ditto.

Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}.

Committed to mainline SVN, willbe backported to release branches.

Uros.

Attachment: p.diff.txt
Description: Text document

Follow-Ups:
- Re: [PATCH, i386] Fix PR 81294, _subborrow_u64 argument order inconsistent with intrinsic reference
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]