[PATCH] D17283: Deprecate LLVMGetDataLayout and replace it by LLVMGetDataLayoutStr

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 15 16:14:47 PST 2016


joker.eph accepted this revision.
joker.eph added a comment.
This revision is now accepted and ready to land.

LGTM with a mention in the "Release Notes" in the same commit please.


================
Comment at: include/llvm-c/Core.h:487
@@ +486,3 @@
+ * LLVMGetDataLayout is DEPRECATED, as the name is not only incorrect,
+ * but match the name of another method on the module. Prefer the use
+ * of LLVMGetDataLayoutStr, which is not ambiguous.
----------------
Add a note in the release notes


http://reviews.llvm.org/D17283





More information about the llvm-commits mailing list