OptionValueString is actually the next item on my list.  I tried it before this and it was a very big CL, But maybe with this done it will be smaller.<br><br>I can try again but if the cl grows too large i think it's better to do it in a followup.  If nothing else so that if a buildbot fails it's easier to track it down with a more modest sized cl.<br><br>For the printing stuff, it might be cleaner to just use an llvm stream and then pass it to Error::SetErrorString.  The %*s macros will look kind of gross (for the same reason that the PRIx64 style macros look so horrible).<br><br>But all these things combined will make a really big CL, I'd rather do it in chunks.  Maybe format strings next then OptionValueString?<br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 22, 2016 at 3:15 PM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg added a comment.<br class="gmail_msg">
<br class="gmail_msg">
I am not saying we have to do the printf changes, I was just seeing what you think. I would like to see the StringRef variants of functions put in as part of this.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D24847" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D24847</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>