[PATCH] D18964: LLVM C API. Deprecated functions

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 11 11:09:31 PDT 2016


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





More information about the llvm-commits mailing list