kazu added a comment. Thanks for the review!
In D138464#3942626 <https://reviews.llvm.org/D138464#3942626>, @MaskRay wrote: > We have `return llvm::None;` and `return None;`. Shall we always prefer > `llvm::None`? Either one is OK (to me). I can deprecate `None` in my personal tree, and get a warning for both `llvm::None` and `None`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138464/new/ https://reviews.llvm.org/D138464 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits