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

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 13 01:18:07 PST 2022


carlosgalvezp requested changes to this revision.
carlosgalvezp added a comment.
This revision now requires changes to proceed.

AFAIK it's preferred to use the LLVM types instead of the Standard types:

> When both C++ and the LLVM support libraries provide similar functionality, and there isn’t a specific reason to favor the C++ implementation, it is generally preferable to use the LLVM library

https://llvm.org/docs/CodingStandards.html#c-standard-library


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139919



More information about the cfe-commits mailing list