[PATCH] D18964: LLVM C API. Deprecated functions

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 11 11:12:28 PDT 2016


We do already have a place, but I wasn't sure whether or not we wanted a
separate implementation.

On Mon, Apr 11, 2016, 11:09 AM Mehdi AMINI <mehdi.amini at apple.com> wrote:

> joker.eph added inline comments.
>
> ================
> Comment at: include/llvm-c/Definitions.h:27
> @@ +26,2 @@
> +  decl
> +#endif
> ----------------
> Do we have to leave this in  `include/llvm-c`? Could it be somewhere in
> `include/llvm`? We may already have a place for such macros.
> 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.
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D18964
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160411/6aee7b38/attachment.html>


More information about the llvm-commits mailing list