[PATCH] D147991: [LLVM][Casting.h] Fix dyn_cast for std::unique_ptr.

[Aleksandr Bezzubikov via Phabricator via llvm-commits]

zuban32 added inline comments.


================
Comment at: llvm/unittests/Support/Casting.cpp:232
+  EXPECT_NE(BP2.get(), nullptr);
+  BP2.release();
 }
----------------
bzcheeseman wrote:
> I don't believe this is necessary? I think we want it to be deallocated.
I was just following the chunk above, I think gtest reports some memory issue should we omit the `release()`. I don't remember why, let me take a look


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147991/new/

https://reviews.llvm.org/D147991