Optimising ARM DMBs

Renato Golin renato.golin at linaro.org
Wed Apr 2 02:10:35 PDT 2014


On 2 April 2014 09:39, Reinoud Elhorst <re302 at cam.ac.uk> wrote:
> 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.

Makes sense.


> 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.

Me neither. ;)

Applied on r205409.

Thanks!

cheers,
--renato



More information about the llvm-commits mailing list