llvm-c patch for atomicrmw & dibuilder

Chris Lattner clattner at apple.com
Mon Apr 22 10:22:58 PDT 2013


On Apr 20, 2013, at 5:54 AM, Carlo Kok <ck at remobjects.com> wrote:

> Op 20-4-2013 1:21, Eric Christopher schreef:
>> Mind splitting this up into two patches please?
>> 
>> Thanks!
>> 
>> -eric
>> 
>> On Fri, Apr 19, 2013 at 3:42 PM, Carlo Kok <ck at remobjects.com> wrote:
>>> Attached is a patch that adds AtomicRMW and DIBuilder to llvm-c. Code is
>>> tested and works quite well for me.
>>> 
> 
> Split in two, attached

The AtomicRMW patch looks fine, but please fit in 80 columns and add doxygen comments to the enums.  Someone else will need to review the DIBuilder patch.

-Chris
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130422/81abb740/attachment.html>


More information about the llvm-commits mailing list