llvm-c patch for atomicrmw & dibuilder

Eric Christopher echristo at gmail.com
Mon Apr 22 10:24:16 PDT 2013


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