<div dir="ltr">Sorry Renato, I indeed skipped over your remark.<div><br></div><div>It's not entirely the same. A DMB is removable if the previous statement was a DMB, or there has not been an instruction since the last DMB that warrants a new DMB. PreviousIsDMB seems to suggest that we only look at the previous instruction.</div>

<div><br></div><div>> <span style="font-family:arial,sans-serif;font-size:13px">It actually means "have I seen a DMB before" instead of "you can </span><span style="font-family:arial,sans-serif;font-size:13px">remove the next DMB"</span></div>

<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">Actually it means "can I remove this instruction if it is a DMB", but I see your point that IsRemovableNextDMB seems to make a promise towards any subsequent DMB.</span></div>

<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">How about "seenDMBSinceLastMemoryAccess"? Although this encodes the logic (that we can ignore instructions as far as dmbs are concerned based on whether they have memory access) in the variable name, I don't really like that either.</span></div>

<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">Claude</span></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">

On Wed, Apr 2, 2014 at 9:03 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="">On 2 April 2014 07:23, Tim Northover <<a href="mailto:t.p.northover@gmail.com">t.p.northover@gmail.com</a>> wrote:<br>
> I think this patch looks good now too. Do you have commit access or<br>
> would you like me to do it?<br>
<br>
</div>I don't think he does. Reinoud, do you mind if we rename<br>
IsRemovableNextDMB to PreviousIsDMB before commit?<br>
<span class="HOEnZb"><font color="#888888"><br>
--renato<br>
</font></span></blockquote></div><br></div>