[PATCH] D48807: Add llvm::Any

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 16 12:51:25 PDT 2018


zturner added inline comments.


================
Comment at: llvm/unittests/ADT/AnyTest.cpp:114-115
+  // in the process.
+  EXPECT_EQ(8, llvm::any_cast<int>(std::move(E)));
+  EXPECT_TRUE(E.hasValue());
+
----------------
dblaikie wrote:
> Is there a test (didn't spot it) that demonstrates that the result of llvm::any_cast on an rvalue Any is the value moved-from the the Any, and that the Any contains the original move-from?
Is the newly added test in line with what you had in mind?


https://reviews.llvm.org/D48807





More information about the llvm-commits mailing list