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

addr2line updated this revision to Diff 482424. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139919/new/ https://reviews.llvm.org/D139919 Files: clang-tools-extra/clang-tidy/ClangTidyCheck.cpp clang-tools-extra/clang-tidy/ClangTidyCheck.h clang-tools-extra/clang-tidy/readability/I

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

addr2line updated this revision to Diff 482418. addr2line added a comment. finally CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139919/new/ https://reviews.llvm.org/D139919 Files: clang-tools-extra/clang-tidy/ClangTidyCheck.cpp clang-tools-extra/clang-tidy/ClangTidyCheck.h clang

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

addr2line added a comment. In D139919#3991407 , @carlosgalvezp wrote: > Sounds like the change to the YAML parser can potentially affect many other > components other than clang-tidy - should that be done in a separate patch? Only these left. ./clan

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

addr2line updated this revision to Diff 482412. addr2line added a comment. Also in YAMLTraits. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139919/new/ https://reviews.llvm.org/D139919 Files: clang-tools-extra/clang-tidy/ClangTidyCheck.cpp clang-tools-extra/clang-tidy/ClangTidyChe

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

addr2line updated this revision to Diff 482404. addr2line marked an inline comment as done. addr2line added a comment. Herald added subscribers: llvm-commits, hiraditya. Herald added a project: LLVM. It also need to change the return type in YAMLParser. CHANGES SINCE LAST ACTION https://review

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

addr2line marked an inline comment as done. addr2line added a comment. Oops, the check on Debian failed. I'm looking into it... Comment at: clang-tools-extra/clang-tidy/ClangTidyCheck.h:457 - } // namespace tidy } // namespace clang MaskRay wrote: > drop wh

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

addr2line updated this revision to Diff 482395. addr2line added a comment. remove the change to the unrelated lines. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139919/new/ https://reviews.llvm.org/D139919 Files: clang-tools-extra/clang-tidy/ClangTidyCheck.cpp clang-tools-extra/c

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

addr2line added a comment. Sorry, I just saw a lot of changes in the llvm repo that change llvm::Optional to std::optional, like this one . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION h

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

addr2line created this revision. addr2line added a reviewer: MaskRay. addr2line added a project: clang-tools-extra. Herald added subscribers: carlosgalvezp, StephenFan. Herald added a reviewer: njames93. Herald added a project: All. addr2line requested review of this revision. Herald added a subscr