[PATCH] [X86] Make wide loads be managed by AtomicExpand

Robin Morisset morisset at google.com
Thu Sep 18 15:52:34 PDT 2014


Hi jfb,

AtomicExpand already had logic for expanding wide loads and stores on LL/SC
architectures, and for expanding wide stores on CmpXchg architectures, but
not for wide loads on CmpXchg architectures. This patch fills this hole,
and makes use of this new feature in the X86 backend.

Only one functionnal change: we now lose the SynchScope attribute.
It is regrettable, but I have another patch that I will submit soon that will
solve this for all of AtomicExpand (it seemed better to split it apart as it
is a different concern).

http://reviews.llvm.org/D5404

Files:
  lib/CodeGen/AtomicExpandPass.cpp
  lib/Target/X86/X86ISelLowering.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5404.13854.patch
Type: text/x-patch
Size: 4420 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140918/a2cd71d9/attachment.bin>


More information about the llvm-commits mailing list