[PATCH] [Power] Use AtomicExpandPass for fence insertion, and use lwsync where appropriate
klimek at google.com
Thu Sep 4 10:37:05 PDT 2014
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:
> 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,
> ----- 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...
More information about the llvm-commits