I'll try the 2 additional string ref changes first and see how bad it is<br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 22, 2016 at 3:36 PM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">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 class="gmail_msg"><br class="gmail_msg">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 class="gmail_msg"><br class="gmail_msg">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 class="gmail_msg"><br class="gmail_msg">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 class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Thu, Sep 22, 2016 at 3:15 PM Greg Clayton <<a href="mailto:clayborg@gmail.com" class="gmail_msg" target="_blank">clayborg@gmail.com</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" 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></blockquote></div>