<p dir="ltr">Please move the default out of the switch. Otherwise, looks fine. Thanks for the pch test!</p>
<div class="gmail_quote">On Jan 21, 2013 4:54 AM, "Tim Northover" <<a href="mailto:t.p.northover@gmail.com">t.p.northover@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Richard,<br>
<br>
> The AST dumper is a bit of a special case. It's intended to be usable<br>
> by running Expr->dump() from a debugger, and consequently it does not<br>
> have access to the ASTContext.<br>
<br>
I thought it might be, which is why I stopped there.<br>
<br>
> If this isn't working out well, how about replacing the IsIEEE flag in<br>
> FloatingLiteralBitfields with a float kind enumeration?<br>
<br>
Now that seems to have worked better; thanks for the suggestion. I've<br>
attached the updated Clang patch, together with its LLVM counterpart<br>
for reference.<br>
<br>
Does it look reasonable to you?<br>
<br>
Tim.<br>
</blockquote></div>