[PATCH] D139974: [llvm] Change llvm::Any to wrap std::any

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 13 15:07:32 PST 2022


dblaikie added a comment.

In D139974#3993357 <https://reviews.llvm.org/D139974#3993357>, @aeubanks wrote:

> doesn't this basically doom anybody not using MSVC 2022 17.4? I don't think conditionally using something that may be buggy makes sense if we claim to support older MSVC versions

+1 to that. If we've got to keep the whole implementation around anyway, best we use it all the time than use it in only some configurations, lest it bitrot/become undermaintained.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139974



More information about the llvm-commits mailing list