* [Bug libstdc++/104602] std::source_location::current uses cast from void*
2022-02-19 3:31 [Bug libstdc++/104602] New: std::source_location::current uses cast from void* foom at fuhm dot net
@ 2022-02-19 3:50 ` pinskia at gcc dot gnu.org
2022-02-19 12:06 ` redi at gcc dot gnu.org
` (9 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: pinskia at gcc dot gnu.org @ 2022-02-19 3:50 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104602
--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
https://gcc.gnu.org/pipermail/gcc-patches/2019-November/534374.html
Explains why it is currently this way.
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug libstdc++/104602] std::source_location::current uses cast from void*
2022-02-19 3:31 [Bug libstdc++/104602] New: std::source_location::current uses cast from void* foom at fuhm dot net
2022-02-19 3:50 ` [Bug libstdc++/104602] " pinskia at gcc dot gnu.org
@ 2022-02-19 12:06 ` redi at gcc dot gnu.org
2022-02-19 12:56 ` jakub at gcc dot gnu.org
` (8 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: redi at gcc dot gnu.org @ 2022-02-19 12:06 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104602
Jonathan Wakely <redi at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Ever confirmed|0 |1
Status|UNCONFIRMED |NEW
Last reconfirmed| |2022-02-19
--- Comment #2 from Jonathan Wakely <redi at gcc dot gnu.org> ---
But we can certainly move the cast into the default argument.
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug libstdc++/104602] std::source_location::current uses cast from void*
2022-02-19 3:31 [Bug libstdc++/104602] New: std::source_location::current uses cast from void* foom at fuhm dot net
2022-02-19 3:50 ` [Bug libstdc++/104602] " pinskia at gcc dot gnu.org
2022-02-19 12:06 ` redi at gcc dot gnu.org
@ 2022-02-19 12:56 ` jakub at gcc dot gnu.org
2022-02-21 14:58 ` foom at fuhm dot net
` (7 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: jakub at gcc dot gnu.org @ 2022-02-19 12:56 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104602
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jakub at gcc dot gnu.org
--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Can't we make it
static consteval source_location
current(decltype (__builtin_source_location ()) __p
= __builtin_source_location()) noexcept
?
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug libstdc++/104602] std::source_location::current uses cast from void*
2022-02-19 3:31 [Bug libstdc++/104602] New: std::source_location::current uses cast from void* foom at fuhm dot net
` (2 preceding siblings ...)
2022-02-19 12:56 ` jakub at gcc dot gnu.org
@ 2022-02-21 14:58 ` foom at fuhm dot net
2022-02-21 15:05 ` jakub at gcc dot gnu.org
` (6 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: foom at fuhm dot net @ 2022-02-21 14:58 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104602
--- Comment #4 from James Y Knight <foom at fuhm dot net> ---
Yea that should work. Or even just `auto`.
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug libstdc++/104602] std::source_location::current uses cast from void*
2022-02-19 3:31 [Bug libstdc++/104602] New: std::source_location::current uses cast from void* foom at fuhm dot net
` (3 preceding siblings ...)
2022-02-21 14:58 ` foom at fuhm dot net
@ 2022-02-21 15:05 ` jakub at gcc dot gnu.org
2022-02-21 21:06 ` redi at gcc dot gnu.org
` (5 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: jakub at gcc dot gnu.org @ 2022-02-21 15:05 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104602
--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
No, with auto it doesn't work.
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug libstdc++/104602] std::source_location::current uses cast from void*
2022-02-19 3:31 [Bug libstdc++/104602] New: std::source_location::current uses cast from void* foom at fuhm dot net
` (4 preceding siblings ...)
2022-02-21 15:05 ` jakub at gcc dot gnu.org
@ 2022-02-21 21:06 ` redi at gcc dot gnu.org
2022-02-24 23:46 ` cvs-commit at gcc dot gnu.org
` (4 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: redi at gcc dot gnu.org @ 2022-02-21 21:06 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104602
--- Comment #6 from Jonathan Wakely <redi at gcc dot gnu.org> ---
We can make that more readable:
--- a/libstdc++-v3/include/std/source_location
+++ b/libstdc++-v3/include/std/source_location
@@ -43,12 +43,13 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
{
private:
using uint_least32_t = __UINT_LEAST32_TYPE__;
+ using __builtin_ret_type = decltype(__builtin_source_location());
public:
// [support.srcloc.cons], creation
static consteval source_location
- current(const void* __p = __builtin_source_location()) noexcept
+ current(__builtin_ret_type __p = __builtin_source_location()) noexcept
{
source_location __ret;
__ret._M_impl = static_cast <const __impl*>(__p);
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug libstdc++/104602] std::source_location::current uses cast from void*
2022-02-19 3:31 [Bug libstdc++/104602] New: std::source_location::current uses cast from void* foom at fuhm dot net
` (5 preceding siblings ...)
2022-02-21 21:06 ` redi at gcc dot gnu.org
@ 2022-02-24 23:46 ` cvs-commit at gcc dot gnu.org
2022-02-24 23:47 ` redi at gcc dot gnu.org
` (3 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-02-24 23:46 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104602
--- Comment #7 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Jonathan Wakely <redi@gcc.gnu.org>:
https://gcc.gnu.org/g:41cbcf53dc60b7434b9ee059b3a734a47f5bf212
commit r12-7379-g41cbcf53dc60b7434b9ee059b3a734a47f5bf212
Author: Jonathan Wakely <jwakely@redhat.com>
Date: Thu Feb 24 21:33:44 2022 +0000
libstdc++: Fix cast in source_location::current() [PR104602]
This fixes a problem for Clang, which is going to return a non-void
pointer from __builtin_source_location(). The current definition of
std::source_location::current() converts that to void* and then has to
cast it back again in the body (which makes it invalid in a constant
expression). By using the actual type of the returned pointer, we avoid
the problematic cast for Clang.
libstdc++-v3/ChangeLog:
PR libstdc++/104602
* include/std/source_location (source_location::current): Use
deduced type of __builtin_source_location().
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug libstdc++/104602] std::source_location::current uses cast from void*
2022-02-19 3:31 [Bug libstdc++/104602] New: std::source_location::current uses cast from void* foom at fuhm dot net
` (6 preceding siblings ...)
2022-02-24 23:46 ` cvs-commit at gcc dot gnu.org
@ 2022-02-24 23:47 ` redi at gcc dot gnu.org
2022-02-24 23:48 ` redi at gcc dot gnu.org
` (2 subsequent siblings)
10 siblings, 0 replies; 12+ messages in thread
From: redi at gcc dot gnu.org @ 2022-02-24 23:47 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104602
Jonathan Wakely <redi at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Target Milestone|--- |12.0
Resolution|--- |FIXED
--- Comment #8 from Jonathan Wakely <redi at gcc dot gnu.org> ---
I hope this works for you now, please reopen if not.
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug libstdc++/104602] std::source_location::current uses cast from void*
2022-02-19 3:31 [Bug libstdc++/104602] New: std::source_location::current uses cast from void* foom at fuhm dot net
` (7 preceding siblings ...)
2022-02-24 23:47 ` redi at gcc dot gnu.org
@ 2022-02-24 23:48 ` redi at gcc dot gnu.org
2022-07-07 23:31 ` cvs-commit at gcc dot gnu.org
2022-07-07 23:33 ` redi at gcc dot gnu.org
10 siblings, 0 replies; 12+ messages in thread
From: redi at gcc dot gnu.org @ 2022-02-24 23:48 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104602
--- Comment #9 from Jonathan Wakely <redi at gcc dot gnu.org> ---
Hmm, we probably want to backport this, otherwise this check will start to pass
for future clang versions:
#if __cplusplus > 201703L && __has_builtin(__builtin_source_location)
but the code will be invalid.
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug libstdc++/104602] std::source_location::current uses cast from void*
2022-02-19 3:31 [Bug libstdc++/104602] New: std::source_location::current uses cast from void* foom at fuhm dot net
` (8 preceding siblings ...)
2022-02-24 23:48 ` redi at gcc dot gnu.org
@ 2022-07-07 23:31 ` cvs-commit at gcc dot gnu.org
2022-07-07 23:33 ` redi at gcc dot gnu.org
10 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-07-07 23:31 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104602
--- Comment #10 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Jonathan Wakely
<redi@gcc.gnu.org>:
https://gcc.gnu.org/g:ed377f0fb3419fceed25ae378aca2efca1a6b79d
commit r11-10115-ged377f0fb3419fceed25ae378aca2efca1a6b79d
Author: Jonathan Wakely <jwakely@redhat.com>
Date: Thu Feb 24 21:33:44 2022 +0000
libstdc++: Fix cast in source_location::current() [PR104602]
This fixes a problem for Clang, which is going to return a non-void
pointer from __builtin_source_location(). The current definition of
std::source_location::current() converts that to void* and then has to
cast it back again in the body (which makes it invalid in a constant
expression). By using the actual type of the returned pointer, we avoid
the problematic cast for Clang.
libstdc++-v3/ChangeLog:
PR libstdc++/104602
* include/std/source_location (source_location::current): Use
deduced type of __builtin_source_location().
(cherry picked from commit 41cbcf53dc60b7434b9ee059b3a734a47f5bf212)
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Bug libstdc++/104602] std::source_location::current uses cast from void*
2022-02-19 3:31 [Bug libstdc++/104602] New: std::source_location::current uses cast from void* foom at fuhm dot net
` (9 preceding siblings ...)
2022-07-07 23:31 ` cvs-commit at gcc dot gnu.org
@ 2022-07-07 23:33 ` redi at gcc dot gnu.org
10 siblings, 0 replies; 12+ messages in thread
From: redi at gcc dot gnu.org @ 2022-07-07 23:33 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104602
Jonathan Wakely <redi at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|12.0 |11.4
--- Comment #11 from Jonathan Wakely <redi at gcc dot gnu.org> ---
Backported for 11.4
^ permalink raw reply [flat|nested] 12+ messages in thread