Did you send another one, or was the one I got the one you originally sent?<div><br><div class="gmail_quote">On Thu Sep 04 2014 at 9:34:14 AM Hal Finkel <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Manuel,<br>
<br>
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?<br>
<br>
Thanks again,<br>
Hal<br>
<br>
----- Original Message -----<br>
> From: "Robin Morisset" <<a href="mailto:morisset@google.com" target="_blank">morisset@google.com</a>><br>
> To: <a href="mailto:morisset@google.com" target="_blank">morisset@google.com</a>, <a href="mailto:jfb@chromium.org" target="_blank">jfb@chromium.org</a><br>
> Cc: <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> Sent: Wednesday, September 3, 2014 6:23:37 PM<br>
> Subject: [PATCH] [Power] Use AtomicExpandPass for fence insertion,    and use lwsync where appropriate<br>
><br>
> Hi jfb,<br>
><br>
> This patch makes use of AtomicExpandPass in Power for inserting<br>
> fences around<br>
> atomic as part of an effort to remove fence insertion from<br>
> SelectionDAGBuilder.<br>
> As a big bonus, it lets us use sync 1 (lightweight sync, often used<br>
> by the mnemonic<br>
> lwsync) instead of sync 0 (heavyweight sync) in many cases.<br>
><br>
> I also added a test, as there was no test for the barriers emitted by<br>
> the Power<br>
> backend for atomic loads and stores.<br>
><br>
> <a href="http://reviews.llvm.org/D5180" target="_blank">http://reviews.llvm.org/D5180</a><br>
><br>
> Files:<br>
>   include/llvm/IR/<u></u>IntrinsicsPowerPC.td<br>
>   lib/Target/PowerPC/<u></u>PPCISelLowering.cpp<br>
>   lib/Target/PowerPC/<u></u>PPCISelLowering.h<br>
>   lib/Target/PowerPC/<u></u>PPCInstrInfo.td<br>
>   lib/Target/PowerPC/<u></u>PPCTargetMachine.cpp<br>
>   test/CodeGen/PowerPC/atomics.<u></u>ll<br>
><br>
> ______________________________<u></u>_________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
><br>
<br>
--<br>
Hal Finkel<br>
Assistant Computational Scientist<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
</blockquote></div></div>