[PATCH] D18736: [llvm-c] Improve IR Introspection: Add LLVM{Get, Set}ModuleIdentifier

whitequark via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 4 12:58:52 PDT 2016


whitequark added inline comments.

================
Comment at: lib/IR/Core.cpp:167
@@ +166,3 @@
+void LLVMSetModuleIdentifier(LLVMModuleRef M, const char *Ident, size_t Len) {
+  unwrap(M)->setModuleIdentifier({Ident, Len});
+}
----------------
Actually, can you call `StringRef` constructor here? I'm pretty sure I've never seen brace-initializer style anywhere else in LLVM.


http://reviews.llvm.org/D18736





More information about the llvm-commits mailing list