<div dir="ltr">Thanks, I committed this with the parenthesis around the condition in r244513.</div><div class="gmail_extra"><br><div class="gmail_quote">2015-08-10 9:27 GMT-07:00 Juergen Ributzka <span dir="ltr"><<a href="mailto:juergen@apple.com" target="_blank">juergen@apple.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Alex,<br>
<br>
thanks for fixing this.<br>
<br>
<br>
I just have a small nitpick. Please add parenthesis around the condition to increase readability.<br>
<br>
NumDefs = MONum == 0 && MO->isReg() ? NumDefs : 0;<br>
<br>
Besides that LGTM.<br>
<span class="HOEnZb"><font color="#888888"><br>
—Juergen<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
> On Aug 8, 2015, at 11:55 AM, Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com">dexonsmith@apple.com</a>> wrote:<br>
><br>
>> On 2015-Aug-07, at 17:15, Alex L <<a href="mailto:arphaman@gmail.com">arphaman@gmail.com</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> The PATCHPOINT instructions have a single optional defined register operand,<br>
>> but the machine verifier can't verify the optional defined register operands.<br>
><br>
> Seems reasonable to me, but I guess I'm not sure.  Maybe someone familiar with<br>
> patchpoint (Philip? Juergen?) could have a look at this one.<br>
><br>
>> So, this patch makes sure that the machine verifier won't report an error when a PATCHPOINT<br>
>> instruction doesn't have its optional defined register operand.<br>
>><br>
>> This change will allow us to enable the machine verifier for the code generation tests for the<br>
>> patchpoint intrinsics.<br>
>><br>
>> Cheers,<br>
>> Alex<br>
>> <0001-MachineVerifier-Handle-the-optional-def-operand-in-P.patch><br>
><br>
<br>
</div></div></blockquote></div><br></div>