[PATCH] [Power] Use AtomicExpandPass for fence insertion, and use lwsync where appropriate

[Manuel Klimek]

Did you send another one, or was the one I got the one you originally sent?

On Thu Sep 04 2014 at 9:34:14 AM Hal Finkel <hfinkel at anl.gov> wrote:

> Manuel,
>
> I posted a review to this patch about an hour ago, but I don't think a
> corresponding e-mail made it to the list. Can you please check on it?
>
> Thanks again,
> Hal
>
> ----- Original Message -----
> > From: "Robin Morisset" <morisset at google.com>
> > To: morisset at google.com, jfb at chromium.org
> > Cc: llvm-commits at cs.uiuc.edu
> > Sent: Wednesday, September 3, 2014 6:23:37 PM
> > Subject: [PATCH] [Power] Use AtomicExpandPass for fence insertion,
> and use lwsync where appropriate
> >
> > Hi jfb,
> >
> > This patch makes use of AtomicExpandPass in Power for inserting
> > fences around
> > atomic as part of an effort to remove fence insertion from
> > SelectionDAGBuilder.
> > As a big bonus, it lets us use sync 1 (lightweight sync, often used
> > by the mnemonic
> > lwsync) instead of sync 0 (heavyweight sync) in many cases.
> >
> > I also added a test, as there was no test for the barriers emitted by
> > the Power
> > backend for atomic loads and stores.
> >
> > http://reviews.llvm.org/D5180
> >
> > Files:
> >   include/llvm/IR/IntrinsicsPowerPC.td
> >   lib/Target/PowerPC/PPCISelLowering.cpp
> >   lib/Target/PowerPC/PPCISelLowering.h
> >   lib/Target/PowerPC/PPCInstrInfo.td
> >   lib/Target/PowerPC/PPCTargetMachine.cpp
> >   test/CodeGen/PowerPC/atomics.ll
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> >
>
> --
> Hal Finkel
> Assistant Computational Scientist
> Leadership Computing Facility
> Argonne National Laboratory
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140904/c4632af0/attachment.html>