kazu added a comment. In D131368#3708542 <https://reviews.llvm.org/D131368#3708542>, @dblaikie wrote:
> Sounds good - I'd be probably OK with removing these outright as they're so > niche/generally only for use by `llvm::Optional` anyway - but not much harm > in keeping them around for the same period the other more visible APIs are > deprecated and cleaning it all up together later/then. Yeah, I was actually sitting on the fence, wondering whether I should just remove them now as they are internal details of `llvm::Optional` as you point out. I think I'll just remove them in three months or so along with other deprecated methods related to `llvm::Optional`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131368/new/ https://reviews.llvm.org/D131368 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
