public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug libstdc++/53543] New: [unordered_map] conflict with __is_convertible clang intrinsic
@ 2012-05-31 17:48 zeratul976 at hotmail dot com
  2012-05-31 18:36 ` [Bug libstdc++/53543] " redi at gcc dot gnu.org
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: zeratul976 at hotmail dot com @ 2012-05-31 17:48 UTC (permalink / raw)
  To: gcc-bugs

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

             Bug #: 53543
           Summary: [unordered_map] conflict with __is_convertible clang
                    intrinsic
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libstdc++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: zeratul976@hotmail.com


The trunk implementation of unordered_map uses the following alias template at
class scope:

template<typename _Pair>
using __is_convertible = std::is_convertible<_Pair, value_type>;

Clang cannot compile this because __is_convertible is a clang intrinsic (much
like how __is_base_of is a gcc intrinsic).

I asked on the clang mailing list what can be done about this, and they
suggested:

"Since GCC 4.8 is not yet released, and Clang versions with the 
__is_convertible intrinsic are, the best solution would be to politely 
ask the libstdc++ maintainers to change this name."

Would it be possible to entertain this request?


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

* [Bug libstdc++/53543] [unordered_map] conflict with __is_convertible clang intrinsic
  2012-05-31 17:48 [Bug libstdc++/53543] New: [unordered_map] conflict with __is_convertible clang intrinsic zeratul976 at hotmail dot com
@ 2012-05-31 18:36 ` redi at gcc dot gnu.org
  2012-05-31 21:18 ` paolo.carlini at oracle dot com
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: redi at gcc dot gnu.org @ 2012-05-31 18:36 UTC (permalink / raw)
  To: gcc-bugs

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

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2012-05-31
   Target Milestone|---                         |4.8.0
     Ever Confirmed|0                           |1

--- Comment #1 from Jonathan Wakely <redi at gcc dot gnu.org> 2012-05-31 18:04:57 UTC ---
Sounds reasonable, thanks for noticing and pointing it out.


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

* [Bug libstdc++/53543] [unordered_map] conflict with __is_convertible clang intrinsic
  2012-05-31 17:48 [Bug libstdc++/53543] New: [unordered_map] conflict with __is_convertible clang intrinsic zeratul976 at hotmail dot com
  2012-05-31 18:36 ` [Bug libstdc++/53543] " redi at gcc dot gnu.org
@ 2012-05-31 21:18 ` paolo.carlini at oracle dot com
  2012-05-31 22:15 ` bkoz at gcc dot gnu.org
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: paolo.carlini at oracle dot com @ 2012-05-31 21:18 UTC (permalink / raw)
  To: gcc-bugs

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

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bkoz at redhat dot com

--- Comment #2 from Paolo Carlini <paolo.carlini at oracle dot com> 2012-05-31 21:13:39 UTC ---
If I'm not mistaken, Benjamin added these alias templates, thus let's politely
ask him which name he prefers.


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

* [Bug libstdc++/53543] [unordered_map] conflict with __is_convertible clang intrinsic
  2012-05-31 17:48 [Bug libstdc++/53543] New: [unordered_map] conflict with __is_convertible clang intrinsic zeratul976 at hotmail dot com
  2012-05-31 18:36 ` [Bug libstdc++/53543] " redi at gcc dot gnu.org
  2012-05-31 21:18 ` paolo.carlini at oracle dot com
@ 2012-05-31 22:15 ` bkoz at gcc dot gnu.org
  2012-05-31 23:04 ` bkoz at gcc dot gnu.org
  2012-05-31 23:57 ` bkoz at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: bkoz at gcc dot gnu.org @ 2012-05-31 22:15 UTC (permalink / raw)
  To: gcc-bugs

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

Benjamin Kosnik <bkoz at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |bkoz at gcc dot gnu.org
         AssignedTo|unassigned at gcc dot       |bkoz at gcc dot gnu.org
                   |gnu.org                     |

--- Comment #3 from Benjamin Kosnik <bkoz at gcc dot gnu.org> 2012-05-31 22:07:12 UTC ---
On it, this is a reasonable request. I've changed it to __is_conv to match the
enable-if typedefs.

-benjamin


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

* [Bug libstdc++/53543] [unordered_map] conflict with __is_convertible clang intrinsic
  2012-05-31 17:48 [Bug libstdc++/53543] New: [unordered_map] conflict with __is_convertible clang intrinsic zeratul976 at hotmail dot com
                   ` (2 preceding siblings ...)
  2012-05-31 22:15 ` bkoz at gcc dot gnu.org
@ 2012-05-31 23:04 ` bkoz at gcc dot gnu.org
  2012-05-31 23:57 ` bkoz at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: bkoz at gcc dot gnu.org @ 2012-05-31 23:04 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #4 from Benjamin Kosnik <bkoz at gcc dot gnu.org> 2012-05-31 23:02:24 UTC ---
Author: bkoz
Date: Thu May 31 23:02:18 2012
New Revision: 188088

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188088
Log:
2012-05-31  Benjamin Kosnik  <bkoz@redhat.com>

    PR libstdc++/53543
    * include/bits/hashtable_policy.h (_Insert::__is_convertible):
    Rename to __is_conv to avoid clash with clang built-in.

Modified:
    trunk/libstdc++-v3/ChangeLog
    trunk/libstdc++-v3/include/bits/hashtable_policy.h


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

* [Bug libstdc++/53543] [unordered_map] conflict with __is_convertible clang intrinsic
  2012-05-31 17:48 [Bug libstdc++/53543] New: [unordered_map] conflict with __is_convertible clang intrinsic zeratul976 at hotmail dot com
                   ` (3 preceding siblings ...)
  2012-05-31 23:04 ` bkoz at gcc dot gnu.org
@ 2012-05-31 23:57 ` bkoz at gcc dot gnu.org
  4 siblings, 0 replies; 6+ messages in thread
From: bkoz at gcc dot gnu.org @ 2012-05-31 23:57 UTC (permalink / raw)
  To: gcc-bugs

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

Benjamin Kosnik <bkoz at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #5 from Benjamin Kosnik <bkoz at gcc dot gnu.org> 2012-05-31 23:55:46 UTC ---

Fixed.


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

end of thread, other threads:[~2012-05-31 23:56 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-31 17:48 [Bug libstdc++/53543] New: [unordered_map] conflict with __is_convertible clang intrinsic zeratul976 at hotmail dot com
2012-05-31 18:36 ` [Bug libstdc++/53543] " redi at gcc dot gnu.org
2012-05-31 21:18 ` paolo.carlini at oracle dot com
2012-05-31 22:15 ` bkoz at gcc dot gnu.org
2012-05-31 23:04 ` bkoz at gcc dot gnu.org
2012-05-31 23:57 ` bkoz 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).