[PATCH] D45346: [LLVM-C] Audit Inline Assembly APIs for Consistency

[whitequark via Phabricator via llvm-commits]

whitequark added inline comments.


================
Comment at: include/llvm-c/Core.h:684
+LLVMValueRef LLVMGetInlineAsm(LLVMTypeRef Ty,
+                              char *AsmString, char *Constraints,
+                              LLVMBool HasSideEffects, LLVMBool IsAlignStack,
----------------
whitequark wrote:
> I think this should be `const char *AsmString, size_t AsmStringSize, const char *Constraints, size_t ConstraintsSize`.
Or rather `const char *AsmString, size_t AsmStringLen, const char *Constraints, size_t ConstraintsLen`.


https://reviews.llvm.org/D45346