[PATCH][llvm-c] expose fence instruction to the C API

Filip Pizlo fpizlo at apple.com
Tue Nov 19 11:50:15 PST 2013


New patch!

-------------- next part --------------
A non-text attachment was scrubbed...
Name: build-fence.patch
Type: application/octet-stream
Size: 4055 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131119/c7315f7a/attachment.obj>
-------------- next part --------------



On Nov 19, 2013, at 11:30 AM, Peter Zotov <whitequark at whitequark.org> wrote:

> Filip Pizlo писал 19.11.2013 23:22:
>> Hi all!
>> This is a fairly trivial patch to expose the fence instruction to the C API.
> 
> 1) You forgot to call map_from_llvmordering from LLVMBuildAtomicRMW.

Ooops! Fixed.

> 2) I would accept a Name parameter in the LLVMBuildFence function, as all other
> LLVMBuild* functions have it.

Fixed.

> 3) I've done some similar work in http://llvm-reviews.chandlerc.com/D2176,
> you may want to take a look.
> 
> -- 
>  WBR, Peter Zotov.



More information about the llvm-commits mailing list