[Lldb-commits] [PATCH] D139973: [llvm] Make llvm::Any similar to std::any

[Sergei Barannikov via Phabricator via lldb-commits]

barannikov88 accepted this revision.
barannikov88 added a comment.

In D139973#4005120 <https://reviews.llvm.org/D139973#4005120>, @sebastian-ne wrote:

> the question gets wether we want to keep llvm::Any around as a wrapper of std::any once we can use it (in this case this patch would be obsolete)
> or if we we want to remove llvm::Any and use std::any only.

Since there are not many uses of Any, it is hard to say which option would be preferable in the long term.
Perhaps, it would be possible to remove llvm::Any and use std::any with llvm::any_isa as an extension.
Anyway, this patch makes it possible to avoid double casting, which is an improvement by itself.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139973