mclow.lists accepted this revision.
mclow.lists added a comment.
This revision is now accepted and ready to land.

The rest of this LGTM.



================
Comment at: 
test/std/numerics/numeric.ops/transform.reduce/transform_reduce_iter_iter_iter_init.pass.cpp:61
 
+inline MoveOnly operator+(const MoveOnly& lhs, const MoveOnly& rhs)
+{
----------------
BillyONeal wrote:
> Should these go in MoveOnly.h ?
I think so. While you're at it, you can remove the `friend` declaration in 
`MoveOnly` - that's a remnant.


Repository:
  rL LLVM

https://reviews.llvm.org/D41372



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D41372: [l... Billy Robert O'Neal III via Phabricator via cfe-commits
    • [PATCH] D4137... Billy Robert O'Neal III via Phabricator via cfe-commits
    • [PATCH] D4137... Billy Robert O'Neal III via Phabricator via cfe-commits
    • [PATCH] D4137... Marshall Clow via Phabricator via cfe-commits
    • [PATCH] D4137... Billy Robert O'Neal III via Phabricator via cfe-commits
    • [PATCH] D4137... Billy Robert O'Neal III via Phabricator via cfe-commits

Reply via email to