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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
