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

Reply via email to