From b0b5d389635a54c5aeb490b1b067f3a38e038235 Mon Sep 17 00:00:00 2001 From: Reid Kleckner Date: Tue, 15 Dec 2020 15:59:21 -0800 Subject: [PATCH] Document that AlignedCharArrayUnion exists to work around an MSVC bug Differential Revision: https://reviews.llvm.org/D93355 --- llvm/include/llvm/Support/AlignOf.h | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/llvm/include/llvm/Support/AlignOf.h b/llvm/include/llvm/Support/AlignOf.h index f9dcde4d4ff1..f586d7f182aa 100644 --- a/llvm/include/llvm/Support/AlignOf.h +++ b/llvm/include/llvm/Support/AlignOf.h @@ -20,9 +20,10 @@ namespace llvm { /// A suitably aligned and sized character array member which can hold elements /// of any type. /// -/// These types may be arrays, structs, or any other types. This exposes a -/// `buffer` member which can be used as suitable storage for a placement new of -/// any of these types. +/// This template is equivalent to std::aligned_union_t<1, ...>, but we cannot +/// use it due to a bug in the MSVC x86 compiler: +/// https://github.com/microsoft/STL/issues/1533 +/// Using `alignas` here works around the bug. template struct AlignedCharArrayUnion { using AlignedUnion = std::aligned_union_t<1, T, Ts...>; alignas(alignof(AlignedUnion)) char buffer[sizeof(AlignedUnion)];