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



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

Reply via email to