<div dir="ltr">I actually think it can just be included. It doesn't include any other files and seems to be C only.<div><br></div><div>-eric</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Apr 11, 2016 at 11:30 AM Alex Denisov <<a href="mailto:1101.debian@gmail.com">1101.debian@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">AlexDenisov added a comment.<br>
<br>
In <a href="http://reviews.llvm.org/D18964#397364" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18964#397364</a>, @echristo wrote:<br>
<br>
> We do already have a place, but I wasn't sure whether or not we wanted a<br>
>  separate implementation.<br>
<br>
<br>
This is exactly my intention. I copied the definition from `llvm/Support/Compiler.h`, but besides that it contains a lot of other stuff, that might not be suitable for C API (e.g. sanitizers).<br>
Another possible solution is to introduce another header and include it into both `llvm/Support/Compiler.h` and C API interface.<br>
<br>
> Also sometimes we keep alive some C++ APIs just for the purpose of supporting a C API, I think we should mark these as deprecated as well so we know we can remove them.<br>
<br>
<br>
This is a very good idea, I will skim through the implementations and check whether we can deprecate something there.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D18964" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18964</a><br>
<br>
<br>
<br>
</blockquote></div>