Subject: Re: [boost] [variant] Heads-up!
From: Steven Watanabe (watanabesj_at_[hidden])
Date: 2019-04-29 18:08:39
On 4/29/19 12:04 PM, Antony Polukhin wrote:
> Ð¿Ð½, 29 Ð°Ð¿Ñ. 2019 Ð³. Ð² 20:44, Steven Watanabe via Boost <boost_at_[hidden]>:
>> On 4/29/19 11:30 AM, Antony Polukhin via Boost wrote:
>>> I've merged a very cool optimization by Nikita Kniazev into the master
>>> branch. From now on boost::variant does pointer stealing for recursive
>>> This significantly improves the performance of the variants move constructors.
>>> However if you use a variant variable after the std::move for anything
>>> except destruction and assignment then you're getting an UB. Beware!
>> boost::variant goes to great lengths to prevent
>> exactly this situation. You just broke it. This
>> change is unacceptable. Please revert it. This
>> optimization can be used iff. you have a way to
>> construct a valid object in the rhs.
> You can restore the old slow pre-rvalue era behavior by defining
That doesn't make it okay. Look, this optimization
would be fine with a different variant, one that doesn't
provide the never-empty guarantee. That isn't boost::variant.
This isn't some simple optimization. It's a major breaking
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk