llvm-c patch for atomicrmw & dibuilder

Eric Christopher echristo at gmail.com
Mon Apr 22 16:14:36 PDT 2013


OK, I've committed my patch. You'll need to rebase and make some
changes to your current set of patches so that we keep the C++ out of
the C headers.

-eric

On Mon, Apr 22, 2013 at 6:24 PM, Eric Christopher <echristo at gmail.com> wrote:
> I will, but I'd appreciate if you could hold off for a few hours. I'm
> removing some c++ dependencies in the c headers and would like to
> finish that before we add any more of them :)
>
> -eric
>
> On Mon, Apr 22, 2013 at 10:22 AM, Chris Lattner <clattner at apple.com> wrote:
>>
>> 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



More information about the llvm-commits mailing list