On Sat, Feb 23, 2013 at 6:39 PM, Sean Silva <span dir="ltr"><<a href="mailto:silvas@purdue.edu" target="_blank">silvas@purdue.edu</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">-      else if (const VariableArrayType *VAT = cast<VariableArrayType>(AT))<br>
<br>
</div>Isn't this tautological? I thought that cast<> required its argument<br>
to be nonnull (and hence this would always be true (or otherwise UB)).<br>
If not, then the existence of cast_or_null<> seems suspect.</blockquote><div><br></div><div>Yes, it was supposed to be a dyn_cast. That's the bug this change fixed. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 </blockquote></div>