<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Apr 20, 2013, at 5:54 AM, Carlo Kok <<a href="mailto:ck@remobjects.com">ck@remobjects.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Op 20-4-2013 1:21, Eric Christopher schreef:<br><blockquote type="cite">Mind splitting this up into two patches please?<br><br>Thanks!<br><br>-eric<br><br>On Fri, Apr 19, 2013 at 3:42 PM, Carlo Kok <<a href="mailto:ck@remobjects.com">ck@remobjects.com</a>> wrote:<br><blockquote type="cite">Attached is a patch that adds AtomicRMW and DIBuilder to llvm-c. Code is<br>tested and works quite well for me.<br><br></blockquote></blockquote><br>Split in two, attached<br></div></blockquote></div><br><div>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.</div><div><br></div><div>-Chris</div></body></html>