<div dir="ltr">+  if (const EnumType* ET = dyn_cast<EnumType>(T)) {<br><div><br></div><div>* on the right please, and use T->getAs<EnumType>() in case there is type sugar here (there probably can't be for a TemplateArgument that's been resolved to an integral value, but dyn_cast on a type is a red flag that we don't need here).</div><div><br></div><div>Otherwise, LGTM, thanks!</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 11, 2014 at 8:28 PM, Will Wilson <span dir="ltr"><<a href="mailto:will@indefiant.com" target="_blank">will@indefiant.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Richard,
<div><br></div><div>Here's a simple patch that improves pretty printing for templates args specialized to enum constants. If an enum constant isn't available to represent the value it falls back on printing the integer value.</div><div><br></div><div>Tested against latest trunk and includes a new test case and a updated existing test case.</div><div><br></div><div>Let me know if it looks good to go...</div><div><br></div><div>Cheers,</div><div>Will.</div></div>
</blockquote></div><br></div>