<p dir="ltr">LGTM, thanks!</p>
<p dir="ltr">It looks like you didn't add -std=gnu++14. Is there a reason for that? It seems useful, even if GCC doesn't support it yet.</p>
<p dir="ltr">Also, I spied a CXX1YCOMPAT macro in one of the tests you updated :)</p>
<div class="gmail_quote">On 19 Aug 2014 08:36, "Aaron Ballman" <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This patch moves us forward so we can use -std=c++14 instead of<br>
-std=c++1y. It similarly fixes up the diagnostic options, and changes<br>
the identifiers away from 1y. While it looks big, it's mostly changing<br>
identifiers, updating diagnostic text, and fixing tests to<br>
accommodate.<br>
<br>
~Aaron<br>
</blockquote></div>