OK - thanks for the context!
On Mon, Nov 18, 2019 at 5:07 PM Ben Langmuir wrote:
> Nice find.
>
> The change LGTM, and is clearly what I always intended it to do. I was
> confused how this ever worked at all, but I see that (at least in libc++)
> we move the values into place rather than swap t
Nice find.
The change LGTM, and is clearly what I always intended it to do. I was
confused how this ever worked at all, but I see that (at least in libc++) we
move the values into place rather than swap them, so the values at the end are
still the largest values for types with a trivial move c
Hey Ben - if you're still involved with this part of the project - could
you check that this change (to code you committed back in 2014, r215810) is
correct & add a test case if possible?
On Thu, Nov 7, 2019 at 10:39 AM Reid Kleckner via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
>
> Author
Author: Reid Kleckner
Date: 2019-11-07T10:39:29-08:00
New Revision: dd870f6929ee0b1dfb5fd000c7b826a1bd2d2571
URL:
https://github.com/llvm/llvm-project/commit/dd870f6929ee0b1dfb5fd000c7b826a1bd2d2571
DIFF:
https://github.com/llvm/llvm-project/commit/dd870f6929ee0b1dfb5fd000c7b826a1bd2d2571.diff
