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

Hal Finkel hfinkel at anl.gov
Thu Sep 4 10:37:42 PDT 2014


----- Original Message -----
> From: "Robin Morisset" <morisset at google.com>
> To: morisset at google.com, jfb at chromium.org
> Cc: hfinkel at anl.gov, llvm-commits at cs.uiuc.edu
> Sent: Thursday, September 4, 2014 12:34:15 PM
> Subject: Re: [PATCH] [Power] Use AtomicExpandPass for fence insertion, and use lwsync where appropriate
> 
> Fix formatting based on hfinkel comments.
> 
> About replacing the BookE flag by a special flag for the sync
> instruction,
> should I try doing it in this patch, or is it best as a separate
> thing ?

Let's do that in a follow-up commit. Please add a FIXME in this patch so we remember to do it.

Thanks again,
Hal

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

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory



More information about the llvm-commits mailing list