* [PATCH] Limit workaround for Clang bug to __clang_major__ <= 7
@ 2018-09-13 14:01 Jonathan Wakely
0 siblings, 0 replies; only message in thread
From: Jonathan Wakely @ 2018-09-13 14:01 UTC (permalink / raw)
To: libstdc++, gcc-patches
[-- Attachment #1: Type: text/plain, Size: 343 bytes --]
The bug https://bugs.llvm.org/show_bug.cgi?id=33222 is now fixed on
Clang trunk, so the workaround won't be needed for Clang 8.0 and later.
* include/std/variant (variant) [__clang__]: Limit workaround to
Clang 7 and older.
Tested x86_64-linux, committed to trunk.
I hope somebody will check with Clang trunk soon and verify this
works.
[-- Attachment #2: patch.txt --]
[-- Type: text/plain, Size: 916 bytes --]
commit 6112f62b32c84edd2e2369116ad233f6fef5b585
Author: Jonathan Wakely <jwakely@redhat.com>
Date: Thu Sep 13 14:30:05 2018 +0100
Limit workaround for Clang bug to __clang_major__ <= 7
The bug https://bugs.llvm.org/show_bug.cgi?id=33222 is now fixed on
Clang trunk, so the workaround won't be needed for Clang 8.0 and later.
* include/std/variant (variant) [__clang__]: Limit workaround to
Clang 7 and older.
diff --git a/libstdc++-v3/include/std/variant b/libstdc++-v3/include/std/variant
index 2d86a704c63..5a77e9e2d84 100644
--- a/libstdc++-v3/include/std/variant
+++ b/libstdc++-v3/include/std/variant
@@ -1296,7 +1296,7 @@ namespace __variant
#undef _VARIANT_RELATION_FUNCTION_TEMPLATE
-#ifdef __clang__
+#if defined(__clang__) && __clang_major__ <= 7
public:
using _Base::_M_u; // See https://bugs.llvm.org/show_bug.cgi?id=31852
private:
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2018-09-13 13:49 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-13 14:01 [PATCH] Limit workaround for Clang bug to __clang_major__ <= 7 Jonathan Wakely
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).