Optimising ARM DMBs

Reinoud Elhorst re302 at cam.ac.uk
Tue Apr 1 14:35:18 PDT 2014


No objections from my side, everything works still as expected with that
change, new patch attached.


On Tue, Apr 1, 2014 at 12:24 PM, Tim Northover <t.p.northover at gmail.com>wrote:

> Hi Reinoud,
>
> On 1 April 2014 11:35, Renato Golin <renato.golin at linaro.org> wrote:
> > The patch looks good to me, but I'd rather wait for other people's
> > views on the atomics side (Tim?).
>
> I've taken a look at the patch, and I think the only suggestion I'd
> have is to put this pass before ConstantIslands. It's probably OK
> because you're only removing instructions, but ConstantIslands does
> some very delicate computations on reachability of instruction offsets
> and the less that happens afterwards, the better.
>
> Cheers.
>
> Tim.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140401/188117cf/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ARMBarriers_1.patch
Type: application/octet-stream
Size: 6841 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140401/188117cf/attachment.obj>


More information about the llvm-commits mailing list