<div dir="ltr">Hi James,<div><br></div><div>> <span style="font-size:13px">And thus, if ORC is at that level (probably is?), I think it should just go in there with all the rest of them.</span></div><div><span style="font-size:13px"><br></span></div><div>ORC is definitely not at that level yet - the attached patch was for a prototype of the APIs and lacks many features. If it goes in, I expect it to change reasonably quickly over the next few weeks/months as features are added and bugs uncovered and fixed. After that I would expect it to calm down a bit, but it'll be a while before it could approach the kind of stability described in the document you posted for review.</div><div><br></div><div> I'll keep an eye on this thread tomorrow and see what other people have to say as they get back from vacation, but I suspect the next step for the ORC APIs will be to go in to llvm/include/llvm-c/unstable. They can live there while they're under development and move to llvm-c if/when we're ready.</div><div><br></div><div>Cheers,</div><div>Lang.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 7, 2015 at 10:41 PM, James Y Knight <span dir="ltr"><<a href="mailto:jyknight@google.com" target="_blank">jyknight@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><span class=""><div><div>On Sep 8, 2015, at 12:55 AM, Eric Christopher via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:</div><blockquote type="cite"><div><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 7, 2015 at 5:37 PM Lang Hames <<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Jauhien,<div><br></div><div>A few people have requested a C API for ORC. I don't think ORC's ready for a stable C API, but I'm not opposed to providing C bindings that will probably be reasonably stable in practice (though with no guarantees). I've actually already knocked up some trivial prototype bindings for Hayden Livingston that could serve as a base (see attached).</div><div><br></div><div>The next question is where unstable bindings should live. Juergen, Eric, anyone else who wants to weigh in: I looked back over the C API thread, but I don't think we settled on a home for this kind of thing. Any thoughts? I could see either introducing a new include directory (something along the lines of include/llvm/llvm-c-bindings) or a new c-bindings project on <a href="http://llvm.org/" target="_blank">llvm.org</a>. The former would put all LLVM developers on the hook for maintaining the bindings, the latter would leave maintenance to users of the bindings project, and any volunteers. I prefer the second option: I don't think core developers should be on the hook for maintaining unstable bindings - that kind of special treatment should be reserved for the stable API.</div><div><br></div></div></blockquote><div><br></div><div>We hadn't figured out a location. I was the one that wanted to move stable to a new directory and Jim wanted to move unstable. I think either will work ultimately, just a matter of personal preference in the "when do we count restart".</div><div><br></div><div>That said, as long as you put a comment at the top saying that the ORC bindings are unstable and put them in a new file I'm not opposed to just putting them in llvm-c for now. I also don't think I'm going to win the argument of "stable moves" so if you'd like to make a new unstable directory I'm perfectly happy with that direction too.</div><div><br></div><div>Thoughts?</div></div></div></div></blockquote><br></div></span><div>+1 to putting it in llvm-c for now (but also for good).</div><div><br></div><div>I really think the existing LLVM-C APIs should all be considered "reasonably stable in practice", not "absolutely 100% stable forever". "Reasonably stable in practice" means to me: if you need to remove a function, then, ok, do that. But make a reasonable effort to avoid doing so first, for users' sake. And don't change the ABI of existing functions.</div><div><br></div><div>And thus, if ORC is at that level (probably is?), I think it should just go in there with all the rest of them.</div><div><br></div><div>BTW, have a patch: <a href="http://reviews.llvm.org/D12685" target="_blank">http://reviews.llvm.org/D12685</a> saying that in more words.</div></div></blockquote></div><br></div>