[PATCH] LLDB formatter for llvm::Optional

Justin Bogner mail at justinbogner.com
Thu Feb 19 18:58:55 PST 2015


Enrico Granata <egranata at apple.com> writes:
> Instead of str(), one could use .GetSummary() on the casted SBValue. And if
> that returns None, fallback to GetValue(). IIRC, str(SBValue) does a little
> more than necessary.

I tried changing this to use GetSummary and fall back to GetValue, but
they often both return None. I guess the extra work str() does is
necessary after all.

I've committed this in r229956.



More information about the llvm-commits mailing list