<div><div dir="auto">What about the static asserts protecting a Log call and another in the parser?</div><br><div class="gmail_quote"><div>On Wed, Jul 19, 2017 at 2:26 PM Krzysztof Parzyszek <<a href="mailto:kparzysz@codeaurora.org">kparzysz@codeaurora.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 7/19/2017 4:18 PM, Craig Topper wrote:<br>
> LaneMask isn't as self contained as it should be. 64 bits is enough<br>
> here. The problem is accidental leaking of the current size.<br>
><br>
> For example there was a hard coded compare with 32 in tablegen until I<br>
> fixed it recently.<br>
<br>
This is most likely the only example.  I actually tested it with<br>
uint64_t (but without exceeding 32 lanes).<br>
<br>
-Krzysztof<br>
<br>
--<br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,<br>
hosted by The Linux Foundation<br>
</blockquote></div></div><div dir="ltr">-- <br></div><div data-smartmail="gmail_signature">~Craig</div>