<div dir="ltr">Thanks for the review.  Committed as r213574.<br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 21, 2014 at 8:37 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron.ballman@gmail.com" target="_blank">aaron.ballman@gmail.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>> +      std::string DiagnosticName = "#pragma unroll";<br>
</div><div>
> +      return DiagnosticName + getValueString();<br>
<br>
</div>No need for the std::string declaration.<br></blockquote><div><br></div><div>Done.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

> +    } else {<br>
<div>> +      assert(SpellingIndex == Pragma_clang_loop && "Unexpected spelling");<br>
</div><div>> +      std::string DiagnosticName = getOptionName(option);<br>
> +      return DiagnosticName + getValueString();<br>
<br>
</div>Same here.<br></blockquote><div><br></div><div>Done.  Wrapped getOptionName in std::string to make operator+ work. </div><div><br></div><div>Mark</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">



> +    }<br>
<br>
LGTM!<br>
<span><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div></div>