<div dir="ltr">OK to commit?<div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Sep 8, 2013 at 5:52 PM, Virgile Bello <span dir="ltr"><<a href="mailto:virgile.bello@gmail.com" target="_blank">virgile.bello@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Thinking about it again, including <atomic> from LLDB headers has some drawbacks:<div>- any project including LLDB to also be C++11 (not sure if LLDB includes should support usage from non C++11 projects?)<div>



- also for Win32, it prevents including LLDB headers in any CLR (C++/CLI) project since <atomic> is not CLR compatible (happening in my case when writing a LLDB MSVC debugger).</div><div><br></div><div>Actually I found out that on Windows, there is <intrin.h> that provides _InterlockedIncrement without pulling the whole <Windows.h>, so it wouldn't be a problem anymore to keep it in .h, factorized in a common LLDB header, similar to LLVM Atomic.h.<br>



</div><div><br></div><div>Patch is attached. Let me know your thoughts.</div><span class="HOEnZb"><font color="#888888"><div><div><br></div><div>Virgile</div><div><br></div></div></font></span></div></div><div class="HOEnZb">

<div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 6, 2013 at 5:09 AM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Looks good.<br>
<br>
On Sep 5, 2013, at 5:48 AM, Virgile Bello <<a href="mailto:virgile.bello@gmail.com" target="_blank">virgile.bello@gmail.com</a>> wrote:<br>
<br>
> <lldb-atomic.patch><br>
<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div></div>