[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