public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/57224] New: Remove __builtin_ia32_cmpngtss and __builtin_ia32_cmpngess
@ 2013-05-09  9:23 glisse at gcc dot gnu.org
  2013-06-27 20:10 ` [Bug target/57224] " glisse at gcc dot gnu.org
  0 siblings, 1 reply; 2+ messages in thread
From: glisse at gcc dot gnu.org @ 2013-05-09  9:23 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57224

             Bug #: 57224
           Summary: Remove __builtin_ia32_cmpngtss and
                    __builtin_ia32_cmpngess
    Classification: Unclassified
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: minor
          Priority: P3
         Component: target
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: glisse@gcc.gnu.org
            Target: x86_64-linux-gnu


Hello,

the builtins __builtin_ia32_cmpngtss and __builtin_ia32_cmpngess are unused
(_mm_cmpngt_ss uses __builtin_ia32_cmpnltss instead), undocumented, and produce
confusing results (they swap operands, so the high elements of the vector are
taken from the second operand instead of the first one like all other cmp*ss).
I think it would be best to remove them.


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [Bug target/57224] Remove __builtin_ia32_cmpngtss and __builtin_ia32_cmpngess
  2013-05-09  9:23 [Bug target/57224] New: Remove __builtin_ia32_cmpngtss and __builtin_ia32_cmpngess glisse at gcc dot gnu.org
@ 2013-06-27 20:10 ` glisse at gcc dot gnu.org
  0 siblings, 0 replies; 2+ messages in thread
From: glisse at gcc dot gnu.org @ 2013-06-27 20:10 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57224

Marc Glisse <glisse at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
           Assignee|unassigned at gcc dot gnu.org      |glisse at gcc dot gnu.org

--- Comment #1 from Marc Glisse <glisse at gcc dot gnu.org> ---
r200492 | glisse | 2013-06-27 22:07:32 +0200 (Thu, 27 Jun 2013) | 7 lines

2013-06-27  Marc Glisse  <marc.glisse@inria.fr>

        PR target/57224
        * config/i386/i386.c (enum ix86_builtins, bdesc_args): Remove
        IX86_BUILTIN_CMPNGTSS and IX86_BUILTIN_CMPNGESS.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-06-27 20:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-05-09  9:23 [Bug target/57224] New: Remove __builtin_ia32_cmpngtss and __builtin_ia32_cmpngess glisse at gcc dot gnu.org
2013-06-27 20:10 ` [Bug target/57224] " glisse at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).