DavidSpickett added a comment.

> Are we going to use C++20 or something else? But I see the function 
> transform() we need was introduced in C++23.

Good point, it would be c++20. If you want to look into the status of the 
switchover feel free, but you can leave them as llvm::Optional for now.

If someone does get around to changing this code to std::optional they will be 
adding you on review anyway.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140032

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to