jeffreytan81 wrote: > LGTM. It may be a good idea to move these APIs over to use std::optional > instead of taking a default value so we can use 'value_or' instead of having > to write logic to handle the default value through a parameter.
My goal is fixing the regression in this diff. I will let @walter-erquinigo to comment/follow-up on this suggestion. https://github.com/llvm/llvm-project/pull/72902 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits