[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