On 20/09/16 11:03 -0700, Tim Shen wrote:
On Mon, Sep 19, 2016 at 1:06 PM, Tim Shen <tims...@google.com> wrote:
I believe that it's a "typo" from me - it should be
is_trivially_destructible, not is_default_constructible (so that we
can avoid using aligned_storage in the corresponding specialization).
is_literal_type works, since literal types are trivially destructible.


Sorry I misunderstood your patch.

The underlying problem is that _Variant_storage wasn't always default
constructible, but it should be.

Jon, your fix doesn't fix the following constexpr variation of your test case:
 struct X {
   constexpr X() { }
 };

 constexpr std::variant<X> v1 = X{};

So I have a patch here to make _Variant_storage's internal union
default constructible.

Tested on x86_64-linux-gnu.

THanks, OK for trunk.

Reply via email to