Optimising ARM DMBs

Renato Golin renato.golin at linaro.org
Tue Apr 1 03:35:52 PDT 2014


On 1 April 2014 09:17, Reinoud Elhorst <re302 at cam.ac.uk> wrote:
> I discussed the patch off-list with several people, but am more than happy
> to get some more feedback.

Hi Reinoud,

The patch looks good to me, but I'd rather wait for other people's
views on the atomics side (Tim?).

Though, I think the IsRemovableNextDMB variable is slightly misnamed.
It actually means "have I seen a DMB before" instead of "you can
remove the next DMB". So, maybe a name like PreviousIsDMB would be
more appropriate.

cheers,
--renato



More information about the llvm-commits mailing list