[PATCH] D139919: use std::optional in ClangTidyCheck

[Carlos Galvez via Phabricator via cfe-commits]

carlosgalvezp added a comment.

In D139919#3991247 <https://reviews.llvm.org/D139919#3991247>, @addr2line wrote:

> Sorry, I just saw a lot of changes in the llvm repo that change llvm::Optional to std::optional, like this one <https://github.com/llvm/llvm-project/commit/67819a72c6ba39267effe8edfc1befddc3f3f2f9>.

Interesting, then I don't know, perhaps the guidelines have changed? Would be good to get input from the other reviewers


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139919