[PATCH] D155890: [clang-tidy] Add folly::Optional to unchecked-optional-access

Anton Dukeman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 24 10:28:15 PDT 2023


adukeman marked 3 inline comments as done.
adukeman added a comment.

Thanks for all the comments! I think I've got them all addressed and tests are passing locally.

In D155890#4528560 <https://reviews.llvm.org/D155890#4528560>, @carlosgalvezp wrote:

> The review is marked as accepted, should we land it? Let me know if you need help with that :)

Yes, will need help with that since I don't have commit access. Anybody with commit access, feel free to land this for me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155890



More information about the cfe-commits mailing list