[Lldb-commits] [PATCH] D26971: Introduce chrono to more gdb-remote functions

[Zachary Turner via lldb-commits]

zturner added a comment.

In https://reviews.llvm.org/D26971#602738, @clayborg wrote:

> It would be nice if we can find a way to be able to use any time units we want when specifying the timeout. I would rather not have everyone have to convert their times to microseconds. Maybe we just add some overloads to the public facing functions and have just one internal function that uses llvm::Optional.
>
> As a side note I would also like us to fix llvm::Optional so it can be debugged. Right now if you expand an llvm::Optional<T> you see the an array of characters instead of your type T which means when debugging you can't see your type's value. Boost correctly uses C++11 unions so that we can see the type when expanded in the debugger.


Couldn't someone just add a data formatter for llvm::Optional<> to lldb?


https://reviews.llvm.org/D26971